Consider status codes <200

snowplow · Oct 23, 2023 · f43467e · f43467e
1 parent 1df5e9f
commit f43467e
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/...low/browser-tracker-core/issue-1244-consider-status-codes-under-200_2023-10-20-16-20.json b/...low/browser-tracker-core/issue-1244-consider-status-codes-under-200_2023-10-20-16-20.json
@@ -0,0 +1,10 @@
+{
+  "changes": [
+    {
+      "packageName": "@snowplow/browser-tracker-core",
+      "comment": "Consider status codes <200",
+      "type": "none"
+    }
+  ],
+  "packageName": "@snowplow/browser-tracker-core"
+}
diff --git a/libraries/browser-tracker-core/src/tracker/out_queue.ts b/libraries/browser-tracker-core/src/tracker/out_queue.ts
@@ -371,9 +371,9 @@ export function OutQueueManager(
       };
 
       xhr.onreadystatechange = function () {
-        if (xhr.readyState === 4 && xhr.status >= 200) {
+        if (xhr.readyState === 4) {
           clearTimeout(xhrTimeout);
-          if (xhr.status < 300) {
+          if (xhr.status >= 200 && xhr.status < 300) {
             onPostSuccess(numberToSend);
           } else {
             if (!shouldRetryForStatusCode(xhr.status)) {