[PR #11259] fix: catch collectible access non-collectible exception #12818

Closed
opened 2026-02-07 07:02:17 +03:00 by OVERLORD · 0 comments
Owner

Original Pull Request: https://github.com/jellyfin/jellyfin/pull/11259

State: closed
Merged: No


Changes

Catch the NotSupportedException caused by plugins directly using the server's XML serializer and throw a new one with informative messages, notifying the plugin developer that directly using this serializer is not a supported use case.

This is an alternative to #11254 to retain the in-process restart functionality.

This change could have a long-term impact, meaning that our plugins need to use their own XML serializers from now on, and this change may not be limited to 10.9, but also in later versions.

Issues

Fixes #11251

**Original Pull Request:** https://github.com/jellyfin/jellyfin/pull/11259 **State:** closed **Merged:** No --- <!-- Ensure your title is short, descriptive, and in the imperative mood (Fix X, Change Y, instead of Fixed X, Changed Y). For a good inspiration of what to write in commit messages and PRs please review https://chris.beams.io/posts/git-commit/ and our documentation. --> **Changes** <!-- Describe your changes here in 1-5 sentences. --> Catch the `NotSupportedException` caused by plugins directly using the server's XML serializer and throw a new one with informative messages, notifying the plugin developer that directly using this serializer is not a supported use case. This is an alternative to #11254 to retain the in-process restart functionality. This change could have a long-term impact, meaning that our plugins need to use their own XML serializers from now on, and this change may not be limited to 10.9, but also in later versions. **Issues** <!-- Tag any issues that this PR solves here. ex. Fixes # --> Fixes #11251
OVERLORD added the pull-request label 2026-02-07 07:02:17 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#12818