ENDED event is not raised on live stream, even though #EXT-X-ENDLIST arrived and no more manifests

[MosheMaorKaltura]

What version of Hls.js are you using?

1.3.5

What browser (including version) are you using?

Chrome Version 115.0.5790.170 (Official Build) (x86_64)

What OS (including version) are you using?

MacOs Ventura 13.2.1

Test stream

https://codepen.io/Semarche/full/NWeKjYE

Configuration

{}
//No special config

Additional player setup steps

No response

Checklist

• The issue observed is not already reported by searching on Github under https://github.com/video-dev/hls.js/issues
• The issue occurs in the stable client (latest release) on https://hlsjs.video-dev.org/demo and not just on my page
• The issue occurs in the latest client (main branch) on https://hlsjs-dev.video-dev.org/demo and not just on my page
• The stream has correct Access-Control-Allow-Origin headers (CORS)
• There are no network errors such as 404s in the browser console when trying to play the stream

Steps to reproduce

1. Open https://codepen.io/Semarche/full/NWeKjYE on you browser
2. Wait for live streaming to start (it start every 3 minutes and plays for 90 seconds)
3. When no more manifests and "#EXT-X-ENDLIST" is returned in the last manifest, video will show an endless spinner
   Notice that the emitted events are monitored below.

Expected behaviour

ENDED event should be emitted.

It worked as expected up until HLSJS version 1.3.0, from that version the event is not being raised.

What actually happened?

Last event emitted is BUFFERING

Console output

None special

Chrome media internals output

No response

[robwalch]

Can the issue be reproduced with the dev and or latest version of HLS.js standalone without kaltura player?

[robwalch]

Hi @MosheMaorKaltura,

The issue is that your live playlist adds the endlist without adding a new segment. HLS.js only recognizes the end list when it is added with the last segment, not after:

Second to last update:

#EXTINF:10.520,
seg-115464510-s32.ts?...

Last update adds ENDLIST only. As it doesn't contain a new segment, end list is not tracked:

#EXTINF:10.520,
seg-115464510-s32.ts?...
#EXT-X-ENDLIST

HLS.js logs no changes before stall error

[log] > [level-controller]: live playlist 0 MISSED

[robwalch]

It's perfectly valid to update the playlist with an end list tag so I came up with a fix.

Please take a look at #5778 and let me know if you or anyone else at Kaltura can review the PR. This fix will go in v1.5.0.

If you need a patch please open a PR against the "patch/v1.4.x" branch.