Skip to content

Log and redact query string params#6521

Merged
rainsxng merged 49 commits intomainfrom
dbohdanov/log-query-string-params
Sep 1, 2025
Merged

Log and redact query string params#6521
rainsxng merged 49 commits intomainfrom
dbohdanov/log-query-string-params

Conversation

@rainsxng
Copy link
Copy Markdown
Contributor

@rainsxng rainsxng commented Jun 16, 2025 •
edited
Loading

This feature allows to log and redact request query string parameters

Microsoft Reviewers: Open in CodeFlow

@rainsxng rainsxng linked an issue Jun 17, 2025 that may be closed by this pull request
HttpClientLogger doesn't log query string parameters #6385
Closed
@rainsxng rainsxng marked this pull request as ready for review June 18, 2025 11:37
@rainsxng rainsxng requested a review from a team as a code owner June 18, 2025 11:37
evgenyfedorov2
evgenyfedorov2 requested changes Jul 7, 2025
View reviewed changes
@dotnet-policy-service dotnet-policy-service bot added the waiting-author-feedback 📭 The author of this issue needs to respond in order for us to continue investigating this issue. label Jul 7, 2025
@dotnet-policy-service dotnet-policy-service bot removed the waiting-author-feedback 📭 The author of this issue needs to respond in order for us to continue investigating this issue. label Jul 7, 2025
@rainsxng @iliar-turdushev
Update src/Libraries/Microsoft.Extensions.Http.Diagnostics/Logging/In…
f682266 
…ternal/LogRecord.cs

Co-authored-by: Iliar Turdushev <iliarturdu@microsoft.com>
@rainsxng rainsxng enabled auto-merge (squash) August 29, 2025 13:51
@rainsxng rainsxng merged commit 0b4c4d5 into main Sep 1, 2025
6 checks passed
@rainsxng rainsxng deleted the dbohdanov/log-query-string-params branch September 1, 2025 14:54
@github-actions github-actions bot locked and limited conversation to collaborators Oct 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

Copilot code review Copilot Copilot left review comments

@dariusclay dariusclay dariusclay approved these changes

@iliar-turdushev iliar-turdushev iliar-turdushev approved these changes

@evgenyfedorov2 evgenyfedorov2 evgenyfedorov2 approved these changes

@omsoftdev omsoftdev Awaiting requested review from omsoftdev

@rosebyte rosebyte Awaiting requested review from rosebyte

Assignees

@rainsxng rainsxng

Labels

area-telemetry

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

HttpClientLogger doesn't log query string parameters

5 participants

@rainsxng @dariusclay @iliar-turdushev @evgenyfedorov2