Skip to content

Conversation

@rahul0216
Copy link
Collaborator

Change(s):

  • See guidance below

Reason for Change(s):

  • See guidance below

Version Updated:

  • Required only for Detections/Analytic Rule templates
  • See guidance below

Testing Completed:

  • See guidance below

Checked that the validations are passing and have addressed any issues that are present:

  • See guidance below

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.

Details of the code changes in your submitted PR. Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process. Providing associated Issues that this resolves also easily connects the reason.

Change(s):

  • Added comments to suppress codeql alerts for unallowed hash algorithm usages

Reason for Change(s):

  • To suppress codeql alerts

Version updated:

  • NA

Testing Completed:

  • Yes

Added CodeQL comments to suppress alerts for various hash algorithm usages in cryptography package files for compatibility and backwards compatibility purposes. No functional changes were made.
@rahul0216 rahul0216 requested review from a team as code owners January 8, 2026 10:25
@v-shukore v-shukore self-assigned this Jan 8, 2026
@v-shukore v-shukore added the Solution Solution specialty review needed label Jan 8, 2026
@rahul0216 rahul0216 merged commit ecab128 into master Jan 9, 2026
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Solution Solution specialty review needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants