-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
feat: redirect to /login when CSRF expired #14675
Conversation
Codecov Report
@@ Coverage Diff @@
## master #14675 +/- ##
==========================================
- Coverage 77.55% 77.54% -0.01%
==========================================
Files 958 958
Lines 48549 48556 +7
Branches 5702 5702
==========================================
+ Hits 37651 37654 +3
- Misses 10697 10701 +4
Partials 201 201
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!
* feat: redirect to /login when CSRF expired * Show exceptions on API requests * Use is_json
* feat: redirect to /login when CSRF expired * Show exceptions on API requests * Use is_json
* feat: redirect to /login when CSRF expired * Show exceptions on API requests * Use is_json
SUMMARY
Redirect user to
/login
when the CSRF token is expired. Currently this shows a JSON payload with a generic error.BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
N/A
TEST PLAN
I did a
curl
with an expired CSRF token and got redirected:ADDITIONAL INFORMATION