Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Security Solution] [Fix] Alert Page Controls do not recover from invalid query. #156542

Merged
merged 3 commits into from
May 3, 2023

Conversation

logeekal
Copy link
Contributor

@logeekal logeekal commented May 3, 2023

Summary

This PR handles #156016 .

Previously, if user supplied an invalid kql or lucene query, Alert Page controls will go in error state and not recover until user reloaded the page or navigated away and back to the Alert Page.

This PR prevents Alert Page Controls going in that error state.

Before After
Screen.Recording.2023-05-03.at.15.32.37.mov
Screen.Recording.2023-05-03.at.15.28.37.mov

Checklist

Delete any items that are not applicable to this PR.

@logeekal logeekal requested a review from a team as a code owner May 3, 2023 13:35
Copy link
Contributor

@janmonschke janmonschke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏

@logeekal logeekal enabled auto-merge (squash) May 3, 2023 14:54
@logeekal logeekal merged commit ff65ca4 into elastic:main May 3, 2023
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 9.1MB 9.1MB +232.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
securitySolution 399 402 +3
total +5

References to deprecated APIs

id before after diff
securitySolution 545 550 +5

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
securitySolution 479 482 +3
total +5

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request May 3, 2023
…alid query. (elastic#156542)

## Summary

This PR handles elastic#156016 .

Previously, if user supplied an invalid kql or lucene query, Alert Page
controls will go in error state and not recover until user reloaded the
page or navigated away and back to the Alert Page.

This PR prevents Alert Page Controls going in that error state.

| Before | After |
|--|--|
| <video
src="https://user-images.githubusercontent.com/7485038/235931286-6da23567-4ae8-454a-92b8-a595a20f5655.mov"
/> | <video
src="https://user-images.githubusercontent.com/7485038/235930584-485df881-d22c-44f3-9d53-f673820eb673.mov"
/> |

### Checklist

Delete any items that are not applicable to this PR.

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit ff65ca4)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.8

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request May 3, 2023
…om invalid query. (#156542) (#156567)

# Backport

This will backport the following commits from `main` to `8.8`:
- [[Security Solution] [Fix] Alert Page Controls do not recover from
invalid query. (#156542)](#156542)

<!--- Backport version: 8.9.7 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Jatin
Kathuria","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-05-03T14:57:11Z","message":"[Security
Solution] [Fix] Alert Page Controls do not recover from invalid query.
(#156542)\n\n## Summary\r\n\r\nThis PR handles #156016
.\r\n\r\nPreviously, if user supplied an invalid kql or lucene query,
Alert Page\r\ncontrols will go in error state and not recover until user
reloaded the\r\npage or navigated away and back to the Alert
Page.\r\n\r\nThis PR prevents Alert Page Controls going in that error
state.\r\n\r\n| Before | After |\r\n|--|--|\r\n|
<video\r\nsrc=\"https://user-images.githubusercontent.com/7485038/235931286-6da23567-4ae8-454a-92b8-a595a20f5655.mov\"\r\n/>
|
<video\r\nsrc=\"https://user-images.githubusercontent.com/7485038/235930584-485df881-d22c-44f3-9d53-f673820eb673.mov\"\r\n/>
|\r\n\r\n### Checklist\r\n\r\nDelete any items that are not applicable
to this PR.\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"ff65ca42f2f09589b09a8e61abe22a4a69885170","branchLabelMapping":{"^v8.9.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Team:Threat
Hunting:Investigations","v8.8.0","v8.9.0"],"number":156542,"url":"https://github.com/elastic/kibana/pull/156542","mergeCommit":{"message":"[Security
Solution] [Fix] Alert Page Controls do not recover from invalid query.
(#156542)\n\n## Summary\r\n\r\nThis PR handles #156016
.\r\n\r\nPreviously, if user supplied an invalid kql or lucene query,
Alert Page\r\ncontrols will go in error state and not recover until user
reloaded the\r\npage or navigated away and back to the Alert
Page.\r\n\r\nThis PR prevents Alert Page Controls going in that error
state.\r\n\r\n| Before | After |\r\n|--|--|\r\n|
<video\r\nsrc=\"https://user-images.githubusercontent.com/7485038/235931286-6da23567-4ae8-454a-92b8-a595a20f5655.mov\"\r\n/>
|
<video\r\nsrc=\"https://user-images.githubusercontent.com/7485038/235930584-485df881-d22c-44f3-9d53-f673820eb673.mov\"\r\n/>
|\r\n\r\n### Checklist\r\n\r\nDelete any items that are not applicable
to this PR.\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"ff65ca42f2f09589b09a8e61abe22a4a69885170"}},"sourceBranch":"main","suggestedTargetBranches":["8.8"],"targetPullRequestStates":[{"branch":"8.8","label":"v8.8.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.9.0","labelRegex":"^v8.9.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/156542","number":156542,"mergeCommit":{"message":"[Security
Solution] [Fix] Alert Page Controls do not recover from invalid query.
(#156542)\n\n## Summary\r\n\r\nThis PR handles #156016
.\r\n\r\nPreviously, if user supplied an invalid kql or lucene query,
Alert Page\r\ncontrols will go in error state and not recover until user
reloaded the\r\npage or navigated away and back to the Alert
Page.\r\n\r\nThis PR prevents Alert Page Controls going in that error
state.\r\n\r\n| Before | After |\r\n|--|--|\r\n|
<video\r\nsrc=\"https://user-images.githubusercontent.com/7485038/235931286-6da23567-4ae8-454a-92b8-a595a20f5655.mov\"\r\n/>
|
<video\r\nsrc=\"https://user-images.githubusercontent.com/7485038/235930584-485df881-d22c-44f3-9d53-f673820eb673.mov\"\r\n/>
|\r\n\r\n### Checklist\r\n\r\nDelete any items that are not applicable
to this PR.\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"ff65ca42f2f09589b09a8e61abe22a4a69885170"}}]}]
BACKPORT-->

Co-authored-by: Jatin Kathuria <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants