Skip to content
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

[BUG] Expired session should redirect to Login Page #293

Open
DMalone87 opened this issue Aug 6, 2023 · 1 comment
Open

[BUG] Expired session should redirect to Login Page #293

DMalone87 opened this issue Aug 6, 2023 · 1 comment
Labels
backend bug Something isn't working frontend good first issue Good for newcomers

Comments

@DMalone87
Copy link
Collaborator

DMalone87 commented Aug 6, 2023

Describe the bug
When an API call is made that requires an authenticated user, the app currently fails without giving a clear reason to the user. In cases where the API returns an "Unauthorized" response, we should redirect the user to the login page.

To Reproduce
Steps to reproduce the behavior:

  1. Log in to the application
  2. Visit the search page
  3. Allow the session token to expire, but stay on the search page
  4. Attempt to make a search.

-- OR --

  1. Navigate to the search page by using the URL without being logged in
  2. Attempt to make a search

Expected behavior
When a user attempts to perform a task that requires them to be logged in, they should be taken to a login page.

Screenshots
Screenshot 2023-08-06 at 1 16 30 AM (2)

Additional Context
Server returns a 422 status code in this case.
Response: {"msg":"Signature verification failed"}

@DMalone87 DMalone87 added bug Something isn't working frontend backend labels Aug 16, 2023
@DMalone87 DMalone87 added the good first issue Good for newcomers label Aug 30, 2023
@mikeyavorsky
Copy link
Collaborator

Anirudh is investigating this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend bug Something isn't working frontend good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants