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

Supporting Flask and Quart's APPLICATION_ROOT #41

Merged
merged 2 commits into from
Oct 19, 2024
Merged

Conversation

rayluo
Copy link
Owner

@rayluo rayluo commented Oct 13, 2024

Thanks for @ralcini for bringing #39 to our attention.

This PR uses an approach different than #40, and added test cases.

@ralcini , would you please help test this in your real world setup, and let us know if it works for you?

The merge of this PR will close #39

@ralcini
Copy link

ralcini commented Oct 16, 2024

Hi @rayluo it seems more "elegant" :-)
it works for me.

@rayluo rayluo merged commit a3aa5fc into dev Oct 19, 2024
6 checks passed
@rayluo rayluo deleted the application-root branch October 19, 2024 20:28
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.

Error managing the "script_root" part of original requested URL on the redirect after login
2 participants