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

WIP: Sentinel artifacts #10

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

pemontto
Copy link

@pemontto pemontto commented Jan 8, 2021

This PR adds entity extraction functionality. Set to WIP for discussion. It currently uses an undocumented preview API the Azure portal uses.

Notes:

  • Extract all entity types with some special case handling
  • Configurable option to add incident level labels in Sentinel as tags to TheHive
  • Optionally configurable AzureSentinel.lookback on Sentinel getIncidents. Specify only grabbing incidents updated within X hours or days:
  • Handle some of the smaller changes between TH3 and TH4 webhooks
  • Handle closing all Sentinel incidents when multiple alerts are linked in a case

Questions:

  • What's the best way to deal with additional data. Currently I shove it in the message, but it doesn't look very nice. Would putting "{key}={value}" tags be desirable. An example of that data would be URL detonation results and final URL etc.

@pemontto pemontto force-pushed the sentinel-artifacts branch from b4da1f8 to da92add Compare January 8, 2021 19:57
@pemontto pemontto force-pushed the sentinel-artifacts branch from da92add to 007a878 Compare January 8, 2021 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant