Manifest refresh logic

[shaharmor]

Hey @mangui,

I've noticed that the refresh logic for a manifest (located here: https://github.com/dailymotion/hls.js/blob/master/src/controller/level-controller.js#L239) can be problematic in cases where the length of segments is changing drastically within the manifest.

For example, if there is a manifest with segments ranging 2s - 15s, the #EXT-X-TARGETDURATION value will be 15, but its possible there will be 3 (or more) consecutive segments with a total duration of less than 15s, causing the player to stall due to having no knowledge of new segments to load (Until the next tick).

[mangui]

having such fluctuations in fragment duration is a bit weird ?
anyway, it would make more sense to use average or min fragment duration to refresh live playlists.
PR welcome

[gjanblaszczyk]

I added PR using average fragment duration. I hope that is enough for your case @shaharmor

[capagrisdesu]

My understanding of hls is that there is no master playlist (manifest) refetching/refreshing at all within hls.js unless loadSource is being called multiple times. Only media playlists/levels are being refreshed within the logic of the library. Maybe is just a simple naming convention misunderstanding with the title, since the library only uses manifest to refer to the Master playlist. Am I correct?

[stale]

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.