Closed sutterj closed 3 months ago
I don't see it here but may have missed it, if refreshing the token for the user doesn't work should we be clearing the session so they are logged out instead of left in the hanging state described in #30 ?
I don't see it here but may have missed it, if refreshing the token for the user doesn't work should we be clearing the session so they are logged out instead of left in the hanging state described in #30 ?
We don't have to clear the session. The middleware that we have forces the user to be authed and the check in AuthProvider.tsx
signs the user out is expired or if we don't have a session. The AuthProvider
is at the highest possible level so it catches all pages.
Pull Request
Proposed Changes
Readiness Checklist
Author/Contributor
npm run lint
and fix any linting issues that have been introducednpm run test
and run tests@jeffrey-luszcz
Reviewer
bug
,documentation
,enhancement
,infrastructure
,maintenance
, orbreaking