Skip to content

Detection : External User Added to Team and Immediately Uploads File#3556

Merged
shainw merged 5 commits intoAzure:masterfrom
samikroy:patch-14
Dec 30, 2021
Merged

Detection : External User Added to Team and Immediately Uploads File#3556
shainw merged 5 commits intoAzure:masterfrom
samikroy:patch-14

Conversation

@samikroy
Copy link
Copy Markdown
Contributor

Proposed Changes

This detection identifies an external user is added to a Team or Teams chat
and within 1 minute of being added upload a file via the chat.

This detection identifies an external user is added to a Team or Teams chat
 and within 1 minute of being added upload a file via the chat.
@ep3p
Copy link
Copy Markdown
Contributor

ep3p commented Nov 27, 2021

You may want to specify the kind of join to not lose events of the left table.

@samikroy
Copy link
Copy Markdown
Contributor Author

You may want to specify the kind of join to not lose events of the left table.

Thank you for the review @ep3p .
Have added inner as a join kind to avoid default innerunique picking a random one.
Please let know further.

@shainw
Copy link
Copy Markdown
Contributor

shainw commented Nov 30, 2021

Adding @petebryan as he did the Hunting query that this is based on and he may have tested to see if this type of detection was too noisy for a detection - https://github.com/Azure/Azure-Sentinel/blob/master/Hunting%20Queries/OfficeActivity/UserAddToTeamsAndUploadsFile.yaml.. @petebryan - thoughts?

@samikroy
Copy link
Copy Markdown
Contributor Author

Adding @petebryan as he did the Hunting query that this is based on and he may have tested to see if this type of detection was too noisy for a detection - https://github.com/Azure/Azure-Sentinel/blob/master/Hunting%20Queries/OfficeActivity/UserAddToTeamsAndUploadsFile.yaml.. @petebryan - thoughts?

@petebryan - please share your thoughts

@petebryan
Copy link
Copy Markdown
Contributor

@samikroy so when I first created this we didn't have an official Teams connector publicly available yet so we had limited data to determine FP rate. However in the data did show this was potentially quite noisy. It would be good to re-evaluate to see if that is still the case.

@samikroy
Copy link
Copy Markdown
Contributor Author

samikroy commented Dec 2, 2021

@samikroy so when I first created this we didn't have an official Teams connector publicly available yet so we had limited data to determine FP rate. However in the data did show this was potentially quite noisy. It would be good to re-evaluate to see if that is still the case.

Agree @petebryan , this detection now applies few more filters

External User Added > File Uploaded > File is accessed by many users > External user is removed.
Please have a look and let me know
Thank you.

shainw
shainw previously approved these changes Dec 2, 2021
Copy link
Copy Markdown
Contributor

@shainw shainw left a comment

Choose a reason for hiding this comment

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

I did some testing and looks like looking only for #EXT# (external users) and checking for within 1m has very low results. Approving and we will watch if alerts start to flood for some customers.

@shainw shainw added the Detection Detection specialty review needed label Dec 30, 2021
@shainw shainw merged commit bb976ea into Azure:master Dec 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Detection Detection specialty review needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants