Error handling for PlayReady key with no status change (#7527)

* Error handling for PlayReady key with no status change
#7508

* Remove hanging promises

* Log append queing

Author: robwalch

api-extractor/report/hls.js.api.md

@@ -1170,6 +1170,8 @@ export class EMEController extends Logger implements ComponentAPI {
     // (undocumented)
     destroy(): void;
     // (undocumented)
+    getKeyStatus(decryptdata: LevelKey): MediaKeyStatus | undefined;
+    // (undocumented)
     getKeySystemAccess(keySystemsToAttempt: KeySystems[]): Promise<void>;
     // (undocumented)
     getSelectedKeySystemFormats(): KeySystemFormats[];
@@ -3379,6 +3381,8 @@ export class LevelKey implements DecryptData {
     // (undocumented)
     pssh: Uint8Array<ArrayBuffer> | null;
     // (undocumented)
+    static setKeyIdForUri(uri: string, keyId: Uint8Array<ArrayBuffer>): void;
+    // (undocumented)
     readonly uri: string;
 }
 
@@ -3992,7 +3996,11 @@ export interface MediaKeySessionContext {
     // (undocumented)
     decryptdata: LevelKey;
     // (undocumented)
-    keyStatus: MediaKeyStatus;
+    keyStatus?: MediaKeyStatus;
+    // (undocumented)
+    keyStatusTimeouts?: {
+        [keyId: string]: number;
+    };
     // (undocumented)
     keySystem: KeySystems;
     // (undocumented)

src/controller/base-stream-controller.ts

@@ -24,7 +24,11 @@ import {
   getAesModeFromFullSegmentMethod,
   isFullSegmentEncryption,
 } from '../utils/encryption-methods-util';
-import { getRetryDelay, offlineHttpStatus } from '../utils/error-helper';
+import {
+  getRetryDelay,
+  isUnusableKeyError,
+  offlineHttpStatus,
+} from '../utils/error-helper';
 import {
   addEventListener,
   removeEventListener,
@@ -727,7 +731,9 @@ export default class BaseStreamController
     ) {
       const media = this.media;
       const error = new Error(
-        `Encrypted track with no key in ${this.fragInfo(frag)} (media ${media ? 'attached mediaKeys: ' + media.mediaKeys : 'detached'})`,
+        __USE_EME_DRM__
+          ? `Encrypted track with no key in ${this.fragInfo(frag)} (media ${media ? 'attached mediaKeys: ' + media.mediaKeys : 'detached'})`
+          : 'EME not supported (light build)',
       );
       this.warn(error.message);
       // Ignore if media is detached or mediaKeys are set
@@ -737,7 +743,7 @@ export default class BaseStreamController
       this.hls.trigger(Events.ERROR, {
         type: ErrorTypes.KEY_SYSTEM_ERROR,
         details: ErrorDetails.KEY_SYSTEM_NO_KEYS,
-        fatal: false,
+        fatal: !__USE_EME_DRM__,
         error,
         frag,
       });
@@ -1052,6 +1058,7 @@ export default class BaseStreamController
         this.handleFragLoadAborted(data.frag, data.part);
       } else if (data.frag && data.type === ErrorTypes.KEY_SYSTEM_ERROR) {
         data.frag.abortRequests();
+        this.resetStartWhenNotLoaded();
         this.resetFragmentLoading(data.frag);
       } else {
         this.hls.trigger(Events.ERROR, data as ErrorData);
@@ -1904,7 +1911,8 @@ export default class BaseStreamController
       noAlternate &&
       isMediaFragment(frag) &&
       !frag.endList &&
-      live
+      live &&
+      !isUnusableKeyError(data)
     ) {
       this.resetFragmentErrors(filterType);
       this.treatAsGap(frag);

src/controller/buffer-controller.ts

@@ -937,6 +937,9 @@ transfer tracks: ${stringify(transferredTracks, (key, value) => (key === 'initSe
         this.hls.trigger(Events.ERROR, event);
       },
     };
+    this.log(
+      `queuing "${type}" append sn: ${sn}${part ? ' p: ' + part.index : ''} of ${frag.type === PlaylistLevelType.MAIN ? 'level' : 'track'} ${frag.level} cc: ${cc}`,
+    );
     this.append(operation, type, this.isPending(this.tracks[type]));
   }