Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HLS #EXT-X-ALLOW-CACHE deprecated #878

Closed
andrew867 opened this issue May 10, 2017 · 4 comments
Closed

HLS #EXT-X-ALLOW-CACHE deprecated #878

andrew867 opened this issue May 10, 2017 · 4 comments
Assignees
Labels
Bug It might be a bug. EnglishNative This issue is conveyed exclusively in English.
Milestone

Comments

@andrew867
Copy link

andrew867 commented May 10, 2017

From Apple's mediastreamvalidator:

--------------------------------------------------------------------------------
SHOULD fix issues
--------------------------------------------------------------------------------

Warning: #EXT-X-ALLOW-CACHE has been deprecated
--> Detail:  Line: #EXT-X-ALLOW-CACHE:YES
--> Source:  hls/HLSTest.m3u8

It looks like it is hardcoded:

<< "#EXT-X-ALLOW-CACHE:YES" << SRS_CONSTS_LF;

Would be great to make an option in the HLS config to disable or enable output of that line in HLS m3u8 playlists.

@winlinvip
Copy link
Member

Why deprecated?

@winlinvip winlinvip added the Bug It might be a bug. label May 14, 2017
@winlinvip winlinvip added this to the srs 3.0 release milestone May 14, 2017
@winlinvip
Copy link
Member

Will it cause any problem? Or just a warning? Or is there any substitution for this field?

@mbeacom
Copy link

mbeacom commented May 21, 2017

It appears that it was deprecated in v7 and removed from the draft in v14 https://tools.ietf.org/html/draft-pantos-http-live-streaming-14#section-7 with no direct replacement.

@winlinvip
Copy link
Member

It seems we can remove it.
I will remove it from SRS3.

Stumble added a commit to lino-network/srs that referenced this issue Oct 29, 2018
Stumble added a commit to lino-network/srs that referenced this issue Oct 29, 2018
@winlinvip winlinvip self-assigned this Sep 15, 2021
@winlinvip winlinvip added the EnglishNative This issue is conveyed exclusively in English. label Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug It might be a bug. EnglishNative This issue is conveyed exclusively in English.
Projects
None yet
Development

No branches or pull requests

3 participants