videojs / mpd-parser

https://videojs.github.io/mpd-parser/
Other
78 stars 54 forks source link

manifest endList is true for dynamic MPD instead of false #168

Open erivni opened 1 year ago

erivni commented 1 year ago

Isn't endList should be set to false if manifest type is "dynamic"?

suggestion:

https://github.com/videojs/mpd-parser/blob/3fc0486cbc06114635055d540606f850477c8c5d/src/toM3u8.js#L419

should be changed to:

endList: type === 'static'
gkatsev commented 1 year ago

The other instances of setting nedList do check the type, I think those are more important, but yeah, this should probably reflect the type as well.