@venncity/errors - npm Package Compare versions

Comparing version 2.1.3 to 2.1.4

CHANGELOG.md

@@ -6,2 +6,13 @@ # Change Log
 
+## [2.1.4](https://github.com/venn-city/npm-shelf/compare/@venncity/errors@2.1.3...@venncity/errors@2.1.4) (2021-03-08)
+
+
+### Bug Fixes
+
+* **errors:** do not fail sentry on missing authDetailsFromRequest ([#1627](https://github.com/venn-city/npm-shelf/issues/1627)) ([5f0950d](https://github.com/venn-city/npm-shelf/commit/5f0950d0c8d2508641c2b1f491a53832abbacd3f))
+
+
+
+
+
 ## [2.1.3](https://github.com/venn-city/npm-shelf/compare/@venncity/errors@2.1.2...@venncity/errors@2.1.3) (2021-02-28)
@@ -8,0 +19,0 @@

lib/cjs/errorReporting/sentry.d.ts

 import * as Sentry from '@sentry/node';
 import { APIGatewayProxyEvent, Context as LambdaContext } from 'aws-lambda';
-export declare function setupSentry(event: APIGatewayProxyEvent, context: LambdaContext, authDetailsFromRequest: AuthDetailsFromRequest): void;
+export declare function setupSentry(event: APIGatewayProxyEvent, context: LambdaContext, authDetailsFromRequest?: AuthDetailsFromRequest): void;
 export declare const sentry: {
@@ -5,0 +5,0 @@ captureException: typeof Sentry.captureException;

lib/cjs/errorReporting/sentry.js

@@ -54,3 +54,3 @@ "use strict";
         Sentry.configureScope((scope) => {
-            if (authDetailsFromRequest.email) {
+            if (authDetailsFromRequest && authDetailsFromRequest.email) {
                 scope.setUser({
@@ -57,0 +57,0 @@ id: authDetailsFromRequest.id,

lib/esm/errorReporting/sentry.d.ts

 import * as Sentry from '@sentry/node';
 import { APIGatewayProxyEvent, Context as LambdaContext } from 'aws-lambda';
-export declare function setupSentry(event: APIGatewayProxyEvent, context: LambdaContext, authDetailsFromRequest: AuthDetailsFromRequest): void;
+export declare function setupSentry(event: APIGatewayProxyEvent, context: LambdaContext, authDetailsFromRequest?: AuthDetailsFromRequest): void;
 export declare const sentry: {
@@ -5,0 +5,0 @@ captureException: typeof Sentry.captureException;

lib/esm/errorReporting/sentry.js

@@ -32,3 +32,3 @@ import { pick, get } from 'lodash';
         Sentry.configureScope((scope) => {
-            if (authDetailsFromRequest.email) {
+            if (authDetailsFromRequest && authDetailsFromRequest.email) {
                 scope.setUser({
@@ -35,0 +35,0 @@ id: authDetailsFromRequest.id,

package.json

 {
   "name": "@venncity/errors",
-  "version": "2.1.3",
+  "version": "2.1.4",
   "author": "Venn Engineering",
@@ -36,3 +36,3 @@ "main": "lib/cjs/index.js",
     "@sentry/types": "^5.22.3",
-    "@venncity/nested-config": "^3.1.8",
+    "@venncity/nested-config": "^3.1.9",
     "@venncity/schema-consts": "^4.2.0",
@@ -46,3 +46,3 @@ "cls-hooked": "^4.2.2",
   },
-  "gitHead": "ff4326da884b33361046d4fc957bb80bb424ea87"
+  "gitHead": "265b9d3d30100eea039635677910f161a2803f75"
 }

src/errorReporting/sentry.ts

@@ -43,3 +43,3 @@ import { pick, get } from 'lodash';
 
-export function setupSentry(event: APIGatewayProxyEvent, context: LambdaContext, authDetailsFromRequest: AuthDetailsFromRequest) {
+export function setupSentry(event: APIGatewayProxyEvent, context: LambdaContext, authDetailsFromRequest?: AuthDetailsFromRequest) {
   if (isSentryEnabled()) {
@@ -49,3 +49,3 @@ const tags = buildTags(event, authDetailsFromRequest, context);
     Sentry.configureScope((scope) => {
-      if (authDetailsFromRequest.email) {
+      if (authDetailsFromRequest && authDetailsFromRequest.email) {
         scope.setUser({
@@ -52,0 +52,0 @@ id: authDetailsFromRequest.id,

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

Improved metrics

Total package byte prevSize

223768

increased by0.21%

Dependency changes

• Updated@venncity/nested-config@^3.1.9