Fix a bug in the CommonAccessTokenController.js that leads to null va…

…lues for the token. (#4434)
Dash-Industry-Forum · Mar 21, 2024 · e96f7ce · e96f7ce
1 parent 4ef8359
commit e96f7ce
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/src/streaming/controllers/CommonAccessTokenController.js b/src/streaming/controllers/CommonAccessTokenController.js
@@ -44,8 +44,12 @@ function CommonAccessTokenController() {
         }
 
         const commonAccessTokenHeader = httpResponse.headers[Constants.COMMON_ACCESS_TOKEN_HEADER]
-        const host = Utils.getHostFromUrl(httpResponse.request.url)
-        hostTokenMap[host] = commonAccessTokenHeader
+        if (commonAccessTokenHeader) {
+            const host = Utils.getHostFromUrl(httpResponse.request.url)
+            if (host) {
+                hostTokenMap[host] = commonAccessTokenHeader
+            }
+        }
     }
 
     function getCommonAccessTokenForUrl(url) {

diff --git a/src/streaming/net/HTTPLoader.js b/src/streaming/net/HTTPLoader.js
@@ -249,7 +249,7 @@ function HTTPLoader(cfg) {
                 if (aborted) {
                     if (config.abort) {
                         config.abort(requestObject);
-                    }        
+                    }
                     return;
                 }
 
@@ -285,13 +285,13 @@ function HTTPLoader(cfg) {
 
         };
 
-        const _updateRequestTimingInfo = function() {
+        const _updateRequestTimingInfo = function () {
             requestObject.startDate = requestStartTime;
             requestObject.endDate = new Date();
             requestObject.firstByteDate = requestObject.firstByteDate || requestStartTime;
         }
 
-        const _updateResourceTimingInfo = function() {
+        const _updateResourceTimingInfo = function () {
             httpResponse.resourceTiming.responseEnd = Date.now();
 
             // If enabled the ResourceTimingApi we add the corresponding information to the request object.
@@ -315,7 +315,7 @@ function HTTPLoader(cfg) {
                 });
             });
         }
-        
+
         /**
          * Retriggers the request in case we did not exceed the number of retry attempts
          * @private