Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Per-request redirect policy override #2440

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mikayla-maki
Copy link

@mikayla-maki mikayla-maki commented Oct 1, 2024

This is a repeat of #1204, with minimal changes.

Fixes #353

At Zed, our previous HTTP client exposed an API for configuring the redirect policy per-request configuration. We've since used this feature in a few places, as well as exposed it via our extension API. We're now working on upgrading our HTTP client to reqwest and so would like to revive and take responsibility for this old PR!

@mikayla-maki mikayla-maki changed the title 0.12.8 per request redirect Per-request redirect policy override Oct 1, 2024
@mikayla-maki mikayla-maki changed the title Per-request redirect policy override feat: Per-request redirect policy override Oct 1, 2024
@mikayla-maki mikayla-maki force-pushed the 0.12.8-per-request-redirect branch from 6f88db0 to fd110f6 Compare October 1, 2024 04:52
@Sigeth
Copy link

Sigeth commented Nov 18, 2024

I would need this PR also for my project, don't know about the priority but it would be nice if this could be added!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Per-Request RedirectPolicy
2 participants