fix: wrap onwarn values in a message object (#1428)

videojs · Oct 18, 2023 · beccfa1 · beccfa1
1 parent 409e8f2
commit beccfa1
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/manifest.js b/src/manifest.js
@@ -93,7 +93,7 @@ export const parseManifest = ({
     }
 
     if (onwarn) {
-      onwarn(`manifest has no targetDuration defaulting to ${targetDuration}`);
+      onwarn({ message: `manifest has no targetDuration defaulting to ${targetDuration}` });
     }
     manifest.targetDuration = targetDuration;
   }
@@ -104,7 +104,7 @@ export const parseManifest = ({
     const partTargetDuration = parts.reduce((acc, p) => Math.max(acc, p.duration), 0);
 
     if (onwarn) {
-      onwarn(`manifest has no partTargetDuration defaulting to ${partTargetDuration}`);
+      onwarn({ message: `manifest has no partTargetDuration defaulting to ${partTargetDuration}` });
       log.error('LL-HLS manifest has parts but lacks required #EXT-X-PART-INF:PART-TARGET value. See https://datatracker.ietf.org/doc/html/draft-pantos-hls-rfc8216bis-09#section-4.4.3.7. Playback is not guaranteed.');
     }
     manifest.partTargetDuration = partTargetDuration;