Add support for Rendition Report switching when their relative urls c…

…ontain dot segments (#5124)

Fixes #3572

src/controller/base-playlist-controller.ts

@@ -64,17 +64,25 @@ export default class BasePlaylistController implements NetworkComponentAPI {
 
   protected switchParams(
     playlistUri: string,
-    previous?: LevelDetails
+    previous: LevelDetails | undefined
   ): HlsUrlParameters | undefined {
     const renditionReports = previous?.renditionReports;
     if (renditionReports) {
       for (let i = 0; i < renditionReports.length; i++) {
         const attr = renditionReports[i];
-        const uri = '' + attr.URI;
+        let uri: string;
+        try {
+          uri = new self.URL(attr.URI, previous.url).href;
+        } catch (error) {
+          logger.warn(
+            `Could not construct new URL for Rendition Report: ${error}`
+          );
+          uri = attr.URI || '';
+        }
         if (uri === playlistUri.slice(-uri.length)) {
           const msn = parseInt(attr['LAST-MSN']) || previous?.lastPartSn;
           let part = parseInt(attr['LAST-PART']) || previous?.lastPartIndex;
-          if (previous && this.hls.config.lowLatencyMode) {
+          if (this.hls.config.lowLatencyMode) {
             const currentGoal = Math.min(
               previous.age - previous.partTarget,
               previous.targetduration

src/types/level.ts

@@ -78,7 +78,7 @@ export class HlsUrlParameters {
     if (this.skip) {
       url.searchParams.set('_HLS_skip', this.skip);
     }
-    return url.toString();
+    return url.href;
   }
 }