[PR #13182] [MERGED] Don't fall back to ffprobe results for multi-value audio tags #13495

New Issue

OVERLORD · 2026-02-07T07:13:54+03:00

OVERLORD commented

2026-02-07 07:13:54 +03:00

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/13182
Author: @gnattu
Created: 12/8/2024
Status: ✅ Merged
Merged: 12/21/2024
Merged by: @Bond-009

Base: release-10.10.z ← Head: no-multivalue-ffprobe-fallback

📝 Commits (1)

d49bb1d Don't fall back to ffprobe results for multi-value audio tags

📊 Changes

1 file changed (+3 additions, -3 deletions)

View changed files

📝 MediaBrowser.Providers/MediaInfo/AudioFileProber.cs (+3 -3)

📄 Description

Changes

The intention behind using ffprobe as a fallback for the ATL library was to obtain information when ATL failed due to errors, hoping that ffprobe would be more robust. However, for multi-value tags, the old ffprobe normalizer still incorporates a lot of Emby-era logic. Such logic includes multi-level fallbacks and splits tags using delimiters that are not user-customizable. For instance, the normalizer might prefer the ARTISTS tag even if the user has disabled it in the library settings, or use delimiters that the user has excluded. If ATL fails to retrieve any data for those multi-value fields, the tag should be considered empty to avoid wrong data be filled.

Issues

Fixes #12981

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/13182 **Author:** [@gnattu](https://github.com/gnattu) **Created:** 12/8/2024 **Status:** ✅ Merged **Merged:** 12/21/2024 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `release-10.10.z` ← **Head:** `no-multivalue-ffprobe-fallback` --- ### 📝 Commits (1) - [`d49bb1d`](https://github.com/jellyfin/jellyfin/commit/d49bb1d86da60fd9dc658f7554fc1aaa958f9c71) Don't fall back to ffprobe results for multi-value audio tags ### 📊 Changes **1 file changed** (+3 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `MediaBrowser.Providers/MediaInfo/AudioFileProber.cs` (+3 -3) </details> ### 📄 Description  **Changes**  The intention behind using ffprobe as a fallback for the ATL library was to obtain information when ATL failed due to errors, hoping that ffprobe would be more robust. However, for multi-value tags, the old ffprobe normalizer still incorporates a lot of Emby-era logic. Such logic includes multi-level fallbacks and splits tags using delimiters that are not user-customizable. For instance, the normalizer might prefer the ARTISTS tag even if the user has disabled it in the library settings, or use delimiters that the user has excluded. If ATL fails to retrieve any data for those multi-value fields, the tag should be considered empty to avoid wrong data be filled. **Issues**  Fixes #12981 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2026-02-07 07:13:54 +03:00

OVERLORD closed this issue

2026-02-07 07:13:57 +03:00

Sign in to join this conversation.

Branches Tags

master

release-10.11.z

renovate/skiasharp-monorepo

JPVenson-patch-1

renovate/morestachio-5.x

release-10.9.z

openapi-cache

release-10.10.z

explicit-load

explicit-load-v2

feature/entity-framework

only-count-folder-item

fix/ci-workflow-refactor

release-10.8.z

release-10.7.z

EraYaN-add-management-interface

release-10.6.z

release-10.5.z

release-10.4.z

release-10.3.z

release-10.2.z

release-10.1.0

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/jellyfin#13495