-
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
Githubwebhook #10416
Githubwebhook #10416
Conversation
Update ASimAuditEventGithub.yml
Raising a PR for GithubWebhook AuditEvent Parser
This reverts commit f4f7b24.
Sample Data/ASIM/Microsoft_GithubWebhook_AuditEvent_IngestedLogs_old.csv
Outdated
Show resolved
Hide resolved
Sample Data/ASIM/Microsoft_Github_ASimAuditEvent_IngestedLogs.csv
Outdated
Show resolved
Hide resolved
@@ -0,0 +1,53 @@ | |||
ColumnName,ColumnOrdinal,DataType,ColumnType |
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.
if this upto date?
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.
Yes
"installation","Success","Create","Informational", | ||
"team","Success","Create","Informational", | ||
"organization","Success","Other","Informational", | ||
"label","Success","Other","Informational" |
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.
does github webhook provide these 12 event_s or are there more? if more, we should configure all.
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.
Configured 25 with reference to Product document
Hi @gokulkate96, |
…on_SchemaTest.csv
Hi @gokulkate96, |
Hi @gokulkate96, |
Hi @gokulkate96, |
Hi @gokulkate96, |
Hi @gokulkate96, |
Hi @gokulkate96, |
Hi @gokulkate96, |
Since we have not received a response in the last 7 days, we are closing your PR #10416 per our standard operating procedures. If you still need support for this issue, you can re-open the PR at any time. |
Hi @gokulkate96, |
Hi @gokulkate96, |
Hi @gokulkate96, |
Hi @gokulkate96, |
We wanted to check on the status of PR #10416. PR is pending for more than 60 days. Please let us know if you need any assistance to review this PR. Per our standard operating procedures if no response is received in the next 7 business days, we will close this PR. Thank you for your cooperation |
Since we have not received a response in the last 7 days, we are closing your PR #10416 per our standard operating procedures. If you still need support for this issue, you can re-open the PR at any time. If you do re-open, we simply request that you ensure the PR has response to the last request. Thank you for your cooperation |
Required items, please complete
Change(s):
Reason for Change(s):
Version Updated:
Testing Completed:
Checked that the validations are passing and have addressed any issues that are present:
Guidance <- remove section before submitting
Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.
Change(s):
Reason for Change(s):
Version updated:
Testing Completed:
Note: If updating a detection, you must update the version field.
Checked that the validations are passing and have addressed any issues that are present:
Note: Let us know if you have tried fixing the validation error and need help.