-
Notifications
You must be signed in to change notification settings - Fork 14k
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
fix: Redirect to full url on 401 #19357
Conversation
…redirect-full-url
…bator-superset into fix/redirect-full-url
Codecov Report
@@ Coverage Diff @@
## master #19357 +/- ##
=======================================
Coverage 66.50% 66.51%
=======================================
Files 1684 1686 +2
Lines 64570 64594 +24
Branches 6630 6636 +6
=======================================
+ Hits 42941 42963 +22
- Misses 19931 19932 +1
- Partials 1698 1699 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
* Redirect to full url * Redirect to full url * Update test (cherry picked from commit b8e5954)
🏷️ preset:2022.15 |
* Redirect to full url * Redirect to full url * Update test
SUMMARY
Redirects to full url when a 401 occurs. This is to support the integration of Superset in third-party apps.
TESTING INSTRUCTIONS
next
query param is filled with the full URL, host includedADDITIONAL INFORMATION