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

[DOCS] Updating "Manage detection alerts" topic #666

Merged

Conversation

nastasha-solomon
Copy link
Contributor

@nastasha-solomon nastasha-solomon commented May 3, 2021

Addresses #664, elastic/kibana#96096 (see the fifth bullet under the section "View and filter detection alerts"), and elastic/kibana#97185 (see the section titled "View alert details").

Preview here.

@nastasha-solomon
Copy link
Contributor Author

nastasha-solomon commented May 11, 2021

@ecezalp and @rylnd elastic/kibana#96096 docs are available for review at #666. I've drafted an outline for elastic/kibana#97185 and will be adding to it once I have more information. Thanks in advance for your help!

Copy link
Contributor

@rylnd rylnd left a comment

Choose a reason for hiding this comment

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

@nastasha-solomon this is looking great so far! I had one comment about terminology, but please let us know if you're looking for specific feedback and/or need more info.

** `source (threat.indicator.provider)``
** `first_seen`
** `last_seen`
* *Threat Intel*: Shows the number of threat intelligence sources. If you you haven't enabled any, this tab displays the messaage `No Threat Intel Enrichment Found`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo:

Suggested change
* *Threat Intel*: Shows the number of threat intelligence sources. If you you haven't enabled any, this tab displays the messaage `No Threat Intel Enrichment Found`.
* *Threat Intel*: Shows the number of threat intelligence sources. If you haven't enabled any, this tab displays the messaage `No Threat Intel Enrichment Found`.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it may be more accurate to describe these as "indicator enrichments" or "matched indicators," or something to that effect; "intelligence sources" conveys (to me) a broader idea than what we're displaying on this tab. I don't know if we have standardized this language.

@MikePaquette @shimonmodi any thoughts here? The Enrichment RFC is currently describing these fields generally as "threat enrichments", of which "indicator rule match" may be a subtype.

Copy link
Contributor

Choose a reason for hiding this comment

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

@rylnd can you clarify what is actually displayed in that location?

Copy link

Choose a reason for hiding this comment

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

what we are displaying is each item within the threat.indicator array on that specific alert. So we are displaying information copied over from the indicator document for each threat match on an alert. Chatted with Nastasha this morning and we agree with Ryland's comment, we will be updating this piece

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @ecezalp SGTM

Copy link

@ecezalp ecezalp left a comment

Choose a reason for hiding this comment

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

Changes look good! I believe that we can expand on the Threat Summary and Threat Intel sections slightly to include some of the information we have discussed during our call today, such as

  • Threat Summary shows the fields in an aggregated fashion
  • Threat Intel tab shows all fields from all matched threats with the specific mentioning of threat.indicator field being the parent of all information displayed
  • Number of matched threats show up in the title of the Threat Intel tab
  • Threats are organized by timestamp with the most recent at the top within the Threat Intel tab
  • source event details visible as a part of each threat match

docs/detections/alerts-ui-manage.asciidoc Outdated Show resolved Hide resolved
* View detection alerts in the *Rule details* page (click
*Manage detection rules* -> rule name in the *All rules* table).
* Use the *Stack by* dropdown in the Detection alert trend histogram to select specific parameters for which to visualize the individual counts. For example, if you choose to stack by `signal.rule.name`, the histogram will display the total counts by alert name.
* Augment the results shown in the Alerts table to include alerts from building blocks or only show alerts from indicator match rules. By default, building block alerts are excluded from the Alerts table. Opting to include them expands the results and displays building block alerts alongside regular detection alerts.
Copy link

Choose a reason for hiding this comment

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

@rylnd @shimonmodi this is an interesting one. So we are filtering the alerts on signal.rule.threat_mapping - which I believe exists in all indicator match rules. Are we anticipating a future scenario in which threat match rules is a sub-category of indicator match rules, where threat_mapping exists only on indicator matches of threat type? Unless I am mistaken, we are using the two concepts of indicator match and threat match interchangeably at the moment, which could create confusion for the end users.

docs/detections/alerts-ui-manage.asciidoc Outdated Show resolved Hide resolved
docs/detections/alerts-ui-manage.asciidoc Outdated Show resolved Hide resolved
@ecezalp ecezalp self-requested a review May 14, 2021 16:23
Copy link

@ecezalp ecezalp left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@jmikell821 jmikell821 left a comment

Choose a reason for hiding this comment

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

Hi @nastasha-solomon - a few suggestions to make some of the sentences a little more concise. Can we edit the following as well, please?

Change alert statuses

In the alert’s row, click the more options icon, and then select the required status (Mark in progress, Close alert, or Open alert).

Change to:

In the alert’s row, click the More actions button (...), then select the appropriate status (Mark in progress, Close alert, or Open alert).

Send alerts to Timeline

If the rule that generated the alert uses a Timeline template, when you investigate the alert in Timeline, the dropzone query values defined in the template are replaced with their corresponding alert values.

Change to

Suppose the rule that generated the alert uses a Timeline template. In this case, when you investigate the alert in Timeline, the dropzone query values defined in the template are replaced with their corresponding alert values.

I tend to miss things the first go round, so will review again once feedback is merged. Thanks!

docs/detections/alerts-ui-manage.asciidoc Outdated Show resolved Hide resolved
docs/detections/alerts-ui-manage.asciidoc Outdated Show resolved Hide resolved
docs/detections/alerts-ui-manage.asciidoc Outdated Show resolved Hide resolved
docs/detections/alerts-ui-manage.asciidoc Outdated Show resolved Hide resolved
docs/detections/alerts-ui-manage.asciidoc Outdated Show resolved Hide resolved
docs/detections/alerts-ui-manage.asciidoc Outdated Show resolved Hide resolved
docs/detections/alerts-ui-manage.asciidoc Outdated Show resolved Hide resolved
docs/detections/alerts-ui-manage.asciidoc Outdated Show resolved Hide resolved
docs/detections/alerts-ui-manage.asciidoc Outdated Show resolved Hide resolved
docs/detections/alerts-ui-manage.asciidoc Outdated Show resolved Hide resolved
@nastasha-solomon nastasha-solomon merged commit e320e71 into master May 25, 2021
nastasha-solomon added a commit to nastasha-solomon/security-docs that referenced this pull request May 25, 2021
* Updating topic title.

* Expanded the section about vieiwing and filtering detection alerts.

* Fixed minor typos.

* Updated section for customizing the Alerts table and added a section for viewing alert details.

* Added float tag before the Customize the Alerts table section and fixed typo in alt text within security-ui file.

* Added images and started the view alert details section.

* Minor updates to drafted section.

* Fixed minor issues.

* Commiting drafted changes. Additional changes incoming.

* Incorporated additional comments from Ece.

* Removed duplicate text in description of Threat Intel tab.

* Re-adding feedback from Ece.

* Incorporated Ece's latest suggestion.

* Adding missing commas.

* Incorporating feedback from Janeen,

* Minor typos.

* Updated screenshot showing Detections page.

* Updates to new prebuilt rules integration

Co-authored-by: Janeen Mikell-Straughn <[email protected]>
Co-authored-by: jmikell821 <[email protected]>
nastasha-solomon added a commit to nastasha-solomon/security-docs that referenced this pull request May 25, 2021
* Updating topic title.

* Expanded the section about vieiwing and filtering detection alerts.

* Fixed minor typos.

* Updated section for customizing the Alerts table and added a section for viewing alert details.

* Added float tag before the Customize the Alerts table section and fixed typo in alt text within security-ui file.

* Added images and started the view alert details section.

* Minor updates to drafted section.

* Fixed minor issues.

* Commiting drafted changes. Additional changes incoming.

* Incorporated additional comments from Ece.

* Removed duplicate text in description of Threat Intel tab.

* Re-adding feedback from Ece.

* Incorporated Ece's latest suggestion.

* Adding missing commas.

* Incorporating feedback from Janeen,

* Minor typos.

* Updated screenshot showing Detections page.

* Updates to new prebuilt rules integration

Co-authored-by: Janeen Mikell-Straughn <[email protected]>
Co-authored-by: jmikell821 <[email protected]>
nastasha-solomon added a commit that referenced this pull request May 25, 2021
* Updating topic title.

* Expanded the section about vieiwing and filtering detection alerts.

* Fixed minor typos.

* Updated section for customizing the Alerts table and added a section for viewing alert details.

* Added float tag before the Customize the Alerts table section and fixed typo in alt text within security-ui file.

* Added images and started the view alert details section.

* Minor updates to drafted section.

* Fixed minor issues.

* Commiting drafted changes. Additional changes incoming.

* Incorporated additional comments from Ece.

* Removed duplicate text in description of Threat Intel tab.

* Re-adding feedback from Ece.

* Incorporated Ece's latest suggestion.

* Adding missing commas.

* Incorporating feedback from Janeen,

* Minor typos.

* Updated screenshot showing Detections page.

* Updates to new prebuilt rules integration

Co-authored-by: Janeen Mikell-Straughn <[email protected]>
Co-authored-by: jmikell821 <[email protected]>

Co-authored-by: Janeen Mikell-Straughn <[email protected]>
Co-authored-by: jmikell821 <[email protected]>
nastasha-solomon added a commit that referenced this pull request May 25, 2021
* Updating topic title.

* Expanded the section about vieiwing and filtering detection alerts.

* Fixed minor typos.

* Updated section for customizing the Alerts table and added a section for viewing alert details.

* Added float tag before the Customize the Alerts table section and fixed typo in alt text within security-ui file.

* Added images and started the view alert details section.

* Minor updates to drafted section.

* Fixed minor issues.

* Commiting drafted changes. Additional changes incoming.

* Incorporated additional comments from Ece.

* Removed duplicate text in description of Threat Intel tab.

* Re-adding feedback from Ece.

* Incorporated Ece's latest suggestion.

* Adding missing commas.

* Incorporating feedback from Janeen,

* Minor typos.

* Updated screenshot showing Detections page.

* Updates to new prebuilt rules integration

Co-authored-by: Janeen Mikell-Straughn <[email protected]>
Co-authored-by: jmikell821 <[email protected]>

Co-authored-by: Janeen Mikell-Straughn <[email protected]>
Co-authored-by: jmikell821 <[email protected]>
@nastasha-solomon nastasha-solomon deleted the issue-664-update-managing-detection-alerts-topic branch August 18, 2021 13:01
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.

5 participants