stormcloud-video-player 0.3.7 → 0.3.9

package/lib/player/StormcloudVideoPlayer.d.cts

@@ -40,6 +40,7 @@ declare class StormcloudVideoPlayer {
     private fetchedAdDurations;
     private targetAdBreakDurationMs;
     private isAdaptiveMode;
+    private failedVastUrls;
     constructor(config: StormcloudVideoPlayerConfig);
     private createAdPlayer;
     load(): Promise<void>;

package/lib/players/HlsPlayer.cjs

@@ -344,12 +344,8 @@ function createImaController(video, options) {
   let adsLoadedReject;
   function makeAdsRequest(google, vastTagUrl) {
     const adsRequest = new google.ima.AdsRequest();
-    const preloadedResponse = preloadedVast.get(vastTagUrl);
-    if (preloadedResponse) {
-      adsRequest.adsResponse = preloadedResponse;
-    } else {
-      adsRequest.adTagUrl = vastTagUrl;
-    }
+    console.log("[IMA] \u{1F4E1} Requesting VAST from URL (letting IMA fetch fresh)");
+    adsRequest.adTagUrl = vastTagUrl;
     const videoWidth = video.offsetWidth || video.clientWidth || 640;
     const videoHeight = video.offsetHeight || video.clientHeight || 360;
     adsRequest.linearAdSlotWidth = videoWidth;
@@ -372,11 +368,8 @@ function createImaController(video, options) {
         console.warn("[IMA] Failed to call setAdWillPlayMuted:", error);
       }
     }
-    adsRequest.vastLoadTimeout = 5e3;
+    adsRequest.vastLoadTimeout = 1e4;
     adsLoader.requestAds(adsRequest);
-    if (preloadedResponse) {
-      preloadedVast.delete(vastTagUrl);
-    }
   }
   function ensurePlaceholderContainer() {
     var _a;
@@ -2091,6 +2084,7 @@ var StormcloudVideoPlayer = class {
     this.fetchedAdDurations = /* @__PURE__ */ new Map();
     this.targetAdBreakDurationMs = null;
     this.isAdaptiveMode = false;
+    this.failedVastUrls = /* @__PURE__ */ new Set();
     initializePolyfills();
     const browserOverrides = getBrowserConfigOverrides();
     this.config = { ...config, ...browserOverrides };
@@ -3070,6 +3064,7 @@ var StormcloudVideoPlayer = class {
       this.vastToMediaUrlMap.clear();
       this.preloadedMediaUrls.clear();
       this.preloadingMediaUrls.clear();
+      this.failedVastUrls.clear();
       const currentMuted = this.video.muted;
       const currentVolume = this.video.volume;
       this.ima.updateOriginalMutedState(currentMuted, currentVolume);
@@ -3270,6 +3265,11 @@ var StormcloudVideoPlayer = class {
     if (this.ima.isAdPlaying()) {
       return;
     }
+    if (this.failedVastUrls.has(vastTagUrl)) {
+      console.warn("[AD-ERROR] Skipping already-failed VAST URL:", vastTagUrl.substring(0, 60));
+      this.handleAdFailure();
+      return;
+    }
     const requestToken = ++this.adRequestTokenCounter;
     this.activeAdRequestToken = requestToken;
     this.startAdRequestWatchdog(requestToken);
@@ -3284,6 +3284,7 @@ var StormcloudVideoPlayer = class {
         await this.ima.play();
       } catch (playError) {
         console.error("[AD-ERROR] Failed to play ad:", playError);
+        this.failedVastUrls.add(vastTagUrl);
         this.clearAdFailsafeTimer();
         if (this.activeAdRequestToken === requestToken) {
           this.activeAdRequestToken = null;
@@ -3293,6 +3294,7 @@ var StormcloudVideoPlayer = class {
       }
     } catch (error) {
       console.error("[AD-ERROR] Ad request failed:", error == null ? void 0 : error.message);
+      this.failedVastUrls.add(vastTagUrl);
       this.clearAdRequestWatchdog();
       this.clearAdFailsafeTimer();
       if (this.activeAdRequestToken === requestToken) {
@@ -3337,7 +3339,8 @@ var StormcloudVideoPlayer = class {
   }
   handleAdFailure() {
     const remaining = this.getRemainingAdMs();
-    if (remaining > 500 && this.adPodQueue.length > 0) {
+    const availableAds = this.adPodQueue.filter((url) => !this.failedVastUrls.has(url)).length;
+    if (remaining > 500 && availableAds > 0) {
       if (this.isAdaptiveMode && this.currentAdIndex <= 1) {
         console.log("[ADAPTIVE-POD] \u23F3 First ad failed, waiting for sequential preload to catch up...");
         setTimeout(() => {
@@ -3354,6 +3357,7 @@ var StormcloudVideoPlayer = class {
         return;
       }
     }
+    console.error("[AD-ERROR] All ads failed or time expired. Failed URLs:", this.failedVastUrls.size);
     this.handleAdPodComplete();
   }
   tryNextAdWithRetry(retryCount) {
@@ -3785,6 +3789,12 @@ var StormcloudVideoPlayer = class {
           );
         }
         mediaUrls = await this.fetchAndParseVastXml(vastTagUrl);
+        if (this.config.debugAdTiming) {
+          console.log(
+            `[StormcloudVideoPlayer] Extracted ${mediaUrls.length} media URLs:`,
+            mediaUrls
+          );
+        }
         if (mediaUrls.length > 0) {
           this.vastToMediaUrlMap.set(vastTagUrl, mediaUrls);
         }
@@ -3809,6 +3819,10 @@ var StormcloudVideoPlayer = class {
     for (let i = 0; i < this.adPodQueue.length; i++) {
       const vastTagUrl = this.adPodQueue[i];
       if (!vastTagUrl) continue;
+      if (this.failedVastUrls.has(vastTagUrl)) {
+        console.warn("[AD-ERROR] Skipping failed URL in queue");
+        continue;
+      }
       const hasImaPreload = (_c = (_b = (_a = this.ima).hasPreloadedAd) == null ? void 0 : _b.call(_a, vastTagUrl)) != null ? _c : false;
       const mediaUrls = this.vastToMediaUrlMap.get(vastTagUrl);
       const hasMediaPreload = mediaUrls && mediaUrls.length > 0 ? this.preloadedMediaUrls.has(mediaUrls[0]) : false;