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

Add redirect support #2

Closed
wants to merge 1 commit into from

Conversation

ISilviu
Copy link

@ISilviu ISilviu commented May 10, 2022

Currently, redirects received from the NextJS server are not supported. More specifically, I'm referring to this kind of redirects:

image

I have roughly added support for 308 codes, yet the implementation is not tested nor does it feel complete. Could you please guide me on how to complete this implementation?

@danialkeimasi
Copy link
Member

Hi Silviu,
thank you for your contribution. I've implemented this in 8e2344c since it was easier to implement than explain. Hope that it's going to be useful.

@ISilviu ISilviu deleted the add-redirect-support branch May 11, 2022 04:06
@ISilviu
Copy link
Author

ISilviu commented May 11, 2022

Hi Danial,

Thank you for the solution, looking forward to using it!

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.

None yet

2 participants