Skip to content

[9.1] [Security Solution] Display which fields are customized for prebuilt rules (#225939)#227092

Merged
kibanamachine merged 1 commit intoelastic:9.1from
kibanamachine:backport/9.1/pr-225939
Jul 8, 2025
Merged

[9.1] [Security Solution] Display which fields are customized for prebuilt rules (#225939)#227092
kibanamachine merged 1 commit intoelastic:9.1from
kibanamachine:backport/9.1/pr-225939

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.1:

Questions ?

Please refer to the Backport tool documentation

…rules (elastic#225939)

## Summary

Addresses elastic#207172

Follow-up to: elastic#219628

Adds per-field badges to the rule details page so that users can see
which fields are modified on their customized prebuilt rules. Clicking
on the badges opens a rule diff flyout that displays more extensive
information.

Also switches the concurrency controls in the flyout to a static view in
which data doesn't change when it is stale, we just now display a
callout informing the user that the information they're viewing is
outdated.

### Screenshots

**Modified fields from the Rule Details page**
![Screenshot 2025-07-02 at 5 38
43 PM](https://github.com/user-attachments/assets/ae1d2865-f3bd-4d5b-a109-213f78ea194c)

**Flyout opened when those badges are clicked**
![Screenshot 2025-07-02 at 1 33
26 PM](https://github.com/user-attachments/assets/793371ae-7a7d-4b39-8889-d16bd7dff9ba)

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [ ]
[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials
- [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 b261abb)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jul 8, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) July 8, 2025 16:00
@kibanamachine kibanamachine merged commit 4ab00eb into elastic:9.1 Jul 8, 2025
16 checks passed
dplumlee added a commit that referenced this pull request Jul 8, 2025
…ebuilt rules (#225939) (#227121)

# Backport

This will backport the following commits from `main` to `8.19`:
- [[Security Solution] Display which fields are customized for prebuilt
rules (#225939)](#225939)

<!--- Backport version: 10.0.1 -->

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

<!--BACKPORT [{"author":{"name":"Davis
Plumlee","email":"56367316+dplumlee@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-07-08T15:53:08Z","message":"[Security
Solution] Display which fields are customized for prebuilt rules
(#225939)\n\n## Summary\n\nAddresses
https://github.com/elastic/kibana/issues/207172\n\nFollow-up to:
https://github.com/elastic/kibana/pull/219628\n\nAdds per-field badges
to the rule details page so that users can see\nwhich fields are
modified on their customized prebuilt rules. Clicking\non the badges
opens a rule diff flyout that displays more
extensive\ninformation.\n\nAlso switches the concurrency controls in the
flyout to a static view in\nwhich data doesn't change when it is stale,
we just now display a\ncallout informing the user that the information
they're viewing is\noutdated.\n\n### Screenshots\n\n**Modified fields
from the Rule Details page**\n![Screenshot 2025-07-02 at 5
38\n43 PM](https://github.com/user-attachments/assets/ae1d2865-f3bd-4d5b-a109-213f78ea194c)\n\n**Flyout
opened when those badges are clicked**\n![Screenshot 2025-07-02 at 1
33\n26 PM](https://github.com/user-attachments/assets/793371ae-7a7d-4b39-8889-d16bd7dff9ba)\n\n\n###
Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers
should verify this PR satisfies this list as well.\n\n- [x] Any text
added follows [EUI's
writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing),
uses\nsentence case text and includes
[i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n-
[
]\n[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)\nwas
added for features that require explanation or tutorials\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"b261abbe52d70ccb562e9abf5f695251cc851142","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:enhancement","Team:Detections
and Resp","Team: SecuritySolution","Team:Detection Rule
Management","Feature:Prebuilt Detection
Rules","backport:version","v9.1.0","v8.19.0","v9.2.0"],"title":"[Security
Solution] Display which fields are customized for prebuilt
rules","number":225939,"url":"https://github.com/elastic/kibana/pull/225939","mergeCommit":{"message":"[Security
Solution] Display which fields are customized for prebuilt rules
(#225939)\n\n## Summary\n\nAddresses
https://github.com/elastic/kibana/issues/207172\n\nFollow-up to:
https://github.com/elastic/kibana/pull/219628\n\nAdds per-field badges
to the rule details page so that users can see\nwhich fields are
modified on their customized prebuilt rules. Clicking\non the badges
opens a rule diff flyout that displays more
extensive\ninformation.\n\nAlso switches the concurrency controls in the
flyout to a static view in\nwhich data doesn't change when it is stale,
we just now display a\ncallout informing the user that the information
they're viewing is\noutdated.\n\n### Screenshots\n\n**Modified fields
from the Rule Details page**\n![Screenshot 2025-07-02 at 5
38\n43 PM](https://github.com/user-attachments/assets/ae1d2865-f3bd-4d5b-a109-213f78ea194c)\n\n**Flyout
opened when those badges are clicked**\n![Screenshot 2025-07-02 at 1
33\n26 PM](https://github.com/user-attachments/assets/793371ae-7a7d-4b39-8889-d16bd7dff9ba)\n\n\n###
Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers
should verify this PR satisfies this list as well.\n\n- [x] Any text
added follows [EUI's
writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing),
uses\nsentence case text and includes
[i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n-
[
]\n[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)\nwas
added for features that require explanation or tutorials\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"b261abbe52d70ccb562e9abf5f695251cc851142"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"9.1","label":"v9.1.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/227092","number":227092,"state":"MERGED","mergeCommit":{"sha":"4ab00ebff2ddb2355feee8ea8b955f53d83348b0","message":"[9.1]
[Security Solution] Display which fields are customized for prebuilt
rules (#225939) (#227092)\n\n# Backport\n\nThis will backport the
following commits from `main` to `9.1`:\n- [[Security Solution] Display
which fields are customized for prebuilt\nrules
(#225939)](https://github.com/elastic/kibana/pull/225939)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Davis Plumlee
<56367316+dplumlee@users.noreply.github.com>"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/225939","number":225939,"mergeCommit":{"message":"[Security
Solution] Display which fields are customized for prebuilt rules
(#225939)\n\n## Summary\n\nAddresses
https://github.com/elastic/kibana/issues/207172\n\nFollow-up to:
https://github.com/elastic/kibana/pull/219628\n\nAdds per-field badges
to the rule details page so that users can see\nwhich fields are
modified on their customized prebuilt rules. Clicking\non the badges
opens a rule diff flyout that displays more
extensive\ninformation.\n\nAlso switches the concurrency controls in the
flyout to a static view in\nwhich data doesn't change when it is stale,
we just now display a\ncallout informing the user that the information
they're viewing is\noutdated.\n\n### Screenshots\n\n**Modified fields
from the Rule Details page**\n![Screenshot 2025-07-02 at 5
38\n43 PM](https://github.com/user-attachments/assets/ae1d2865-f3bd-4d5b-a109-213f78ea194c)\n\n**Flyout
opened when those badges are clicked**\n![Screenshot 2025-07-02 at 1
33\n26 PM](https://github.com/user-attachments/assets/793371ae-7a7d-4b39-8889-d16bd7dff9ba)\n\n\n###
Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers
should verify this PR satisfies this list as well.\n\n- [x] Any text
added follows [EUI's
writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing),
uses\nsentence case text and includes
[i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n-
[
]\n[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)\nwas
added for features that require explanation or tutorials\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"b261abbe52d70ccb562e9abf5f695251cc851142"}}]}]
BACKPORT-->
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