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

Fix redirect loop when token is already expired #63 #143

Merged
merged 1 commit into from
Dec 20, 2023
Merged

Conversation

hupf
Copy link
Collaborator

@hupf hupf commented Dec 19, 2023

Siehe #63

Es gab ein Redirect Loop, wenn das Access Token ablief oder wen. die Seite neu geladen wurde und das Access Token war bereits expired. Mit diesem Change wird nur ein Timer gesetzt, wenn das Token nicht bereits abgeloffen ist – bei Tokens die bereits abgelaufen sind greift die normale Logik in auth.ts, wenn man auf die Seite kommt.

Kann getestet werden mit der browserstack Client ID mit npm run start:browserstack (Access Token ist dort 2 Minuten gültig). Für mehr Transparent in logging.ts die Variable const loggingEnabled = true; setzen (ist ein Production Build, deshalb wird sonst nicht geloggt).

@hupf hupf requested a review from caebr December 19, 2023 15:20
@hupf hupf self-assigned this Dec 19, 2023
@hupf hupf merged commit 69a2753 into main Dec 20, 2023
2 checks passed
@hupf hupf deleted the bug/63-redirect-loop branch December 20, 2023 15:26
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