-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
feat(frontend): Add authentication debug window #9120
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(frontend): Add authentication debug window #9120
Conversation
…feat(frontend)-add-authentication-debug-window
…feat(frontend)-add-authentication-debug-window
✅ Deploy Preview for inventree-web-pui-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #9120 +/- ##
==========================================
- Coverage 87.37% 87.37% -0.01%
==========================================
Files 1266 1266
Lines 56486 56482 -4
Branches 2064 2061 -3
==========================================
- Hits 49357 49351 -6
- Misses 6621 6624 +3
+ Partials 508 507 -1
🚀 New features to boost your workflow:
|
…feat(frontend)-add-authentication-debug-window
…feat(frontend)-add-authentication-debug-window
…feat(frontend)-add-authentication-debug-window
…feat(frontend)-add-authentication-debug-window
@SchrodingersGat ready for review and merge |
I still think it would be generally useful to be able to see this in the "admin center" view (still only for superusers) to view all logged in session information. Much like the view we had in the old UI. All the API / data is already there, just needs to be formatted into a superuser-only table? |
Not sure; login sessions is something different than what it achieved here. |
…feat(frontend)-add-authentication-debug-window
@matmair this is the last outstanding |
looking into it now |
@SchrodingersGat can you be more specific what errors you are referring to? It seems to work as intended |
@matmair it was just failing the frontend tests, seems to be OK now. I have a slightly adjusted branch which moves this auth session table into the "user settings / security" panel - would you be OK with this? I think this is much cleaner while retaining functionality ![]() |
I guess so. The whole window is only shown for superusers and mostly helpful for quick passkey debugging but 2 extra clicks are probably not too bad in every debug. |
Replaced by #10271 |
Follow up to #6293


Adds a debug window that shows past authentications (only for superusers)
And here after a mandatory re-auth to change or view security-relevant information