Skip to content

bug: ui: user not redirected to sign in after API key invalidated #6133

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.

Sign up for GitHub

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

Jump to bottom
Closed
johnstcn opened this issue Feb 9, 2023 · 0 comments · Fixed by #6870
Closed

bug: ui: user not redirected to sign in after API key invalidated #6133

johnstcn opened this issue Feb 9, 2023 · 0 comments · Fixed by #6870
Assignees
@rodrimaia
Labels
s2 Broken use cases or features (with a workaround). Only humans may set this. site Area: frontend dashboard

Comments

@johnstcn
Copy link
Member

johnstcn commented Feb 9, 2023 •
edited
Loading

Problem:

When a user's API key is invalidated, they are not redirected to login until they perform a hard refresh of the page.

Screen.Recording.2023-02-09.at.14.28.26.mov

Unknown when this behaviour started, but reproducible on current main.

Expected behaviour:

After their credentials expire, the user should be redirected to /login to re-authenticate.

Steps to reproduce:

  1. Login with a user (username+password auth)
  2. Change the password but do not refresh the page
  3. Click anywhere else in the UI and observe errors everywhere
  4. Refresh the page and note the logion page.
@johnstcn johnstcn added bug site Area: frontend dashboard labels Feb 9, 2023
@bpmct bpmct added the s2 Broken use cases or features (with a workaround). Only humans may set this. label Feb 9, 2023
@rodrimaia rodrimaia mentioned this issue Mar 29, 2023
Merged
@matifali matifali added the bug label Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rodrimaia rodrimaia

Labels
s2 Broken use cases or features (with a workaround). Only humans may set this. site Area: frontend dashboard
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(site): fix redirection to login after logout/change password coder/coder
4 participants
@rodrimaia @johnstcn @matifali @bpmct