apply readystatechange logic to sessions

bugsnag · Mar 13, 2023 · 785ef64 · 785ef64
1 parent b2f804a
commit 785ef64
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/packages/delivery-xml-http-request/delivery.js b/packages/delivery-xml-http-request/delivery.js
@@ -37,9 +37,20 @@ module.exports = (client, win = window) => ({
     try {
       const url = client._config.endpoints.sessions
       const req = new win.XMLHttpRequest()
+
       req.onreadystatechange = function () {
-        if (req.readyState === win.XMLHttpRequest.DONE) cb(null)
+        if (req.readyState === win.XMLHttpRequest.DONE) {
+          const status = req.status
+          if (status === 0 || status >= 400) {
+            const err = new Error(`Request failed with status ${status}`)
+            client._logger.error('Session failed to send…', err)
+            cb(err)
+          } else {
+            cb(null)
+          }
+        }
       }
+
       req.open('POST', url)
       req.setRequestHeader('Content-Type', 'application/json')
       req.setRequestHeader('Bugsnag-Api-Key', client._config.apiKey)