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

Audit Sentinel Detection Rules #7725

Merged
merged 8 commits into from
Jul 7, 2023
Merged

Audit Sentinel Detection Rules #7725

merged 8 commits into from
Jul 7, 2023

Conversation

samikroy
Copy link
Contributor

@samikroy samikroy commented Apr 2, 2023

Required items, please complete

Change(s):

  • Added a query to audit Sentinel Detection rules changes,
  • Added a saved query to be deployed as function.

Reason for Change(s):

  • New tool addition.

Testing Completed:

  • Tested with pre-requisites qualified and works good.

@v-atulyadav
Copy link
Contributor

Hi @v-sabiraj @v-rucdu,
Please provide your feedback on this.
Thanks

@samikroy
Copy link
Contributor Author

Hi @v-sabiraj @v-rucdu & @v-atulyadav
Please provide your inputs on this.
Thanks

@samikroy
Copy link
Contributor Author

Hi @v-sabiraj @v-rucdu & @v-atulyadav
Please provide your inputs on this.
Thanks

@v-atulyadav
Copy link
Contributor

Hi @samikroy,
We will soon provide you an update on this.
Thanks

@v-atulyadav
Copy link
Contributor

Looking into this

@v-atulyadav
Copy link
Contributor

Hi @v-sabiraj @v-rucdu,
Please provide your feedback on this.
Thanks

@v-prasadboke
Copy link
Contributor

Hello @v-rucdu and @v-sabiraj waiting for your feddback

Tools/Audit-Sentinel-Detection/Readme.md Outdated Show resolved Hide resolved
Tools/Audit-Sentinel-Detection/Readme.md Outdated Show resolved Hide resolved
Tools/Audit-Sentinel-Detection/Readme.md Outdated Show resolved Hide resolved
@v-atulyadav
Copy link
Contributor

Hi @sreedharande, please provide your feedback on this. Thanks

@v-atulyadav
Copy link
Contributor

Hi @sreedharande, could you please provide your feedback on this. Thanks

@v-prasadboke
Copy link
Contributor

Hello @sreedharande waiting for your feedback

1 similar comment
@v-prasadboke
Copy link
Contributor

Hello @sreedharande waiting for your feedback

@samikroy
Copy link
Contributor Author

Hi All,
Could you please help with the merge of this
or let me know for any updates to it.
Thank you.

@samikroy
Copy link
Contributor Author

Hi @sreedharande , @v-prasadboke , @v-atulyadav , @v-sabiraj ,
Request your help for the merger of this PR.
Thank you,

@v-atulyadav
Copy link
Contributor

Hi @samikroy, could you please add support information to the readme file? Thanks

@v-atulyadav v-atulyadav self-assigned this Jun 22, 2023
@samikroy
Copy link
Contributor Author

Hi @samikroy, could you please add support information to the readme file? Thanks

@v-atulyadav - Could you please help with an example !

@v-atulyadav
Copy link
Contributor

Hi @samikroy, adding the following details to your read me file will help users to get support if they have any issues.

{
"support": {
"name": "Publisher Name/ID",
"email": "Email for template Support",
"link": "Link of Support contacts for this template"
}
} 

@samikroy
Copy link
Contributor Author

samikroy commented Jul 4, 2023

Hi @samikroy, adding the following details to your read me file will help users to get support if they have any issues.

{ "support": { "name": "Publisher Name/ID", "email": "Email for template Support", "link": "Link of Support contacts for this template" } }

@v-atulyadav - Please have a look and let me know.

@v-atulyadav v-atulyadav merged commit 6ada490 into Azure:master Jul 7, 2023
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants