@sailfish-ai/recorder 1.2.7 → 1.2.8

package/dist/index.js

@@ -157,7 +157,7 @@ export function matchUrlWithWildcard(url, patterns) {
         return true;
     });
 }
-// Updated XMLHttpRequest interceptor with single check function
+// Updated XMLHttpRequest interceptor with CORS protection
 function setupXMLHttpRequestInterceptor(domainsToNotPropagateHeaderTo, domainsToPropagateHeadersTo = []) {
     const originalOpen = XMLHttpRequest.prototype.open;
     const originalSend = XMLHttpRequest.prototype.send;
@@ -167,31 +167,47 @@ function setupXMLHttpRequestInterceptor(domainsToNotPropagateHeaderTo, domainsTo
         ...DOMAINS_TO_NOT_PROPAGATE_HEADER_TO_DEFAULT,
         ...domainsToNotPropagateHeaderTo,
     ];
-    // Store URL during open()
+    // Exclude certain resource types (like images) from being intercepted
+    const resourceExcludeList = ["image", "font", "script"];
+    // Intercept open()
     XMLHttpRequest.prototype.open = function (method, url, ...args) {
-        console.log("Sailfish XMLHttpRequest.prototype.open");
+        console.log("Sailfish XMLHttpRequest.prototype.open", method, url);
         this._requestUrl = typeof url === "string" && url.length > 0 ? url : null;
+        // Skip interception for excluded resource types (e.g., images)
+        const resourceType = this.getResponseHeader("content-type");
+        if (resourceType &&
+            resourceExcludeList.some((type) => resourceType.includes(type))) {
+            console.log(`[XML] Skipping resource request for ${url} (type: ${resourceType})`);
+            return originalOpen.apply(this, [method, url, ...args]);
+        }
         return originalOpen.apply(this, [method, url, ...args]);
     };
     // Intercept send()
     XMLHttpRequest.prototype.send = function (...args) {
-        console.log("Sailfish XMLHttpRequest.prototype.send");
+        console.log("Sailfish XMLHttpRequest.prototype.send", this._requestUrl);
         const url = this._requestUrl;
         if (!url)
             return originalSend.apply(this, args);
-        // Check if domain should be ignored based on combined ignore list
+        // Skip domain check for certain resources like images or fonts
         if (matchUrlWithWildcard(url, combinedIgnoreDomains)) {
-            console.log([`[XML] [InIgnoreDomains] IGNORE --> ${url}`]);
+            console.log(`[XML] [InIgnoreDomains] IGNORE --> ${url}`);
             return originalSend.apply(this, args);
         }
         // Check if domain should propagate headers
         const shouldPropagateHeader = domainsToPropagateHeadersTo.length === 0 ||
             matchUrlWithWildcard(url, domainsToPropagateHeadersTo);
         if (sessionId && shouldPropagateHeader) {
-            console.log([`[XML] [InPropagateDomans] PROPAGATE FOR --> ${url}`]);
-            this.setRequestHeader("X-Sf3-Rid", sessionId);
+            console.log(`[XML] [InPropagateDomains] PROPAGATE FOR --> ${url}`);
+            try {
+                this.setRequestHeader("X-Sf3-Rid", sessionId);
+            }
+            catch (e) {
+                console.warn(`Could not set X-Sf3-Rid header for ${url}`, e);
+            }
+        }
+        else {
+            console.log(`[XML] [NOT InPropagateDomains] IGNORE --> ${url}`);
         }
-        console.log([`[XML] [NOT InPropagateDomans] IGNORE --> ${url}`]);
         return originalSend.apply(this, args);
     };
 }