Skip to content

Add CodeQL suppressions for NativeCommandProcessor#26729

Merged
daxian-dbw merged 2 commits into
PowerShell:masterfrom
anamnavi:codeql-suppressions-nativecommand
Feb 3, 2026
Merged

Add CodeQL suppressions for NativeCommandProcessor#26729
daxian-dbw merged 2 commits into
PowerShell:masterfrom
anamnavi:codeql-suppressions-nativecommand

Conversation

@anamnavi
Copy link
Copy Markdown
Member

Add CodeQL suppressions for NativeCommandProcessor. Also fix typos in previous CodeQL suppressions in this file.

PR Summary

PR Context

PR Checklist

@anamnavi anamnavi requested a review from a team as a code owner January 27, 2026 23:27
Copilot AI review requested due to automatic review settings January 27, 2026 23:27
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a CodeQL suppression for command-line-injection in the NativeCommandProcessor and fixes spelling inconsistencies in existing CodeQL suppressions.

Changes:

  • Added CodeQL suppression comment for command-line-injection on line 834 with appropriate security justification
  • Fixed typo "Poweshell" to "PowerShell" in two existing CodeQL suppression comments (lines 859 and 1611)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@TravisEz13 TravisEz13 added CL-Engine Indicates that a PR should be marked as an engine change in the Change Log PowerShell-Docs needed The PR was reviewed and a PowerShell Docs update is needed Approved-LowRisk Indicate a PR has been approved and can be merged after a quick review of another maintainer. labels Jan 28, 2026
@microsoft-github-policy-service microsoft-github-policy-service Bot removed the Approved-LowRisk Indicate a PR has been approved and can be merged after a quick review of another maintainer. label Jan 28, 2026
@microsoft-github-policy-service
Copy link
Copy Markdown
Contributor

The Approved-LowRisk label is restricted to authorized maintainers and was removed.

@daxian-dbw daxian-dbw added the Approved-LowRisk Indicate a PR has been approved and can be merged after a quick review of another maintainer. label Jan 28, 2026
@daxian-dbw daxian-dbw merged commit 34f6e26 into PowerShell:master Feb 3, 2026
40 of 41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved-LowRisk Indicate a PR has been approved and can be merged after a quick review of another maintainer. CL-Engine Indicates that a PR should be marked as an engine change in the Change Log PowerShell-Docs needed The PR was reviewed and a PowerShell Docs update is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants