Skip to content

location header rewriting for HTTP 303#1125

Open
jrnail23 wants to merge 1 commit intohttp-party:masterfrom
jrnail23:rewrite-303
Open

location header rewriting for HTTP 303#1125
jrnail23 wants to merge 1 commit intohttp-party:masterfrom
jrnail23:rewrite-303

Conversation

@jrnail23
Copy link
Copy Markdown

resolves #845

This PR enables hostRewrite, autoRewrite, and protocolRewrite on HTTP 303 (See Other) redirects.

@reflexator
Copy link
Copy Markdown

Pls merge this pull request

@mabar
Copy link
Copy Markdown

mabar commented Feb 18, 2018

@jcrugzz Can it be merged, please? 303 code is used after form submit. I am after form submit improperly redirected out of proxy address and it's complicating development with tools like a browsersync

daverted added a commit to daverted/node-http-proxy that referenced this pull request May 9, 2019
pi0 added a commit to unjs/httpxy that referenced this pull request Mar 25, 2026
The `redirectRegex` was missing status code 303 (See Other), so
`hostRewrite`, `autoRewrite`, and `protocolRewrite` did not apply
to 303 responses. Upstream: http-party/node-http-proxy#1125.
pi0 added a commit to unjs/httpxy that referenced this pull request Mar 25, 2026
)

The `redirectRegex` was missing status code 303 (See Other), so
`hostRewrite`, `autoRewrite`, and `protocolRewrite` did not apply
to 303 responses. Upstream: http-party/node-http-proxy#1125.
@pi0
Copy link
Copy Markdown

pi0 commented Mar 25, 2026

This issue has been fixed in unjs/httpxy#119.

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.

Add support of 303 to hostRewrite option

4 participants