Skip to content

Remove Bearer token on logout#925

Merged
TomBursch merged 2 commits intoTomBursch:mainfrom
avdv:remove-bearer-token-auth
Nov 30, 2025
Merged

Remove Bearer token on logout#925
TomBursch merged 2 commits intoTomBursch:mainfrom
avdv:remove-bearer-token-auth

Conversation

@avdv
Copy link
Copy Markdown
Contributor

@avdv avdv commented Nov 23, 2025

  • Remove Bearer token on logout
  • fix typo

@avdv avdv force-pushed the remove-bearer-token-auth branch 2 times, most recently from 8860ce7 to 8d29c4c Compare November 23, 2025 16:45
Copy link
Copy Markdown
Owner

@TomBursch TomBursch left a comment

Choose a reason for hiding this comment

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

Thank you for the fix

avdv added 2 commits November 30, 2025 16:43
After calling `DELETE /api/auth` the auth token should be considered invalid.

Trying to re-use it for login will fail when the logout succeeded server
side.

If the logout did not succeed (e.g. 401 Unauthorized), it is safe to
assume the token is no longer valid anyway.

Also, properly update the connection state (which disconnects from the socket).

Fixes TomBursch#855
@TomBursch TomBursch force-pushed the remove-bearer-token-auth branch from 8d29c4c to fea5ae2 Compare November 30, 2025 15:43
@TomBursch TomBursch merged commit b9d2879 into TomBursch:main Nov 30, 2025
3 checks passed
@TomBursch TomBursch added the bug Something isn't working label Nov 30, 2025
w0s1nsk1 added a commit to w0s1nsk1/kitchenowl that referenced this pull request Feb 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants