Skip to content

Suppress CodeQL false positive in internal HTTP utility - #14223

Merged
SergeyMenshykh merged 1 commit into
microsoft:mainfrom
SergeyMenshykh:sergeymenshykh-suppress-codeql-ssrf-alert
Jul 28, 2026
Merged

Suppress CodeQL false positive in internal HTTP utility#14223
SergeyMenshykh merged 1 commit into
microsoft:mainfrom
SergeyMenshykh:sergeymenshykh-suppress-codeql-ssrf-alert

Conversation

@SergeyMenshykh

@SergeyMenshykh SergeyMenshykh commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Adds an inline CodeQL suppression comment for a confirmed false positive in an internal HTTP utility method. Input validation is handled by callers higher in the stack.

Copilot AI review requested due to automatic review settings July 28, 2026 10:41

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

Adds a CodeQL configuration intended to suppress a confirmed false positive by excluding an internal utility helper from analysis.

Changes:

  • Introduces a CodeQL.yml configuration file with an attempted path exclusion for dotnet/src/InternalUtilities/src/Http/HttpClientExtensions.cs.

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

Comment thread CodeQL.yml Outdated
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 5830be01-63d3-41f6-ae70-8ffeae81d46f
@SergeyMenshykh
SergeyMenshykh force-pushed the sergeymenshykh-suppress-codeql-ssrf-alert branch from 18a6329 to 3f4391b Compare July 28, 2026 11:06
@SergeyMenshykh
SergeyMenshykh requested a review from a team as a code owner July 28, 2026 11:06
@SergeyMenshykh SergeyMenshykh changed the title Add CodeQL.yml to suppress false positive in internal utility Suppress CodeQL false positive in internal HTTP utility Jul 28, 2026
@SergeyMenshykh
SergeyMenshykh enabled auto-merge July 28, 2026 11:12
@SergeyMenshykh SergeyMenshykh self-assigned this Jul 28, 2026
@SergeyMenshykh SergeyMenshykh moved this to In Review in Agent Framework Jul 28, 2026
@SergeyMenshykh SergeyMenshykh moved this to Sprint: In Review in Semantic Kernel Jul 28, 2026
@SergeyMenshykh
SergeyMenshykh added this pull request to the merge queue Jul 28, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 28, 2026
@SergeyMenshykh
SergeyMenshykh added this pull request to the merge queue Jul 28, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 28, 2026
@SergeyMenshykh
SergeyMenshykh added this pull request to the merge queue Jul 28, 2026
Merged via the queue into microsoft:main with commit e78a8f5 Jul 28, 2026
21 of 24 checks passed
@SergeyMenshykh
SergeyMenshykh deleted the sergeymenshykh-suppress-codeql-ssrf-alert branch July 28, 2026 15:51
@github-project-automation github-project-automation Bot moved this from Sprint: In Review to Sprint: Done in Semantic Kernel Jul 28, 2026
@github-project-automation github-project-automation Bot moved this from In Review to Done in Agent Framework Jul 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done
Status: Sprint: Done

Development

Successfully merging this pull request may close these issues.

4 participants