videojs / mpd-parser

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

feat: parse content steering info #174

Closed wseymour15 closed 1 year ago

wseymour15 commented 1 year ago

Feature

Add support for parsing <ContentSteering> tags, as well as content (serverURL), the defaultServiceLocation, proxyServerUrl, and queryBeforeStart attributes associated to that tag. This also adds support for <BaseURL tag properties to be included in our parsing of playlists, but specifically for our case, we ensure the serviceLocation property is passed when it exists on a BaseURL tag.

Changes

References

https://datatracker.ietf.org/doc/html/draft-pantos-hls-rfc8216bis#section-4.4.6.6

codecov[bot] commented 1 year ago

Codecov Report

Merging #174 (42a6990) into main (9ece4ae) will increase coverage by 0.07%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #174      +/-   ##
==========================================
+ Coverage   94.58%   94.66%   +0.07%     
==========================================
  Files          19       19              
  Lines         812      824      +12     
  Branches      247      251       +4     
==========================================
+ Hits          768      780      +12     
  Misses         44       44              
Files Changed Coverage Δ
src/index.js 85.71% <ø> (ø)
src/stringToMpdXml.js 100.00% <ø> (ø)
src/inheritAttributes.js 97.70% <100.00%> (+0.14%) :arrow_up:
src/toM3u8.js 98.79% <100.00%> (+0.01%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

dzianis-dashkevich commented 1 year ago

LGTM