Skip to content

fix: Use token expiration for ephemeral sessions - #61295

Merged
come-nc merged 2 commits into
masterfrom
fix/fix-ephemeral-session-expiration
Jun 15, 2026
Merged

fix: Use token expiration for ephemeral sessions#61295
come-nc merged 2 commits into
masterfrom
fix/fix-ephemeral-session-expiration

Conversation

@come-nc

Copy link
Copy Markdown
Contributor

Summary

Since #56215 ephemeral sessions can be used for all endpoints for 5 minutes, let’s remove the middleware and flagging in-session system and rely on the token expiration stored in database instead.
This avoids any issue related to the PHP session expiring, the renewed session will keep the same expiration.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@come-nccome-nc added this to the Nextcloud 35 milestone Jun 15, 2026
@come-nccome-nc self-assigned this Jun 15, 2026
@come-nc
come-nc requested a review from a team as a code ownerJune 15, 2026 12:54
@come-nccome-nc added the 3. to review Waiting for reviews label Jun 15, 2026
@come-nc
come-nc requested review from CarlSchwan, nfebe, salmart-dev and sorbaugh and removed request for a teamJune 15, 2026 12:54
@come-nc

Copy link
Copy Markdown
ContributorAuthor

/backport to stable34

This simplifies the code a lot.
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
…ession
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc
come-ncforce-pushed the fix/fix-ephemeral-session-expiration branch from 7b3ffa1 to 088dba3CompareJune 15, 2026 13:28
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@come-nc@susnux@cristianscheid