Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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

Merged
daffl merged 1 commit intodovefeathersjs/feathers:dovefrom
auth-client-404feathersjs/feathers:auth-client-404Copy head branch name to clipboard
Nov 25, 2022
Merged

fix(authentication-client): Remove access token for fatal 400 errors#2894
daffl merged 1 commit intodovefeathersjs/feathers:dovefrom
auth-client-404feathersjs/feathers:auth-client-404Copy head branch name to clipboard

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
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.

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

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.