fix(express): fix express overriding 2xx status

changed the redirectRequest method so it returns the original status from WP not only the body of
the response

WP-2961

TICKET: WP-2961

Author: alebusse

modules/express/src/clientRoutes.ts

@@ -1142,7 +1142,11 @@ function redirectRequest(bitgo: BitGo, method: string, url: string, req: express
     if (req.params.enterpriseId) {
       request.set('enterprise-id', req.params.enterpriseId);
     }
-    return request.result();
+
+    return request.result().then((result) => {
+      const status = request.res?.statusCode || 200;
+      return { status, body: result };
+    });
   }
 
   // something has presumably gone wrong
@@ -1217,8 +1221,7 @@ function prepareBitGo(config: Config) {
     next();
   };
 }
-
-type RequestHandlerResponse = string | unknown | undefined;
+type RequestHandlerResponse = string | unknown | undefined | { status: number; body: unknown };
 interface RequestHandler extends express.RequestHandler<ParamsDictionary, any, RequestHandlerResponse> {
   (req: express.Request, res: express.Response, next: express.NextFunction):
     | RequestHandlerResponse
@@ -1265,7 +1268,12 @@ function promiseWrapper(promiseRequestHandler: RequestHandler) {
     debug(`handle: ${req.method} ${req.originalUrl}`);
     try {
       const result = await promiseRequestHandler(req, res, next);
-      res.status(200).send(result);
+      if (typeof result === 'object' && result !== null && 'body' in result && 'status' in result) {
+        const { status, body } = result as { status: number; body: unknown };
+        res.status(status).send(body);
+      } else {
+        res.status(200).send(result);
+      }
     } catch (e) {
       handleRequestHandlerError(res, e);
     }