Fix: Redirect to login page when receiving 401 unauthorized response from API #1373
Conversation
There was a problem hiding this comment.
Pull request overview
This PR fixes authentication handling by correcting the router guard parameter order and improving 401 unauthorized response handling. The changes ensure users are properly redirected to the login page when their session expires.
- Fixed router guard parameters from incorrect
(from, to, next)to correct(to, from, next)order - Moved 401 error handling from success response handler to error response handler where it belongs
- Added comprehensive redirect logic with fallback to window.location.href
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| ui-vue3/src/main.ts | Fixed router guard parameter order and path check logic; removed unused imports |
| ui-vue3/src/base/http/request.ts | Relocated 401 handling to error interceptor with redirect logic and necessary imports |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…d enhance error logging - Introduced new JavaScript files for YAML and XML syntax highlighting and editing capabilities. - Added a new component for updating YAML configurations with a structured editor interface. - Updated the index.html to reference the new JavaScript bundle for improved functionality. - Enhanced the HTTP request module to log errors during redirection on 401 responses for better debugging.
|
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 87 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
2 participants
Please provide a description of this PR:
main.ts- Fixed router guard implementationrequest.ts- Added 401 redirect logic to HTTP interceptorTo help us figure out who should review this PR, please put an X in all the areas that this PR affects.
Please check any characteristics that apply to this pull request.