Skip to content

fix: Fix codeql suppress comment - #204

Merged
Alon Yeshurun (ayeshurun) merged 8 commits into
microsoft:mainfrom
ayeshurun:fix-codeql
Mar 31, 2026
Merged

fix: Fix codeql suppress comment#204
Alon Yeshurun (ayeshurun) merged 8 commits into
microsoft:mainfrom
ayeshurun:fix-codeql

Conversation

@ayeshurun

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the SHA‑1 CodeQL suppression comment placement in the certificate thumbprint computation in fab_auth.py to ensure the intended suppression applies to the flagged usage.

Changes:

  • Adjusted the placement/formatting of the CodeQL suppression comment for the hashes.SHA1() thumbprint calculation in _load_pem_certificate.

Comment thread src/fabric_cli/core/fab_auth.py
@ayeshurun
Alon Yeshurun (ayeshurun) merged commit ae22b09 into microsoft:main Mar 31, 2026
16 of 17 checks passed
@ayeshurun
Alon Yeshurun (ayeshurun) deleted the fix-codeql branch March 31, 2026 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants