[FFmpeg-devel] [PATCH] Revert "avformat/hls: fail on probing non hls/m3u8 file extensions"

[Leo Izen]

On 5/14/23 17:39, Michael Niedermayer wrote:
> On Sun, May 14, 2023 at 09:41:29PM +0200, Anton Khirnov wrote:
>> This reverts commit 6b1f68ccb04d791f0250e05687c346a99ff47ea1, which
>> broke many streams in the wild
>>
>> Fixes #10353.
> 
> This change violates a SHOULD in rfc8216 4.  Playlists
> 
>     Each Playlist file MUST be identifiable either by the path component
>     of its URI or by HTTP Content-Type.  In the first case, the path MUST
>     end with either .m3u8 or .m3u.  In the second, the HTTP Content-Type
>     MUST be "application/vnd.apple.mpegurl" or "audio/mpegurl".
>     
>     Clients SHOULD refuse to parse Playlists that are not so identified.
>     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> 

Except that commit doesn't check the mimetype. Some CDNs such as Azure 
Media Services, only use mimetype to identify HLS streams, and not file 
extensions. See: 
https://learn.microsoft.com/en-us/azure/media-services/latest/encode-dynamic-packaging-concept

This commit breaks more than it helps.

- Leo Izen (Traneptora / thebombzen)