[PR #9029] [MERGED] Add dts to list of audio codecs which require ffmpeg strict -2 #11769

New issue

Closed

opened 2025-12-22 08:56:04 +01:00 by backuprepo · 0 comments

backuprepo commented

2025-12-22 08:56:04 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/9029
Author: @DavidFair
Created: 1/7/2023
Status: ✅ Merged
Merged: 1/9/2023
Merged by: @Bond-009

Base: master ← Head: Support_dts_transcoding

📝 Commits (2)

4eeb522 Add dts to list of audio codecs which require strict -2
abeadc6 Add DavidFair to contributors

📊 Changes

2 files changed (+4 additions, -2 deletions)

View changed files

📝 CONTRIBUTORS.md (+1 -0)
📝 Jellyfin.Api/Controllers/DynamicHlsController.cs (+3 -2)

📄 Description

Changes
Adds dts to the list of audio codecs where ffmpeg will throw asking us to opt into experimental support. This is seen when the original content is based on dts and we don't acopy using ffmpeg.

Note: I haven't tested this locally, as my server is currently 200+ miles away from me with no way to remotely install a new version. Looking at my logs and the logs in the original this happens on DTS media that's transcoded, so it should be easy to test...

Issues
Fixes #8804

_{🔄 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/9029 **Author:** [@DavidFair](https://github.com/DavidFair) **Created:** 1/7/2023 **Status:** ✅ Merged **Merged:** 1/9/2023 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `master` ← **Head:** `Support_dts_transcoding` --- ### 📝 Commits (2) - [`4eeb522`](https://github.com/jellyfin/jellyfin/commit/4eeb522144037d9102b63478d7d5c0195a1ceef8) Add dts to list of audio codecs which require strict -2 - [`abeadc6`](https://github.com/jellyfin/jellyfin/commit/abeadc62daccd5d0df3b4d099df4e718f27b21ad) Add DavidFair to contributors ### 📊 Changes **2 files changed** (+4 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `CONTRIBUTORS.md` (+1 -0) 📝 `Jellyfin.Api/Controllers/DynamicHlsController.cs` (+3 -2) </details> ### 📄 Description **Changes** Adds dts to the list of audio codecs where ffmpeg will throw asking us to opt into experimental support. This is seen when the original content is based on dts and we don't acopy using ffmpeg. Note: I haven't tested this locally, as my server is currently 200+ miles away from me with no way to remotely install a new version. Looking at my logs and the logs in the original this happens on DTS media that's transcoded, so it should be easy to test... **Issues** Fixes #8804 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>