feat(locksmit): better handling of notification errors

locksmith/src/controllers/v2/ticketsController.tsx

@@ -116,7 +116,7 @@ export class TicketsController {
 
       if (event?.data.notifyCheckInUrls) {
         for (const url of event.data.notifyCheckInUrls) {
-          await notify({
+          const response = await notify({
             hookCallback: url,
             body: {
               owner: tokenOwner,
@@ -129,6 +129,13 @@ export class TicketsController {
               },
             },
           })
+          if (response.status !== 200) {
+            logger.info(
+              `Received unxpected response when notifying ${url}: ${
+                response.status
+              } ${await response.text()}`
+            )
+          }
         }
       }
 

locksmith/src/worker/helpers/notify.ts

@@ -41,6 +41,15 @@ export async function notify({
     method: 'POST',
     body: content,
     signal: ac.signal,
+  }).catch((error) => {
+    logger.error('Error in notify function: ', error)
+    return {
+      ok: false,
+      statusText: 'client error',
+      status: '400',
+      text: () => error.message,
+      json: () => Promise.resolve({ message: error.message }),
+    }
   })
 
   clearTimeout(abortTimeout)