Skip to content

Use built-in searches in HttpUtility.ParseQueryString - #136634

Merged
stephentoub merged 1 commit into
mainfrom
stephentoub-query-string-delimiter-scanning
Oct 29, 2026
Merged

stephentoub merged 1 commit into
mainfrom
stephentoub-query-string-delimiter-scanning

Conversation

@stephentoub

@stephentoub stephentoub commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

Use IndexOf to find query-string delimiters instead of scanning each character, preserving decoding and collection behavior. Adds edge-case and encoding coverage.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).
13 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @karelz, @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

@rzikm rzikm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, Thanks!

@stephentoub
stephentoub enabled auto-merge (squash) October 29, 2026 16:51
@MihaZupan MihaZupan added this to the 12.0.0 milestone Oct 29, 2026
@stephentoub
stephentoub merged commit 7a14476 into main Oct 29, 2026
89 of 91 checks passed
@stephentoub
stephentoub deleted the stephentoub-query-string-delimiter-scanning branch October 29, 2026 17:25
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