fix(app): Datadog error logs cleanup (passportxyz#209)

* fix(app): updates datadog error logging messages * fix(app): adds env variable to sample env
0xVikiZ · Jun 15, 2022 · d336783 · d336783
1 parent b8c8c0a
commit d336783
Show file tree

Hide file tree

Showing 9 changed files with 18 additions and 7 deletions.
diff --git a/app/.env-example.env b/app/.env-example.env
@@ -10,8 +10,9 @@ NEXT_PUBLIC_DPOPP_IAM_URL=http://localhost:80/api/
 NEXT_PUBLIC_DPOPP_IAM_ISSUER_DID=did:key:z6MkghvGHLobLEdj1bgRLhS4LPGJAvbMA1tn2zcRyqmYU5LC
 NEXT_PUBLIC_DPOPP_PROCEDURE_URL=http://localhost:80/procedure/
 
-NEXT_PUBLIC_DPOPP_INFURA_KEY=YOUR_INFURA_KEY
+NEXT_PUBLIC_DPOPP_MAINNET_RPC_URL=YOUR_RPC_URL
 NEXT_PUBLIC_CERAMIC_CLIENT_URL=https://ceramic-clay.3boxlabs.com
 
 NEXT_PUBLIC_DATADOG_APPLICATION_ID=some-uid-id
 NEXT_PUBLIC_DATADOG_CLIENT_TOKEN=somepubclienttoken
+NEXT_PUBLIC_DATADOG_ENV=someenvname
diff --git a/app/components/ProviderCards/BrightidCard.tsx b/app/components/ProviderCards/BrightidCard.tsx
@@ -68,6 +68,7 @@ export default function BrightIdCard(): JSX.Element {
         });
       })
       .catch((e: any): void => {
+        datadogLogs.logger.error("Verification Error", { error: e, provider: providerId });
         datadogRum.addError(`Error ${e}`, { provider: providerId });
       })
       .finally((): void => {
@@ -125,6 +126,7 @@ export default function BrightIdCard(): JSX.Element {
         duration: 9000,
         isClosable: true,
       });
+      datadogLogs.logger.error("Error sponsoring user", { provider: providerId });
       datadogRum.addError(data?.response?.error || "Failed to sponsor user on BrightId", { provider: providerId });
     }
     setCredentialResponseIsLoading(false);
@@ -155,6 +157,7 @@ export default function BrightIdCard(): JSX.Element {
     handleAddStamp(credentialResponse!)
       .then(() => datadogLogs.logger.info("Successfully saved Stamp", { provider: providerId }))
       .catch((e): void => {
+        datadogLogs.logger.error("Error Saving Stamp", { error: e, provider: providerId });
         datadogRum.addError(e, { provider: providerId });
         throw e;
       })

diff --git a/app/components/ProviderCards/EnsCard.tsx b/app/components/ProviderCards/EnsCard.tsx
@@ -58,8 +58,7 @@ export default function EnsCard(): JSX.Element {
         });
       })
       .catch((e: any): void => {
-        datadogLogs.logger.error("1) error providing verification", { error: e, provider: providerId });
-        console.error("2) error providing verification");
+        datadogLogs.logger.error("Verification Error", { error: e, provider: providerId });
         datadogRum.addError(e, { provider: providerId });
       })
       .finally((): void => {
@@ -72,6 +71,7 @@ export default function EnsCard(): JSX.Element {
     handleAddStamp(credentialResponse!)
       .then(() => datadogLogs.logger.info("Successfully saved Stamp", { provider: providerId }))
       .catch((e: any): void => {
+        datadogLogs.logger.error("Error Saving Stamp", { error: e, provider: providerId });
         datadogRum.addError(e, { provider: providerId });
       })
       .finally(() => {

diff --git a/app/components/ProviderCards/FacebookCard.tsx b/app/components/ProviderCards/FacebookCard.tsx
@@ -90,6 +90,7 @@ export default function FacebookCard(): JSX.Element {
         });
       })
       .catch((e): void => {
+        datadogLogs.logger.error("Verification Error", { error: e, provider: providerId });
         datadogRum.addError(e, { provider: providerId });
       })
       .finally(() => {

diff --git a/app/components/ProviderCards/GithubCard.tsx b/app/components/ProviderCards/GithubCard.tsx
@@ -102,6 +102,10 @@ export default function GithubCard(): JSX.Element {
           });
           datadogLogs.logger.info("Successfully saved Stamp", { provider: "Github" });
         })
+        .catch((e) => {
+          datadogLogs.logger.error("Verification Error", { error: e, provider: providerId });
+          throw e;
+        })
         .finally(() => {
           setLoading(false);
         });

diff --git a/app/components/ProviderCards/GoogleCard.tsx b/app/components/ProviderCards/GoogleCard.tsx
@@ -65,8 +65,7 @@ export default function GoogleCard(): JSX.Element {
         });
       })
       .catch((e): void => {
-        datadogLogs.logger.error("1) error providing verification", { error: e, provider: providerId });
-        console.error("2) error providing verification");
+        datadogLogs.logger.error("Verification Error", { error: e, provider: providerId });
         datadogRum.addError(e, { provider: providerId });
       })
       .finally(() => {

diff --git a/app/components/ProviderCards/PoapCard.tsx b/app/components/ProviderCards/PoapCard.tsx
@@ -57,6 +57,7 @@ export default function PoapCard(): JSX.Element {
         });
       })
       .catch((e: any): void => {
+        datadogLogs.logger.error("Verification Error", { error: e, provider: providerId });
         datadogRum.addError(e, { provider: providerId });
       })
       .finally((): void => {

diff --git a/app/components/ProviderCards/PohCard.tsx b/app/components/ProviderCards/PohCard.tsx
@@ -35,7 +35,7 @@ export default function PohCard(): JSX.Element {
   const toast = useToast();
 
   const handleFetchCredential = (): void => {
-    datadogLogs.logger.info("Saving Stamp", { provider: providerId });
+    datadogLogs.logger.info("Starting verification", { provider: providerId });
     setCredentialResponseIsLoading(true);
     fetchVerifiableCredential(
       iamUrl,
@@ -57,6 +57,7 @@ export default function PohCard(): JSX.Element {
         });
       })
       .catch((e: any): void => {
+        datadogLogs.logger.error("Verification Error", { error: e, provider: providerId });
         datadogRum.addError(e, { provider: providerId });
       })
       .finally((): void => {
@@ -68,6 +69,7 @@ export default function PohCard(): JSX.Element {
     handleAddStamp(credentialResponse!)
       .then(() => datadogLogs.logger.info("Successfully saved Stamp", { provider: providerId }))
       .catch((e) => {
+        datadogLogs.logger.error("Error Saving Stamp", { error: e, provider: providerId });
         datadogRum.addError(e, { provider: providerId });
       })
       .finally(() => {

diff --git a/app/components/ProviderCards/TwitterCard.tsx b/app/components/ProviderCards/TwitterCard.tsx
@@ -111,7 +111,7 @@ export default function TwitterCard(): JSX.Element {
           });
         })
         .catch((e) => {
-          datadogRum.addError(e, { provider: providerId });
+          datadogLogs.logger.error("Verification Error", { error: e, provider: providerId });
           throw e;
         })
         .finally(() => {