Skip to content

Don't throw on logout if session does not exist - #17228

Merged
crobibero merged 1 commit into
jellyfin:masterfrom
Shadowghost:fix-logout-concurrency
Jul 5, 2026
Merged

crobibero merged 1 commit into
jellyfin:masterfrom
Shadowghost:fix-logout-concurrency

Conversation

@Shadowghost

Copy link
Copy Markdown
Contributor

If a client sends multiple logout requests of the same session, first request will delete, second will throw because we are running in a tracked environment but nothing changed.

Changes

  • Delete by device ID instead of using the tracked entity

Code assistance
Noe

Issues

@jellyfin-bot

Copy link
Copy Markdown
Contributor

Jellyfin API (v 12.0.0)


Result


No differences. Specifications are equivalent

@sonarqubecloud

sonarqubecloud Bot commented Jul 3, 2026

Copy link
Copy Markdown

@crobibero
crobibero merged commit 31d05db into jellyfin:master Jul 5, 2026
16 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in Jellyfin v12 Jul 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants