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

Feature/jwt refresh (see description for testing) #83

Merged
merged 12 commits into from
Jul 26, 2024

Conversation

Alec-Watson
Copy link
Contributor

Changes so that a access JWT may be refreshed and that the correct redirecting occurs.
Test cases:
Token functions can be found in securityFunctions.js

  1. Token refreshing (set the generateToken functions lifetime to less than then the refresh token ie.3s). Token should be refreshed and actions should continue is normal.
  2. Refresh token expiration (set generateRefresh token lifetime to be less than or equal to the token lifetime) upon making requests the user should be redirected to login.
  3. Middleware checks such that the user can navigate when logged in and not when they are not.

@Alec-Watson Alec-Watson self-assigned this Jul 24, 2024
Copy link
Contributor

@JBlixems JBlixems left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noice

@JBlixems JBlixems merged commit c026db0 into develop Jul 26, 2024
6 checks passed
@JBlixems JBlixems deleted the feature/jwtRefresh branch July 26, 2024 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants