-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
feat: file transfers analytics rules based on threshold #9131
Conversation
…o adapt to context
…hold to adapt to context
…threshold to adapt to context
Hi @juju4, please shorten the description of all the analytical rules to 255 characters only. |
description was inspired of Syslog/Analytic Rules/ssh_potentialBruteForce.yaml and many other yaml files seem to be above 255. If too long, where do we put other information? any extended description or note field? Also how to add grouping (aggregationKind) on selected entities? |
Hi @juju4, we acknowledge your concern, let me get back to you by 10 Oct 2023. |
Hi @juju4, sorry for delay in response. Regarding second question, we will get back to you by 12 Oct 2023. |
Hi @juju4, for this question, you can refer to this document - https://github.com/Azure/Azure-Sentinel/wiki/Query-Style-Guide Apart from that, I could see there are some validations which are failing in the PR. |
file extension renaming done. On eventGroupingSettings, this is not the grouping that I was looking for. In ARM template, it would be groupingConfiguration as per https://learn.microsoft.com/en-us/azure/templates/microsoft.securityinsights/2022-01-01-preview/alertrules?pivots=deployment-language-bicep#groupingconfiguration with matchingMethod = Selected |
Solutions/Microsoft 365/Analytic Rules/sharepoint_file_transfer_folders_above_threshold.yaml
Show resolved
Hide resolved
Hi @juju4, for groupingConfiguration, please refer Azure-Sentinel/Solutions/Microsoft 365 Defender/Analytic Rules/PossibleWebpBufferOverflow.yaml at 990bc461901c3f3792c2b26c9c35e01e407a8199 · Azure/Azure-Sentinel (github.com) |
Tests all green now |
Thanks! |
Change(s):
Reason for Change(s):
Version Updated:
Testing Completed:
Checked that the validations are passing and have addressed any issues that are present:
Note: for moveit, I put in Windows Forwarded Event folder but not sure if best. In my case, logs are in Events table collected through AMA.