update to only add segments if there is a sidx box parsed. Fix tests

videojs · Mar 11, 2019 · f48bb8d · f48bb8d
1 parent 1a1e8d1
commit f48bb8d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/index.js b/src/index.js
@@ -7,7 +7,7 @@ import { parseUTCTimingScheme } from './parseUTCTimingScheme';
 
 export const VERSION = version;
 
-export const parse = (manifestString, options) =>
+export const parse = (manifestString, options = {}) =>
   toM3u8(toPlaylists(inheritAttributes(stringToMpdXml(manifestString), options)), options.sidxMapping);
 
 /**

diff --git a/src/toM3u8.js b/src/toM3u8.js
@@ -47,10 +47,10 @@ const addSegmentInfoFromSidx = (playlists, sidxMapping) => {
     const playlist = playlists[i];
     const sidxKey = playlist.sidx.uri + '-' +
       byteRangeToString(playlist.sidx.byterange);
-    const sidxMatch = sidxMapping[sidxKey];
+    const sidxMatch = sidxMapping[sidxKey] && sidxMapping[sidxKey].sidx;
 
     if (playlist.sidx && sidxMatch) {
-      addSegmentsToPlaylist(playlist, sidxMatch.sidx, playlist.sidx.resolvedUri);
+      addSegmentsToPlaylist(playlist, sidxMatch, playlist.sidx.resolvedUri);
     }
   }