Skip to content

More resilient MusicBrainz lookup - #17670

Merged
crobibero merged 1 commit into
jellyfin:masterfrom
Shadowghost:harden-musicbrainz
Aug 18, 2026
Merged

crobibero merged 1 commit into
jellyfin:masterfrom
Shadowghost:harden-musicbrainz

Conversation

@Shadowghost

Copy link
Copy Markdown
Contributor

No longer valid or straight up invalid IDs from either file or stored metadata can crash MusicBrainz lookups. We should handle this gracefully.

Changes

  • Validate IDs before lookup (also enforces that they are proper GUIDs)
  • Handle 404 responses as null instead of crashing the provider

Code assistance
None

Issues
Fixes #12903

@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 moved this from Todo to Review in Jellyfin v12 Aug 18, 2026
@crobibero
crobibero merged commit 04b77ca into jellyfin:master Aug 18, 2026
16 checks passed
@github-project-automation github-project-automation Bot moved this from Review to Done in Jellyfin v12 Aug 18, 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.

Musicbrainz plugin returns HTTP 404 on AlbumMetadataService

4 participants