M3U8 doesn't understand relative paths #11

Closed
jbochi opened this Issue May 17, 2012 · 2 comments

Comments

Projects
None yet
2 participants
Contributor

jbochi commented May 17, 2012

We need to handle segments that have a relative path to the m3u8.

For instance, this is a m3u8 created by FMS 4.5.1:

#EXTM3U
#EXT-X-MEDIA-SEQUENCE:182498
#EXT-X-ALLOW-CACHE:NO
#EXT-X-VERSION:2
#EXT-X-TARGETDURATION:8
#EXTINF:8,
../../../../hls-live/streams/live_hls/events/testevent/testevent/teststreamNum182498.ts
#EXTINF:8,
../../../../hls-live/streams/live_hls/events/testevent/testevent/teststreamNum182499.ts
#EXTINF:8,
../../../../hls-live/streams/live_hls/events/testevent/testevent/teststreamNum182500.ts
#EXTINF:8,
../../../../hls-live/streams/live_hls/events/testevent/testevent/teststreamNum182501.ts
#EXTINF:8,
../../../../hls-live/streams/live_hls/events/testevent/testevent/teststreamNum182502.ts
#EXTINF:8,
../../../../hls-live/streams/live_hls/events/testevent/testevent/teststreamNum182503.ts

hltbra was assigned May 17, 2012

Contributor

hltbra commented May 17, 2012

There is another pathological m3u8 we should cover (with no starting slash):

#EXTM3U
#EXT-X-MEDIA-SEQUENCE:182498
#EXT-X-ALLOW-CACHE:NO
#EXT-X-VERSION:2
#EXT-X-TARGETDURATION:8
#EXTINF:8,
teststreamNum182498.ts

@hltbra hltbra added a commit that referenced this issue May 18, 2012

@hltbra hltbra Fixed issue #11: M3U8 doesn't understand relative paths
All chunk paths are now absolute.
2b72dec
Contributor

hltbra commented May 18, 2012

It is a bad idea to replace content by absolute uris.

It must be changed so Segments and Keys have an accessor called absolute_uri, and let uri unchanged.

Don't know about loads and absolute_uri yet.

@hltbra hltbra added a commit that referenced this issue May 18, 2012

@hltbra hltbra Change M3U8 to have baseuri (issue #11)
Change M3U8 to propagate baseuri to Segments and Keys,
so they can have `.absolute_uri` getter and let `.uri` unchanged.
c9ca10a

hltbra closed this May 18, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment