http-streaming icon indicating copy to clipboard operation
http-streaming copied to clipboard

Support for EXT-X-START in HLS

Open jimmycallin opened this issue 7 years ago • 20 comments

Hi!

I noticed that video.js doesn't seem to respect EXT-X-START as set in my HLS manifest, e.g:

#EXT-X-START:TIME-OFFSET=0.7,PRECISE=YES

I also noticed there is an abandonded(?) PR by @masterful in videojs-contrib-hls: https://github.com/videojs/videojs-contrib-hls/pull/1299

Is this indeed something that this library doesn't currently support? If possible, I would very much appreciate if it could be a part of your roadmap.

jimmycallin avatar Dec 11 '18 21:12 jimmycallin

👋 Thanks for opening your first issue here! 👋

If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can. To help make it easier for us to investigate your issue, please follow the contributing guidelines.

welcome[bot] avatar Dec 11 '18 21:12 welcome[bot]

👋 hi @jimmycallin - I believe I updated the PR as requested - but I never did figure out what was causing the tests to fail... I can revisit the PR once I understand what's failing 🤷‍♂️

masterful avatar Dec 11 '18 23:12 masterful

@masterful That would be great! Perhaps a first step could be to migrate it to this new repo? :)

jimmycallin avatar Dec 11 '18 23:12 jimmycallin

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Feb 10 '19 00:02 stale[bot]

I would like if this wasn't closed.

jimmycallin avatar Feb 10 '19 20:02 jimmycallin

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Apr 11 '19 21:04 stale[bot]

Still interested

jimmycallin avatar Apr 11 '19 21:04 jimmycallin

interested +1

meikidd avatar Apr 22 '19 07:04 meikidd

interested +1

Hayko-1 avatar Jun 16 '19 12:06 Hayko-1

+1

shay-sundaysky avatar Aug 08 '19 04:08 shay-sundaysky

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Oct 07 '19 05:10 stale[bot]

Oops, yeah, we should have support for this.

gkatsev avatar Oct 15 '19 15:10 gkatsev

Guys any idea when this might go live, would be of great help. Thanks in advance.

saurabhkhandelwal avatar Apr 13 '20 08:04 saurabhkhandelwal

Really need this. Otherwise have to parse m3u8 file and skip in code.

vegetablesalad avatar May 08 '20 11:05 vegetablesalad

@vegetablesalad I did a hack for this, implemented another field as an offset in the API in which I get the stream URL, and then on video meta load I just call this currentTime(offset).

saurabhkhandelwal avatar May 08 '20 11:05 saurabhkhandelwal

guys this seems to be supported already but I could be wrong or misunderstanding this issue, if i run: player.tech().vhs.selectPlaylist() I do se the EXT-X-START params that we are settings being parsed

Screen Shot 2021-08-10 at 13 31 18 Screen Shot 2021-08-10 at 13 30 35

abdul-hamid-achik avatar Aug 10 '21 18:08 abdul-hamid-achik

While it gets parsed, I'm pretty sure we don't currently use it at all.

gkatsev avatar Aug 10 '21 18:08 gkatsev

While it gets parsed, I'm pretty sure we don't currently use it at all.

@gkatsev thanks for answering so fast then ill just do the same thing as @saurabhkhandelwal

abdul-hamid-achik avatar Aug 10 '21 18:08 abdul-hamid-achik

Any update on this? Looked for hours only to find out it's not supported xD

You2499 avatar Nov 18 '21 20:11 You2499

+1

blokfyuh avatar Jan 19 '22 01:01 blokfyuh

+1

ohIcee avatar Apr 08 '24 13:04 ohIcee

This was released in http-streaming v3.3.0, in Video.js 8.4.0+

https://github.com/videojs/http-streaming/blob/main/CHANGELOG.md#330-2023-05-03

mister-ben avatar Apr 08 '24 15:04 mister-ben