Sentry: fixed capturing exception/messages using user.username instea…

…d of custom fields
Samagra-Development · Jul 20, 2023 · a1fbfca · a1fbfca
1 parent 570d2c6
commit a1fbfca
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 8 deletions.
diff --git a/src/api/api.controller.ts b/src/api/api.controller.ts
@@ -65,7 +65,7 @@ export class ApiController {
       if (!total || total == 0) {
         Sentry.captureMessage('Phone number not registered', {
           user: {
-            phone: phone,
+            username: phone,
             applicationId: applicationId
           }
         });

diff --git a/src/api/fusionauth/fusionauth.service.ts b/src/api/fusionauth/fusionauth.service.ts
@@ -267,6 +267,7 @@ export class FusionauthService {
         Sentry.captureException(e, {
           user: {
             applicationId: applicationId,
+            username: user.user.username,
             user: user
           }
         });
@@ -310,7 +311,8 @@ export class FusionauthService {
           user: {
             id: userId,
             applicationId: applicationId,
-            user: user
+            user: user,
+            username: user.user.username,
           }
         });
         console.log(`Could not update user ${user.user.id}`, JSON.stringify(e));

diff --git a/src/api/sms/cdac/cdac.service.ts b/src/api/sms/cdac/cdac.service.ts
@@ -65,7 +65,7 @@ export class CdacService extends SmsService implements SMS {
     } catch (error) {
       Sentry.captureException(error, {
         user: {
-          phone: data.phone
+          username: data.phone
         }
       });
       throw new HttpException('TOTP generation failed!', 500);
@@ -101,7 +101,7 @@ export class CdacService extends SmsService implements SMS {
       .catch((e: Error): OTPResponse => {
         Sentry.captureException(e, {
           user: {
-            phone: data.phone
+            username: data.phone
           }
         });
         const error: SMSError = {

diff --git a/src/api/sms/gupshup/gupshup.service.ts b/src/api/sms/gupshup/gupshup.service.ts
@@ -96,7 +96,7 @@ export class GupshupService extends SmsService implements SMS {
         if (r.error) {
           Sentry.captureMessage(JSON.stringify(r), {
             user: {
-              phone: data.phone
+              username: data.phone
             }
           });
         }
@@ -110,7 +110,7 @@ export class GupshupService extends SmsService implements SMS {
       .catch((e: Error): OTPResponse => {
         Sentry.captureException(e, {
           user: {
-            phone: data.phone
+            username: data.phone
           }
         });
         const error: SMSError = {
@@ -174,7 +174,7 @@ export class GupshupService extends SmsService implements SMS {
         if (r.error) {
           Sentry.captureMessage(JSON.stringify(r), {
             user: {
-              phone: data.phone
+              username: data.phone
             }
           });
         }
@@ -188,7 +188,7 @@ export class GupshupService extends SmsService implements SMS {
       .catch((e: Error): OTPResponse => {
         Sentry.captureException(e, {
           user: {
-            phone: data.phone
+            username: data.phone
           }
         });
         const error: SMSError = {