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

feat: remove session, emit SIGNED_OUT when JWT session_id is invalid #905

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

hf
Copy link
Contributor

@hf hf commented May 12, 2024

When the access token (JWT) contains a session_id property which does not correspond to a row in the sessions table, it means that the user has been signed out or the session has been destroyed in some way. Auth will send back a session_not_found error code which can be detected by the client library and removing of the stored session with emitting the SIGNED_OUT event should take place.

@hf hf force-pushed the hf/sign-out-session-missing branch from 50493b0 to 21e52ec Compare May 12, 2024 16:24
@kangmingtay kangmingtay merged commit db41710 into master Aug 21, 2024
3 checks passed
@kangmingtay kangmingtay deleted the hf/sign-out-session-missing branch August 21, 2024 18:43
kangmingtay pushed a commit that referenced this pull request Aug 30, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.65.0](v2.64.4...v2.65.0)
(2024-08-27)


### Features

* add bindings for Multi-Factor Authentication (Phone)
([#932](#932))
([b957c30](b957c30))
* add kakao to sign in with ID token
([#845](#845))
([e2337ba](e2337ba))
* remove session, emit `SIGNED_OUT` when JWT `session_id` is invalid
([#905](#905))
([db41710](db41710))


### Bug Fixes

* Correct typo in GoTrueClient warning message
([#938](#938))
([8222ee1](8222ee1))
* don't throw error in exchangeCodeForSession
([#946](#946))
([6e161ec](6e161ec))
* move docker compose to v2
([#940](#940))
([38eef89](38eef89))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants