[PR #15807] [MERGED] Enforce more strict webm check #14424

New Issue

OVERLORD · 2026-02-07T07:29:35+03:00

OVERLORD commented

2026-02-07 07:29:35 +03:00

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/15807
Author: @gnattu
Created: 12/16/2025
Status: ✅ Merged
Merged: 12/20/2025
Merged by: @crobibero

Base: release-10.11.z ← Head: more-strict-webm-probing

📝 Commits (2)

879706a Enforce more strict webm check
820b5f9 fix missing test file

📊 Changes

3 files changed (+155 additions, -3 deletions)

View changed files

📝 MediaBrowser.MediaEncoding/Probing/ProbeResultNormalizer.cs (+6 -3)
📝 tests/Jellyfin.MediaEncoding.Tests/Probing/ProbeResultNormalizerTests.cs (+12 -0)
➕ tests/Jellyfin.MediaEncoding.Tests/Test Data/Probing/video_web_like_mkv_with_subtitle.json (+137 -0)

📄 Description

The webm spec only supports video and video streams, and other type of streams in the container should disqualify its webm container type and be treated as mkv.

Some tools might allow users to add subtitle streams to webm and still claim it is webm, but such files are not guranteed to play directly everywhere, and the actuall support is implementation specific.

With the advent of the (unreliable) native mkv support in firefox, this will be required to properly trigger remux for such files.

Changes

Issues

Fixes https://github.com/jellyfin/jellyfin-web/issues/7430

_{🔄 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/15807 **Author:** [@gnattu](https://github.com/gnattu) **Created:** 12/16/2025 **Status:** ✅ Merged **Merged:** 12/20/2025 **Merged by:** [@crobibero](https://github.com/crobibero) **Base:** `release-10.11.z` ← **Head:** `more-strict-webm-probing` --- ### 📝 Commits (2) - [`879706a`](https://github.com/jellyfin/jellyfin/commit/879706a9013bdfcd9d774126ec8c298c9fa2fb03) Enforce more strict webm check - [`820b5f9`](https://github.com/jellyfin/jellyfin/commit/820b5f983ddf4fac142209daddbccce26737cd94) fix missing test file ### 📊 Changes **3 files changed** (+155 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `MediaBrowser.MediaEncoding/Probing/ProbeResultNormalizer.cs` (+6 -3) 📝 `tests/Jellyfin.MediaEncoding.Tests/Probing/ProbeResultNormalizerTests.cs` (+12 -0) ➕ `tests/Jellyfin.MediaEncoding.Tests/Test Data/Probing/video_web_like_mkv_with_subtitle.json` (+137 -0) </details> ### 📄 Description The webm spec only supports video and video streams, and other type of streams in the container should disqualify its webm container type and be treated as mkv. Some tools might allow users to add subtitle streams to webm and still claim it is webm, but such files are not guranteed to play directly everywhere, and the actuall support is implementation specific. With the advent of the (unreliable) native mkv support in firefox, this will be required to properly trigger remux for such files.  **Changes**  **Issues**  Fixes https://github.com/jellyfin/jellyfin-web/issues/7430 --- <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:29:35 +03:00

OVERLORD closed this issue

2026-02-07 07:29:36 +03:00

Sign in to join this conversation.

Branches Tags

master

renovate/major-skiasharp-monorepo

renovate/sharpcompress-0.x

renovate/svg.skia-5.x

renovate/ghcr.io-devcontainers-features-docker-in-docker-4.x

release-10.11.z

standards-fix

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#14424