Thanks to visit codestin.com
Credit goes to github.com

Skip to content

stream.hls: remove deprecated url_master arg+attr#6688

Merged
bastimeyer merged 1 commit into
streamlink:masterfrom
bastimeyer:stream/hls/remove-deprecated-url-master
Oct 26, 2025
Merged

stream.hls: remove deprecated url_master arg+attr#6688
bastimeyer merged 1 commit into
streamlink:masterfrom
bastimeyer:stream/hls/remove-deprecated-url-master

Conversation

@bastimeyer

Copy link
Copy Markdown
Member

This is mainly just used by internal code, but since it's part of the class constructors which are public interfaces, it needed a deprecation phase.

It is very unlikely that third-party plugin implementors will ever create HLSStream/MuxedHLSStream instances with a reference to a multivariant/master playlist themselves. This is done automatically via the HLSStream.parse_variant_playlist() classmethod.


Just for additional clarification, this is not about the "multivariant" and "master" terminology. The "master" name is still part of RFC 8216, and it's also still used by the M3U8 implementation (is_master). The multivariant argument and attribute names were added with respect of the (still-in-draft-phase) HLS revisions, where this was renamed.

The switch from a simple multivariant playlist URL reference to the M3U8 instance was done in 7cb0ebe.

This is mainly just used by internal code, but since it's part
of the class constructors which are public interfaces, it needed
a deprecation phase.

It is very unlikely that third-party plugin implementors will ever
create `HLSStream`/`MuxedHLSStream` instances with a reference
to a multivariant/master playlist themselves. This is done automatically
via the `HLSStream.parse_variant_playlist()` classmethod.
@bastimeyer bastimeyer marked this pull request as ready for review October 26, 2025 14:52
@bastimeyer bastimeyer merged commit 9e3cb99 into streamlink:master Oct 26, 2025
20 checks passed
@bastimeyer bastimeyer deleted the stream/hls/remove-deprecated-url-master branch October 26, 2025 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant