Skip to content

Fix playlist encoding recognition - #17930

Merged
crobibero merged 1 commit into
jellyfin:release-12.zfrom
Shadowghost:fix-playlist-accents
Sep 12, 2026
Merged

crobibero merged 1 commit into
jellyfin:release-12.zfrom
Shadowghost:fix-playlist-accents

Conversation

@Shadowghost

Copy link
Copy Markdown
Contributor
  • PlaylistItemsProvider.GetFromStream(stream) decodes as UTF-8 but .m3u has no encoding declaration and Windows media players still write it in the local codepage leading to every accented character becoming U+FFFD, File.Exists fails, and the entry is silently dropped.

  • Accents encode precomposed (NFC) or decomposed (NFD). A playlist written on a platform preferring the other form never matches byte-for-byte.

Changes
Charset-sniff the plain-text formats via UTF.Unknown

Code assistance
Claude Opus 5 for debugging and the tests.

Issues
Fixes #17925

@Shadowghost Shadowghost added the stable backport Backport into the next stable release label Sep 10, 2026
@jellyfin-bot

Copy link
Copy Markdown
Contributor

Jellyfin API (v 12.0.0)


Result


No differences. Specifications are equivalent

@sonarqubecloud

Copy link
Copy Markdown

@theguymadmax
theguymadmax requested a review from a team September 10, 2026 16:20
@theguymadmax theguymadmax moved this from Todo to Review in Jellyfin v12 Sep 10, 2026
@jellyfin-project-automation jellyfin-project-automation Bot moved this from Review to Approved in Jellyfin v12 Sep 12, 2026
@jellyfin-project-automation jellyfin-project-automation Bot added this to the v12.1 milestone Sep 12, 2026
@crobibero
crobibero merged commit e5d9134 into jellyfin:release-12.z Sep 12, 2026
16 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Done in Jellyfin v12 Sep 12, 2026
joshuaboniface pushed a commit that referenced this pull request Sep 15, 2026
Fix playlist encoding recognition

Original-merge: e5d9134

Merged-by: crobibero <cody@robibe.ro>

Backported-by: Cody Robibero <cody@robibe.ro>
@crobibero crobibero removed the stable backport Backport into the next stable release label Sep 15, 2026
dux-nine added a commit to Start9Labs/advisories that referenced this pull request Sep 15, 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.

5 participants