@sentry/utils - npm Package Compare versions

Comparing version 8.22.0 to 8.23.0

build/cjs/instrument/fetch.js

@@ -80,8 +80,7 @@ Object.defineProperty(exports, '__esModule', { value: true });
           } else {
-            const finishedHandlerData = {
+            handlers.triggerHandlers('fetch', {
               ...handlerData,
               endTimestamp: time.timestampInSeconds() * 1000,
               response,
-            };
-            handlers.triggerHandlers('fetch', finishedHandlerData);
+            });
           }
@@ -92,25 +91,21 @@
         (error) => {
-          if (!onFetchResolved) {
-            const erroredHandlerData = {
-              ...handlerData,
-              endTimestamp: time.timestampInSeconds() * 1000,
-              error,
-            };
+          handlers.triggerHandlers('fetch', {
+            ...handlerData,
+            endTimestamp: time.timestampInSeconds() * 1000,
+            error,
+          });
 
-            handlers.triggerHandlers('fetch', erroredHandlerData);
-
-            if (is.isError(error) && error.stack === undefined) {
-              // NOTE: If you are a Sentry user, and you are seeing this stack frame,
-              //       it means the error, that was caused by your fetch call did not
-              //       have a stack trace, so the SDK backfilled the stack trace so
-              //       you can see which fetch call failed.
-              error.stack = virtualStackTrace;
-              object.addNonEnumerableProperty(error, 'framesToPop', 1);
-            }
-
+          if (is.isError(error) && error.stack === undefined) {
             // NOTE: If you are a Sentry user, and you are seeing this stack frame,
-            //       it means the sentry.javascript SDK caught an error invoking your application code.
-            //       This is expected behavior and NOT indicative of a bug with sentry.javascript.
-            throw error;
+            //       it means the error, that was caused by your fetch call did not
+            //       have a stack trace, so the SDK backfilled the stack trace so
+            //       you can see which fetch call failed.
+            error.stack = virtualStackTrace;
+            object.addNonEnumerableProperty(error, 'framesToPop', 1);
           }
+
+          // NOTE: If you are a Sentry user, and you are seeing this stack frame,
+          //       it means the sentry.javascript SDK caught an error invoking your application code.
+          //       This is expected behavior and NOT indicative of a bug with sentry.javascript.
+          throw error;
         },
@@ -122,3 +117,3 @@ );
 
-function resolveResponse(res, onFinishedResolving) {
+async function resolveResponse(res, onFinishedResolving) {
   if (res && res.body) {
@@ -149,11 +144,7 @@ const responseReader = res.body.getReader();
 
-    responseReader
+    return responseReader
       .read()
       .then(consumeChunks)
-      .then(() => {
-        onFinishedResolving();
-      })
-      .catch(() => {
-        // noop
-      });
+      .then(onFinishedResolving)
+      .catch(() => undefined);
   }
@@ -168,3 +159,3 @@ }
   } catch (e) {
-    // noop
+    return;
   }
@@ -171,0 +162,0 @@

build/cjs/version.js

 Object.defineProperty(exports, '__esModule', { value: true });
 
-const SDK_VERSION = '8.22.0';
+const SDK_VERSION = '8.23.0';
 
 exports.SDK_VERSION = SDK_VERSION;
 //# sourceMappingURL=version.js.map

build/esm/instrument/fetch.js

@@ -78,8 +78,7 @@ import { isError } from '../is.js';
           } else {
-            const finishedHandlerData = {
+            triggerHandlers('fetch', {
               ...handlerData,
               endTimestamp: timestampInSeconds() * 1000,
               response,
-            };
-            triggerHandlers('fetch', finishedHandlerData);
+            });
           }
@@ -90,25 +89,21 @@
         (error) => {
-          if (!onFetchResolved) {
-            const erroredHandlerData = {
-              ...handlerData,
-              endTimestamp: timestampInSeconds() * 1000,
-              error,
-            };
+          triggerHandlers('fetch', {
+            ...handlerData,
+            endTimestamp: timestampInSeconds() * 1000,
+            error,
+          });
 
-            triggerHandlers('fetch', erroredHandlerData);
-
-            if (isError(error) && error.stack === undefined) {
-              // NOTE: If you are a Sentry user, and you are seeing this stack frame,
-              //       it means the error, that was caused by your fetch call did not
-              //       have a stack trace, so the SDK backfilled the stack trace so
-              //       you can see which fetch call failed.
-              error.stack = virtualStackTrace;
-              addNonEnumerableProperty(error, 'framesToPop', 1);
-            }
-
+          if (isError(error) && error.stack === undefined) {
             // NOTE: If you are a Sentry user, and you are seeing this stack frame,
-            //       it means the sentry.javascript SDK caught an error invoking your application code.
-            //       This is expected behavior and NOT indicative of a bug with sentry.javascript.
-            throw error;
+            //       it means the error, that was caused by your fetch call did not
+            //       have a stack trace, so the SDK backfilled the stack trace so
+            //       you can see which fetch call failed.
+            error.stack = virtualStackTrace;
+            addNonEnumerableProperty(error, 'framesToPop', 1);
           }
+
+          // NOTE: If you are a Sentry user, and you are seeing this stack frame,
+          //       it means the sentry.javascript SDK caught an error invoking your application code.
+          //       This is expected behavior and NOT indicative of a bug with sentry.javascript.
+          throw error;
         },
@@ -120,3 +115,3 @@ );
 
-function resolveResponse(res, onFinishedResolving) {
+async function resolveResponse(res, onFinishedResolving) {
   if (res && res.body) {
@@ -147,11 +142,7 @@ const responseReader = res.body.getReader();
 
-    responseReader
+    return responseReader
       .read()
       .then(consumeChunks)
-      .then(() => {
-        onFinishedResolving();
-      })
-      .catch(() => {
-        // noop
-      });
+      .then(onFinishedResolving)
+      .catch(() => undefined);
   }
@@ -166,3 +157,3 @@ }
   } catch (e) {
-    // noop
+    return;
   }
@@ -169,0 +160,0 @@

build/esm/version.js

@@ -1,4 +0,4 @@
-const SDK_VERSION = '8.22.0';
+const SDK_VERSION = '8.23.0';
 
 export { SDK_VERSION };
 //# sourceMappingURL=version.js.map

build/types-ts3.8/version.d.ts

@@ -1,2 +0,2 @@
-export declare const SDK_VERSION = "8.22.0";
+export declare const SDK_VERSION = "8.23.0";
 //# sourceMappingURL=version.d.ts.map

build/types/version.d.ts

@@ -1,2 +0,2 @@
-export declare const SDK_VERSION = "8.22.0";
+export declare const SDK_VERSION = "8.23.0";
 //# sourceMappingURL=version.d.ts.map

package.json

 {
   "name": "@sentry/utils",
-  "version": "8.22.0",
+  "version": "8.23.0",
   "description": "Utilities for all Sentry JavaScript SDKs",
@@ -42,3 +42,3 @@ "repository": "git://github.com/getsentry/sentry-javascript.git",
   "dependencies": {
-    "@sentry/types": "8.22.0"
+    "@sentry/types": "8.23.0"
   },
@@ -45,0 +45,0 @@ "devDependencies": {

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Worsened metrics

Total package byte prevSize

1156146

decreased by-0.12%

Lines of code

12727

decreased by-0.13%

Dependency changes

• + Added@sentry/types@8.23.0(transitive)

• - Removed@sentry/types@8.22.0(transitive)

• Updated@sentry/types@8.23.0