🥅 server: set level to sentry exceptions and remove unnecessary one

exactly · Nov 6, 2024 · ec1644f · ec1644f
1 parent 24ef6a1
commit ec1644f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/server/hooks/activity.ts b/server/hooks/activity.ts
@@ -138,20 +138,18 @@ export default app.post(
                     const hash = await startSpan({ name: "deploy account", op: "tx.send" }, () =>
                       keeper.writeContract(request),
                     );
-                    setContext("tx", { transactionHash: hash });
                     setContext("tx", { ...request, transactionHash: hash });
                     const receipt = await startSpan({ name: "tx.wait", op: "tx.wait" }, () =>
                       publicClient.waitForTransactionReceipt({ hash }),
                     );
                     setContext("tx", { ...request, ...receipt });
                     return receipt.status === "success";
                   } catch (error: unknown) {
-                    captureException(error);
+                    captureException(error, { level: "error" });
                     return false;
                   }
                 }))
               ) {
-                captureException(new Error("account deployment reverted"));
                 return;
               }
               await Promise.allSettled(
@@ -182,7 +180,8 @@ export default app.post(
                           publicClient.waitForTransactionReceipt({ hash }),
                         );
                         setContext("tx", { ...request, ...receipt });
-                        if (receipt.status !== "success") captureException(new Error("tx reverted"));
+                        if (receipt.status !== "success")
+                          captureException(new Error("tx reverted"), { level: "error" });
                       } catch (error: unknown) {
                         if (
                           error instanceof BaseError &&
@@ -191,7 +190,7 @@ export default app.post(
                         ) {
                           return;
                         }
-                        captureException(error);
+                        captureException(error, { level: "error" });
                       }
                     },
                   );

diff --git a/server/test/hooks/activity.test.ts b/server/test/hooks/activity.test.ts
@@ -159,7 +159,7 @@ describe("address activity", () => {
 
     const deposits = await waitForDeposit(account, 1);
 
-    expect(captureException).toHaveBeenCalledWith(new Error("account deployment reverted"));
+    expect(captureException).toHaveBeenCalledWith(new Error("Transaction Timeout"), expect.anything());
 
     expect(deposits).toHaveLength(0);
     expect(response.status).toBe(200);