Skip to content

[7.6] [SIEM] [Detection Engine] Update status on rule details page (#55201)#55277

Merged
dhurley14 merged 1 commit intoelastic:7.6from
dhurley14:backport/7.6/pr-55201
Jan 18, 2020
Merged

[7.6] [SIEM] [Detection Engine] Update status on rule details page (#55201)#55277
dhurley14 merged 1 commit intoelastic:7.6from
dhurley14:backport/7.6/pr-55201

Conversation

@dhurley14
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.6:

…#55201)

* adds logic for returning / updating status when a rule is switched from enabled to disabled and vice versa.

* update response for find rules statuses to include current status and failures

* update status on demand and on enable/disable

* adds ternary to allow removal of 'let'

* adds savedObjectsClient to the add and upate prepackaged rules and import rules route.

* fix bug where convertToSnakeCase would throw error 'cannot convert null or undefined to object' if passed null

* genericize snake_case converter and updates isAuthorized to snake_case (different situation)

* renaming to 'going to run' instead of executing because when task manager exits because of api key error it won't write the error status so the actual status is 'going to run' on the next interval. This is more accurate than being stuck on 'executing' because of an error we don't control and can't write a status for.

* fix missed merge conflict

Co-authored-by: Xavier Mouligneau <189600+XavierM@users.noreply.github.com>
@dhurley14 dhurley14 added the backport This PR is a backport of another PR label Jan 18, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@dhurley14 dhurley14 merged commit d3e71d6 into elastic:7.6 Jan 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants