feat(locksmith): better fatal error handling with a logging statement

unlock-protocol · Mar 26, 2024 · 7131390 · 7131390
1 parent 0375dba
commit 7131390
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 4 deletions.
diff --git a/locksmith/src/controllers/purchaseController.ts b/locksmith/src/controllers/purchaseController.ts
@@ -219,7 +219,6 @@ export class PurchaseController {
       )
       logger.info(`Subscription updated for id: ${subscription?.id}`)
     } catch (error) {
-      console.log(error)
       if (response.headersSent) {
         return
       }

diff --git a/locksmith/src/controllers/userController.ts b/locksmith/src/controllers/userController.ts
@@ -63,8 +63,6 @@ export const retrieveEncryptedPrivatekey = async (
     const result = await ethers.Wallet.createRandom().encrypt(
       (Math.random() + 1).toString(36)
     )
-    console.log(result)
-
     return res.json({
       passwordEncryptedPrivateKey: result,
     })

diff --git a/locksmith/src/fulfillment/dispatcher.ts b/locksmith/src/fulfillment/dispatcher.ts
@@ -216,7 +216,7 @@ export default class Dispatcher {
                 new Promise(
                   (resolve) =>
                     (timeout = setTimeout(() => {
-                      console.log(
+                      logger.error(
                         `Timeout while retrieving balance on network ${network.name} (${network.id})`
                       )
                       resolve(0)

diff --git a/locksmith/src/server.ts b/locksmith/src/server.ts
@@ -3,6 +3,13 @@ import logger from './logger'
 
 logger.info('Starting Locksmith...')
 
+process.on('uncaughtException', function (error, origin) {
+  logger.error('Fatal error:', origin)
+  logger.error(error.toString())
+  logger.error(error.stack)
+  process.exit(1)
+})
+
 const port = process.env.PORT || 8080
 
 // in prod, we start immediately