[PR #2573] [MERGED] Add codecs, resolution and frame-rate fields to HLS master playlist #8978

New issue

Closed

opened 2025-12-22 07:14:04 +01:00 by backuprepo · 0 comments

backuprepo commented

2025-12-22 07:14:04 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/2573
Author: @YouKnowBlom
Created: 3/11/2020
Status: ✅ Merged
Merged: 5/1/2020
Merged by: @Bond-009

Base: master ← Head: add-hls-fields

📝 Commits (5)

85da156 Refactor DynamicHlsService.AppendPlaylist to use StringBuilder
f285887 Add CODECS field to HLS master playlist
8a990d1 Add FRAME-RATE field to HLS master playlist
0a2d24a Add RESOLUTION field to HLS master playlist
153ea9f Fix error in HLS codecs field when level is null

📊 Changes

2 files changed (+315 additions, -8 deletions)

View changed files

📝 MediaBrowser.Api/Playback/Hls/DynamicHlsService.cs (+189 -8)
➕ MediaBrowser.Api/Playback/Hls/HlsCodecStringFactory.cs (+126 -0)

📄 Description

According to the HLS spec, these are all recommended fields a EXT-X-STREAM-INF should have.

Spec

Changes
Adds CODECS field specifying what codecs the stream contains
Adds RESOLUTION field specifying what resolution the stream contains
Adds FRAME-RATE field specifying what framerate the video in the stream has

EDIT:
Move BANDWIDTH and AVERAGE-BANDWIDTH field generation to its own method
Move SUBTITLES field generation to its own method

Issues
#2505

_{🔄 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/2573 **Author:** [@YouKnowBlom](https://github.com/YouKnowBlom) **Created:** 3/11/2020 **Status:** ✅ Merged **Merged:** 5/1/2020 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `master` ← **Head:** `add-hls-fields` --- ### 📝 Commits (5) - [`85da156`](https://github.com/jellyfin/jellyfin/commit/85da15685f7a761af3a34f1c591bf129aa5deb5f) Refactor DynamicHlsService.AppendPlaylist to use StringBuilder - [`f285887`](https://github.com/jellyfin/jellyfin/commit/f2858878d166df214aee20f2dc0710b766285c91) Add CODECS field to HLS master playlist - [`8a990d1`](https://github.com/jellyfin/jellyfin/commit/8a990d1d95aa22840bae5c3494cb5371bcf2b4d8) Add FRAME-RATE field to HLS master playlist - [`0a2d24a`](https://github.com/jellyfin/jellyfin/commit/0a2d24aff3d2e78c97b4b2294a418e2cd4a16be1) Add RESOLUTION field to HLS master playlist - [`153ea9f`](https://github.com/jellyfin/jellyfin/commit/153ea9f027d038ae86f644348eac7b43778d751d) Fix error in HLS codecs field when level is null ### 📊 Changes **2 files changed** (+315 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `MediaBrowser.Api/Playback/Hls/DynamicHlsService.cs` (+189 -8) ➕ `MediaBrowser.Api/Playback/Hls/HlsCodecStringFactory.cs` (+126 -0) </details> ### 📄 Description According to the HLS spec, these are all recommended fields a `EXT-X-STREAM-INF` should have. [Spec](https://tools.ietf.org/html/rfc8216#section-4.3.4.2) **Changes** Adds CODECS field specifying what codecs the stream contains Adds RESOLUTION field specifying what resolution the stream contains Adds FRAME-RATE field specifying what framerate the video in the stream has EDIT: Move BANDWIDTH and AVERAGE-BANDWIDTH field generation to its own method Move SUBTITLES field generation to its own method **Issues** #2505 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

backuprepo

2025-12-22 07:14:04 +01:00

closed this issue
added the
pull-request
label

backuprepo referenced this issue

2025-12-22 08:55:31 +01:00

[PR #8978] [MERGED] Fix multi cleaning #11749

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: starred/jellyfin#8978

No description provided.

Rows
Columns