Require redundant levels to have matching RESOLUTION and CODECS attri…

…butes Resolves video-dev#3888
brvier · May 12, 2021 · a5df1fa · a5df1fa
1 parent ca70edb
commit a5df1fa
Show file tree

Hide file tree

Showing 2 changed files with 127 additions and 80 deletions.
diff --git a/src/controller/level-controller.ts b/src/controller/level-controller.ts
@@ -84,7 +84,7 @@ export default class LevelController extends BasePlaylistController {
     let audioTracks: MediaPlaylist[] = [];
     let subtitleTracks: MediaPlaylist[] = [];
     let bitrateStart: number | undefined;
-    const levelSet: { [bitrate: number]: Level } = {};
+    const levelSet: { [key: string]: Level } = {};
     let levelFromSet: Level;
     let resolutionFound = false;
     let videoCodecFound = false;
@@ -109,11 +109,12 @@ export default class LevelController extends BasePlaylistController {
         levelParsed.audioCodec = undefined;
       }
 
-      levelFromSet = levelSet[levelParsed.bitrate]; // FIXME: we would also have to match the resolution here
+      const levelKey = `${levelParsed.bitrate}-${levelParsed.attrs.RESOLUTION}-${levelParsed.attrs.CODECS}`;
+      levelFromSet = levelSet[levelKey];
 
       if (!levelFromSet) {
         levelFromSet = new Level(levelParsed);
-        levelSet[levelParsed.bitrate] = levelFromSet;
+        levelSet[levelKey] = levelFromSet;
         levels.push(levelFromSet);
       } else {
         levelFromSet.url.push(levelParsed.url);