Skip to content

Skip computing lifespan for read-only sessions#41084

Merged
ahus1 merged 1 commit intokeycloak:mainfrom
pruivo:t_40980
Jul 10, 2025
Merged

Skip computing lifespan for read-only sessions#41084
ahus1 merged 1 commit intokeycloak:mainfrom
pruivo:t_40980

Conversation

@pruivo
Copy link
Copy Markdown
Member

@pruivo pruivo commented Jul 10, 2025

Fixes #40980

@pruivo pruivo changed the title Skip computing lifespan for read-only sessins Skip computing lifespan for read-only sessions Jul 10, 2025
@pruivo pruivo requested a review from ahus1 July 10, 2025 15:36
Fixes keycloak#40980

Signed-off-by: Pedro Ruivo <1492066+pruivo@users.noreply.github.com>
@pruivo pruivo marked this pull request as ready for review July 10, 2025 18:24
@pruivo pruivo requested a review from a team as a code owner July 10, 2025 18:24
@ahus1 ahus1 merged commit 5219a33 into keycloak:main Jul 10, 2025
77 checks passed
@ahus1
Copy link
Copy Markdown
Member

ahus1 commented Jul 10, 2025

IMHO it is sufficiently covered by the current tests to adding a test is not strictly necessary.

@pruivo pruivo deleted the t_40980 branch July 10, 2025 20:04
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.

Can't update security-admin-console via admin UI with volatile sessions

2 participants