Skip to content
This repository has been archived by the owner on Jan 24, 2023. It is now read-only.

VerifySession should not return 401 #4533

Closed
nwmac opened this issue Aug 24, 2020 · 0 comments
Closed

VerifySession should not return 401 #4533

nwmac opened this issue Aug 24, 2020 · 0 comments
Assignees

Comments

@nwmac
Copy link
Contributor

nwmac commented Aug 24, 2020

Currently, the REST API verifySession (which is used to determine is the user has a valid session) returns a 401 (Unauthorised) if they do not and 200 is they do.

This leads to errors in the front-end console log and during e2e tests when the user accesses the UI but is not logged in.

We should change the API to always return 200 and indicate whether the user has a valid session in the response body.

This should be a small change on the back-end, but will require a tweak on the front-end and in the e2e test helpers.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants