Skip to content

.NET: fix: do not forward headers on redirect - #8164

Merged
Vincent Biret (baywet) merged 21 commits into
mainfrom
fix/redirection-headers
Sep 10, 2026
Merged

.NET: fix: do not forward headers on redirect#8164
Vincent Biret (baywet) merged 21 commits into
mainfrom
fix/redirection-headers

Conversation

@baywet

@baywet Vincent Biret (baywet) commented Sep 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

This prevents forwarding requests headers to subsequent requests on location for 3XX responses and prescribed by RFC 9110

@baywet Vincent Biret (baywet) self-assigned this Sep 8, 2026
Copilot AI balanced review requested due to automatic review settings September 8, 2026 18:28
@agent-framework-automation agent-framework-automation Bot added .NET Usage: [Issues, PRs], Target: .Net workflows Usage: [Issues, PRs], Target: Workflows labels Sep 8, 2026
@github-actions github-actions Bot changed the title fix: do not forward headers on redirect .NET: fix: do not forward headers on redirect Sep 8, 2026

This comment was marked as outdated.

Sign in to view

github-code-quality[bot]
github-code-quality Bot found potential problems Sep 8, 2026
View reviewed changes
Comment thread .../tests/Microsoft.Agents.AI.Workflows.Declarative.UnitTests/DefaultHttpRequestHandlerTests.cs Fixed
Comment thread .../tests/Microsoft.Agents.AI.Workflows.Declarative.UnitTests/DefaultHttpRequestHandlerTests.cs Fixed
Comment thread .../tests/Microsoft.Agents.AI.Workflows.Declarative.UnitTests/DefaultHttpRequestHandlerTests.cs Fixed
Comment thread .../tests/Microsoft.Agents.AI.Workflows.Declarative.UnitTests/DefaultHttpRequestHandlerTests.cs Fixed
Comment thread .../tests/Microsoft.Agents.AI.Workflows.Declarative.UnitTests/DefaultHttpRequestHandlerTests.cs Fixed
Comment thread .../tests/Microsoft.Agents.AI.Workflows.Declarative.UnitTests/DefaultHttpRequestHandlerTests.cs Fixed
Comment thread .../tests/Microsoft.Agents.AI.Workflows.Declarative.UnitTests/DefaultHttpRequestHandlerTests.cs Fixed
github-actions[bot]

This comment was marked as outdated.

Sign in to view

@github-code-quality

This comment was marked as outdated.

Sign in to view
This was referenced Sep 12, 2026
Closed
Open
Open
Merged
Closed
Closed
Merged
Merged
Open
Open
Open
Open
Open
Open
Open
Merged
Open
Merged
Merged
Open
Open
Closed
Open
Open
Open
Open
Open
Open
@baywet Vincent Biret (baywet) mentioned this pull request Sep 16, 2026
Merged
@dependabot dependabot Bot mentioned this pull request Sep 16, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments
Copilot code review Copilot
Copilot review effort, defaults to Balanced
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
Copilot code review Copilot Copilot left review comments
@rogerbarreto Roger Barreto (rogerbarreto) rogerbarreto approved these changes
@SergeyMenshykh SergeyMenshykh SergeyMenshykh approved these changes
@westey-m westey (westey-m) westey-m approved these changes
@chetantoshniwal chetantoshniwal Awaiting requested review from chetantoshniwal
@peibekwe Peter Ibekwe (peibekwe) Awaiting requested review from peibekwe peibekwe is a code owner
+1 more reviewer
@github-code-quality github-code-quality[bot] github-code-quality[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

@baywet Vincent Biret (baywet)

Labels

.NET Usage: [Issues, PRs], Target: .Net workflows Usage: [Issues, PRs], Target: Workflows

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@baywet @rogerbarreto @SergeyMenshykh @westey-m