Moving JSON parse into try catch

micahriggan · micahriggan · commit 9dff32a4af4f · 2019-08-05T11:04:51.000-04:00
diff --git a/packages/bitcore-node/src/routes/api/wallet.ts b/packages/bitcore-node/src/routes/api/wallet.ts
@@ -45,19 +45,19 @@ const authenticate: RequestHandler = async (req: PreAuthRequest, res: Response,
   } catch (err) {
     return res.status(500).send('Problem authenticating wallet');
   }
-
-  if (req.is('application/octet-stream')) {
-    req.body = JSON.parse(req.body.toString());
-  }
-  if (!wallet) {
-    return res.status(404).send('Wallet not found');
-  }
-  Object.assign(req, { wallet });
-  const walletConfig = Config.for('api').wallets;
-  if (walletConfig && walletConfig.allowUnauthenticatedCalls) {
-    return next();
-  }
   try {
+    if (req.is('application/octet-stream')) {
+      req.body = JSON.parse(req.body.toString());
+    }
+    if (!wallet) {
+      return res.status(404).send('Wallet not found');
+    }
+    Object.assign(req, { wallet });
+    const walletConfig = Config.for('api').wallets;
+    if (walletConfig && walletConfig.allowUnauthenticatedCalls) {
+      return next();
+    }
+
     const validRequestSignature = verifyRequestSignature({
       message: [req.method, req.originalUrl, JSON.stringify(req.body)].join('|'),
       pubKey: wallet.pubKey,
