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
MS Defender Threat Hunting move to Graph API #33413
Conversation
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/MicrosoftGraphSecurity.py
Outdated
Show resolved
Hide resolved
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/MicrosoftGraphSecurity.py
Outdated
Show resolved
Hide resolved
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/MicrosoftGraphSecurity.py
Show resolved
Hide resolved
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.
Good job, see my comments
Please resolve the conflicts |
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.
Please wait also for @jbabazadeh review
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.
Nice work!
A few comments.
Please open another PR that deprecate the old command.
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/MicrosoftGraphSecurity.py
Outdated
Show resolved
Hide resolved
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/MicrosoftGraphSecurity.py
Outdated
Show resolved
Hide resolved
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/MicrosoftGraphSecurity_test.py
Outdated
Show resolved
Hide resolved
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/README.md
Show resolved
Hide resolved
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/MicrosoftGraphSecurity.yml
Outdated
Show resolved
Hide resolved
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/MicrosoftGraphSecurity.yml
Outdated
Show resolved
Hide resolved
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/MicrosoftGraphSecurity.yml
Outdated
Show resolved
Hide resolved
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/MicrosoftGraphSecurity.yml
Outdated
Show resolved
Hide resolved
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/MicrosoftGraphSecurity.yml
Outdated
Show resolved
Hide resolved
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/README.md
Outdated
Show resolved
Hide resolved
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/README.md
Outdated
Show resolved
Hide resolved
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/README.md
Outdated
Show resolved
Hide resolved
Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/README.md
Outdated
Show resolved
Hide resolved
@jbabazadeh @rshunim Doc review completed. |
This PR was automatically updated by a GitHub Action
To stop automatic version bumps, add the |
Status
Related Issues
fixes: https://jira-dc.paloaltonetworks.com/browse/CIAC-9481
Description
MS Defender Threat Hunting move to Graph API.
Must have