[LouisDeconinck]

Fixes #7737

When path redaction is disabled, RequestUri.AbsolutePath already begins with /. The outgoing request formatter was adding another separator, producing messages such as GET github.com//dotnet/extensions.

Use a separator only when the logged path does not already start with /, while leaving the structured url.path value unchanged. The regression test covers the unredacted path mode.

Microsoft Reviewers: Open in CodeFlow

[Copilot]

🟢 Approval recommended

The change is narrowly scoped, matches the reported behavior, and includes a targeted regression test for the failing scenario.

Pull request overview

Fixes an HTTP client logging formatting bug where the rendered message could include a duplicated / between host and path when path redaction is disabled, while keeping the structured logging tags unchanged.

Changes:

• Adjusts the outgoing-request original-format message to conditionally add a / only when the logged path doesn’t already start with /.
• Adds a regression test covering RequestPathParameterRedactionMode = None to ensure the rendered message doesn’t duplicate the separator.

File summaries

File	Description
test/Libraries/Microsoft.Extensions.Http.Diagnostics.Tests/Logging/HttpClientLoggerTest.cs	Adds a regression test validating unredacted path formatting does not produce a double slash.
src/Libraries/Microsoft.Extensions.Http.Diagnostics/Logging/Internal/Log.cs	Updates the message formatter to avoid inserting an extra / when the path already begins with /.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Lite

---

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

[LouisDeconinck]

@dotnet-policy-service agree