- Notifications
You must be signed in to change notification settings - Fork 603
[New Rule] TeamFiltration User-Agents Detected #4868
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
[New Rule] TeamFiltration User-Agents Detected #4868
Conversation
Rule: New - GuidelinesThese guidelines serve as a reminder set of considerations when proposing a new rule. Documentation and Context
Rule Metadata Checks
New BBR Rules
Testing and Validation
|
rules/integrations/azure/discovery_teamfiltration_user_agents_detected.toml Outdated Show resolved Hide resolved
event.dataset:(azure.signinlogs or o365.audit) | ||
and user_agent.name:Electron | ||
and user_agent.os.name:Windows | ||
and user_agent.version:8.5.1 or | ||
user_agent.original:"Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Teams/1.3.00.30866 Chrome/80.0.3987.165 Electron/8.5.1 Safari/537.36" |
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.
@terrancedejesus I think that with no ()s this query is effective like this:
Condition 1:
event.dataset:(azure.signinlogs or o365.audit) and user_agent.name:Electron and user_agent.os.name:Windows and user_agent.version:8.5.1
Condition 2:
user_agent.original:"Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Teams/1.3.00.30866 Chrome/80.0.3987.165 Electron/8.5.1 Safari/537.36"
Which is probably not prone to FPs, but it can still match unintended events
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.
@w0rk3r - Thanks for the insight. I originally changed it due to KQL having issues with the "optimization" of it.
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.
Adjusted the query to include a nested object instead to avoid any unintended events and fix query optimization.
Pull Request
Issue link(s):
Summary - What I changed
Adds detection for TeamFiltration tool via hardcoded user agent. Checks for usage across M365 and Entra ID via Sign-In Logs.
How To Test
Checklist
bug
,enhancement
,schema
,maintenance
,Rule: New
,Rule: Deprecation
,Rule: Tuning
,Hunt: New
, orHunt: Tuning
so guidelines can be generatedmeta:rapid-merge
label if planning to merge within 24 hoursContributor checklist