Skip to content

[stable28] fix(Session): avoid race conditions on clustered setups - #46436

Merged
blizzz merged 2 commits into
stable28from
backport/46398/stable28
Jul 11, 2024
Merged

[stable28] fix(Session): avoid race conditions on clustered setups#46436
blizzz merged 2 commits into
stable28from
backport/46398/stable28

Conversation

@backportbot

Copy link
Copy Markdown

Backport of PR #46398

blizzz added 2 commits July 11, 2024 07:27
- re-stablishes old behaviour with cache to return null instead of throwing
an InvalidTokenException when the token is cached as non-existing
- token invalidation and re-generation are bundled in a DB transaction now
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
prefer to keep this check as PublicEmitter should be dropped sooner or
later
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzzblizzz mentioned this pull request Jul 11, 2024
@blizzzblizzz added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jul 11, 2024
@blizzz

Copy link
Copy Markdown
Member

Cypress failure is unrelated

@blizzz
blizzz merged commit 7096ef2 into stable28Jul 11, 2024
@blizzz
blizzz deleted the backport/46398/stable28 branch July 11, 2024 11:33
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugfeature: authentication

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@blizzz@AndyScherzinger