KEMBAR78
Add Codeql Suppressions by anamnavi · Pull Request #25943 · PowerShell/PowerShell · GitHub
Skip to content

Conversation

@anamnavi
Copy link
Member

@anamnavi anamnavi commented Aug 29, 2025

This pull request includes several comments added to the code to address CodeQL warnings in the PowerShell codebase. The comments explain why the flagged issues are expected behavior and not security concerns. The comments specifically address CodeQL warnings for command-line injection and server-side request forgery (SSRF)

CodeQL suppression comments were added in Process.cs, NativeCommandProcessor.cs, RunspaceConnectionInfo.cs, and FileSystemProvider.cs, WebRequestPSCmdlet.Common.cs

PR Summary

PR Context

PR Checklist

@anamnavi anamnavi requested a review from TravisEz13 August 29, 2025 22:18
@TravisEz13 TravisEz13 changed the title Codeql suppresions2 Codeql suppresions Sep 3, 2025
@TravisEz13 TravisEz13 self-assigned this Sep 3, 2025
@anamnavi anamnavi changed the title Codeql suppresions Add Codeql Suppressions Sep 3, 2025
@TravisEz13 TravisEz13 added CL-Tools Indicates that a PR should be marked as a tools change in the Change Log BackPort-7.4.x-Consider BackPort-7.5.x-Consider labels Sep 3, 2025
@TravisEz13
Copy link
Member

/azp run PowerShell-CI-linux-packaging, PowerShell-Windows-Packaging-CI

@azure-pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@TravisEz13 TravisEz13 merged commit 1e46d89 into PowerShell:master Sep 3, 2025
36 of 42 checks passed
@microsoft-github-policy-service
Copy link
Contributor

microsoft-github-policy-service bot commented Sep 3, 2025

📣 Hey @@anamnavi, how did we do? We would love to hear your feedback with the link below! 🗣️

🔗 https://aka.ms/PSRepoFeedback

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport-7.4.x-Migrated Backport-7.5.x-Migrated CL-Tools Indicates that a PR should be marked as a tools change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants