feat(express): typed router for verifycoinaddress

Typed router for verifycoinaddress

TICKET: WP-5436

Author: danielzhao122

modules/express/encryptedPrivKeys.json

@@ -1,3 +1 @@
-{
-   "61f039aad587c2000745c687373e0fa9": "{\"iv\":\"O74H8BBv86GBpoTzjVyzWw==\",\"v\":1,\"iter\":10000,\"ks\":256,\"ts\":64,\"mode\":\"ccm\",\"adata\":\"\",\"cipher\":\"aes\",\"salt\":\"7n8pAjXCfug=\",\"ct\":\"14MjiKBksaaayrwuc/w8vJ5C3yflQ15//dhLiOgYVqjhJJ7iKrcrjtgfLoI3+MKLaKCycNKi6vTs2xs8xJeSm/XhsOE9EfapkfGHdYuf4C6O1whNOyugZ0ZSOA/buDC3rvBbvCNtLDOxN5XWJN/RADOnZdHuVGk=\"}"
-}
+{}

modules/express/src/clientRoutes.ts

@@ -356,26 +356,18 @@ function handleV2UserREST(req: express.Request, res: express.Response, next: exp
  * handle v2 address validation
  * @param req
  */
-function handleV2VerifyAddress(req: express.Request): { isValid: boolean } {
-  if (!_.isString(req.body.address)) {
-    throw new Error('Expected address to be a string');
-  }
-
-  if (req.body.supportOldScriptHashVersion !== undefined && !_.isBoolean(req.body.supportOldScriptHashVersion)) {
-    throw new Error('Expected supportOldScriptHashVersion to be a boolean.');
-  }
-
+function handleV2VerifyAddress(req: ExpressApiRouteRequest<'express.verifycoinaddress', 'post'>): { isValid: boolean } {
   const bitgo = req.bitgo;
   const coin = bitgo.coin(req.params.coin);
 
   if (coin instanceof Coin.AbstractUtxoCoin) {
     return {
-      isValid: coin.isValidAddress(req.body.address, !!req.body.supportOldScriptHashVersion),
+      isValid: coin.isValidAddress(req.decoded.address, !!req.decoded.supportOldScriptHashVersion),
     };
   }
 
   return {
-    isValid: coin.isValidAddress(req.body.address),
+    isValid: coin.isValidAddress(req.decoded.address),
   };
 }
 
@@ -1720,7 +1712,7 @@ export function setupAPIRoutes(app: express.Application, config: Config): void {
 
   // Miscellaneous
   app.post('/api/v2/:coin/canonicaladdress', parseBody, prepareBitGo(config), promiseWrapper(handleCanonicalAddress));
-  app.post('/api/v2/:coin/verifyaddress', parseBody, prepareBitGo(config), promiseWrapper(handleV2VerifyAddress));
+  router.post('express.verifycoinaddress', [prepareBitGo(config), typedPromiseWrapper(handleV2VerifyAddress)]);
   app.put(
     '/api/v2/:coin/pendingapprovals/:id',
     parseBody,

modules/express/src/typedRoutes/api/index.ts

@@ -12,6 +12,7 @@ import { PostAcceptShare } from './v1/acceptShare';
 import { PostSimpleCreate } from './v1/simpleCreate';
 import { PutPendingApproval } from './v1/pendingApproval';
 import { PostSignTransaction } from './v1/signTransaction';
+import { PostVerifyCoinAddress } from './v2/verifyAddress';
 
 export const ExpressApi = apiSpec({
   'express.ping': {
@@ -44,6 +45,9 @@ export const ExpressApi = apiSpec({
   'express.v1.wallet.signTransaction': {
     post: PostSignTransaction,
   },
+  'express.verifycoinaddress': {
+    post: PostVerifyCoinAddress,
+  },
 });
 
 export type ExpressApi = typeof ExpressApi;

modules/express/test/unit/typedRoutes/decode.ts

@@ -4,6 +4,7 @@ import { DecryptRequestBody } from '../../../src/typedRoutes/api/common/decrypt'
 import { EncryptRequestBody } from '../../../src/typedRoutes/api/common/encrypt';
 import { LoginRequest } from '../../../src/typedRoutes/api/common/login';
 import { VerifyAddressBody } from '../../../src/typedRoutes/api/common/verifyAddress';
+import { VerifyAddressV2Body, VerifyAddressV2Params } from '../../../src/typedRoutes/api/v2/verifyAddress';
 import { SimpleCreateRequestBody } from '../../../src/typedRoutes/api/v1/simpleCreate';
 
 export function assertDecode<T>(codec: t.Type<T, unknown>, input: unknown): T {
@@ -92,6 +93,34 @@ describe('io-ts decode tests', function () {
       address: 'some-address',
     });
   });
+  it('express.verifycoinaddress', function () {
+    // invalid coin param type
+    assert.throws(() =>
+      assertDecode(t.type(VerifyAddressV2Params), {
+        coin: 123,
+      })
+    );
+    // valid coin param
+    assertDecode(t.type(VerifyAddressV2Params), {
+      coin: 'btc',
+    });
+
+    // invalid address type in body
+    assert.throws(() =>
+      assertDecode(t.type(VerifyAddressV2Body), {
+        address: 123,
+      })
+    );
+    // valid body without optional flag
+    assertDecode(t.type(VerifyAddressV2Body), {
+      address: 'some-address',
+    });
+    // valid body with optional flag
+    assertDecode(t.type(VerifyAddressV2Body), {
+      address: 'some-address',
+      supportOldScriptHashVersion: true,
+    });
+  });
   it('express.v1.wallet.simplecreate', function () {
     // passphrase is required
     assert.throws(() => assertDecode(t.type(SimpleCreateRequestBody), {}));