Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(authentication-client): Remove access token for fatal 400 errors #2894

Merged
merged 1 commit into from
Nov 25, 2022

Conversation

daffl
Copy link
Member

@daffl daffl commented Nov 25, 2022

This pull request removes the authentication client access token from storage for all 400 errors that are unrecoverable (e.g. 404, Forbidden) etc.

Closes #2835

@netlify
Copy link

netlify bot commented Nov 25, 2022

Deploy Preview for feathers-dove canceled.

Name Link
🔨 Latest commit 2e7ac61
🔍 Latest deploy log https://app.netlify.com/sites/feathers-dove/deploys/638120ea33f3fc00091f4908

@daffl daffl merged commit cfc6c7a into dove Nov 25, 2022
@daffl daffl deleted the auth-client-404 branch November 25, 2022 21:07
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[dove/feathers-authentication] Access token not removed on logout when user is not found
1 participant