-
Notifications
You must be signed in to change notification settings - Fork 613
[Rule Tuning] Entra ID Protection Sign-in and User Risk Detection Rules - Filter Remediated Risk States #5535
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
base: main
Are you sure you want to change the base?
Conversation
…es - Filter Remediated Risk States Fixes #5534
Rule: Tuning - GuidelinesThese guidelines serve as a reminder set of considerations when tuning an existing rule. Documentation and Context
Rule Metadata Checks
Testing and Validation
|
|
|
||
| query = ''' | ||
| event.dataset: "azure.identity_protection" and | ||
| event.action: "User Risk Detection" and |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just double checking, since both rules now filter by azure.identityprotection.properties.activity do we still need event.action and is it correct? Do both rules use the same event action across user/signin?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for review @Mikaayenson.
just double checking, since both rules now filter by azure.identityprotection.properties.activity do we still need event.action and is it correct? Do both rules use the same event action across user/signin?
azure.identityprotection.properties.activity is to differentiate between sign-ins that are risky (atypical travel, token anomalies, etc.) under the Entra ID "Risk Detections" heuristics, where user is reported that Entra ID classifies a user is typically at risk. Risky users have User Risk Detection as an event.action just as users at risk do, however users at risk also have Risky user as event.action sometimes depending on what Entra ID Protection flagged it for. I kept event.action in to scope the specific alerts prompted by Entra ID Protection to user risk, pending different types of alerts in the future; but not it is not necessary to differentiate between these two rules. I do believe it is best to keep it.
Fixes #5534
Pull Request
Issue link(s):
Summary - What I changed
Removes unnecessary noise from Entra ID Protection Risk Detection rules related to remediated/dismissed action taken by Entra ID Admins/Users. Please see related issue for more details.
How To Test
Checklist
bug,enhancement,schema,maintenance,Rule: New,Rule: Deprecation,Rule: Tuning,Hunt: New, orHunt: Tuningso guidelines can be generatedmeta:rapid-mergelabel if planning to merge within 24 hoursContributor checklist