You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We have a single page app with USE_JWT set, and we want users who have authenticated to also be able to access the Django admin console.
In the original version of this library the login function was called first then use_jwt was checked. So a user would get a 302 redirect to the single page app, but would also have cookies set so they could access the Django admin console.
This library checks use_jwt first and redirects the user without setting any cookies.
I'd be happy to do a PR to change this, it would essentially be moving the login section to be above the use_jwt section.
I wanted to check if that's something that would be accepted? or if it was a specific decision that when USE_JWT is set, users are not logged in to the Django console? I did search to see if it had been raised before and found that #271 is pretty similar (not exactly the same, but this might have done what they wanted).
The text was updated successfully, but these errors were encountered:
We have a single page app with
USE_JWT
set, and we want users who have authenticated to also be able to access the Django admin console.In the original version of this library the login function was called first then use_jwt was checked. So a user would get a 302 redirect to the single page app, but would also have cookies set so they could access the Django admin console.
This library checks use_jwt first and redirects the user without setting any cookies.
I'd be happy to do a PR to change this, it would essentially be moving the login section to be above the use_jwt section.
I wanted to check if that's something that would be accepted? or if it was a specific decision that when
USE_JWT
is set, users are not logged in to the Django console? I did search to see if it had been raised before and found that #271 is pretty similar (not exactly the same, but this might have done what they wanted).The text was updated successfully, but these errors were encountered: