Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Comments

Add a new option to be able to follow redirects with body (307 and 308)#772

Merged
kpacha merged 2 commits intomasterfrom
redirect_with_body
Sep 3, 2025
Merged

Add a new option to be able to follow redirects with body (307 and 308)#772
kpacha merged 2 commits intomasterfrom
redirect_with_body

Conversation

@taik0
Copy link
Contributor

@taik0 taik0 commented Sep 1, 2025

Currently, the body cannot be cloned, and then the redirect is returned directly to the client and generates a 500 (without no-op).

…8). Currently the body cannot be cloned and then the redirect is returned directly to the client and generates a 500 (without no-op).

Signed-off-by: Daniel Ortiz <[email protected]>
@taik0 taik0 requested review from dhontecillas and kpacha September 1, 2025 12:31
Resolved issues in proxy/http_test.go with DeepSource Autofix
Copy link
Contributor

@dhontecillas dhontecillas left a comment

Choose a reason for hiding this comment

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

Looks good to me

@kpacha kpacha merged commit 4461b2a into master Sep 3, 2025
5 checks passed
@kpacha kpacha deleted the redirect_with_body branch September 3, 2025 14:29
@github-actions
Copy link

github-actions bot commented Dec 3, 2025

This pull request was marked as resolved a long time ago and now has been automatically locked as there has not been any recent activity after it. You can still open a new issue and reference this link.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants