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

Append querystring to redirects #304

Merged
merged 4 commits into from
Apr 16, 2022
Merged

Append querystring to redirects #304

merged 4 commits into from
Apr 16, 2022

Conversation

ofhouse
Copy link
Member

@ofhouse ofhouse commented Apr 15, 2022

Changes

  • Appends the original querystring to redirects
  • Cache-Control header is set to public, max-age=0, must-revalidate for redirect responses
  • Content-Type for redirects is now text/plain
  • Adds response body to redirects, e.g. Redirecting to /newplacetwo/some/path?foo=xxx (308)

Fixes #296.

@ofhouse ofhouse merged commit 9e8f86b into main Apr 16, 2022
@ofhouse ofhouse deleted the fix/proxy/redirect branch April 16, 2022 07:47
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.

nextjs redirects strip query parameters
1 participant