[IND-458]: In case of errors in comlink log request (dydxprotocol#722)

indexer/services/comlink/src/controllers/api/v4/addresses-controller.ts

@@ -269,6 +269,7 @@ router.get(
         'AddressesController GET /:address',
         'Addresses error',
         error,
+        req,
         res,
       );
     }
@@ -308,6 +309,7 @@ router.get(
         'AddressesController GET /:address/subaccountNumber/:subaccountNumber',
         'Addresses subaccount error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/controllers/api/v4/asset-positions-controller.ts

@@ -174,6 +174,7 @@ router.get(
         'AssetPositionsController GET /',
         'Asset positions error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/controllers/api/v4/candles-controller.ts

@@ -101,6 +101,7 @@ router.get(
         'CandlesController GET /perpetualMarkets/:ticker',
         'Candles error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/controllers/api/v4/compliance-controller.ts

@@ -144,6 +144,7 @@ router.get(
         'ComplianceController GET /',
         'Compliance error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/controllers/api/v4/fills-controller.ts

@@ -161,6 +161,7 @@ router.get(
         'FillsController GET /',
         'Fills error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/controllers/api/v4/height-controller.ts

@@ -37,7 +37,7 @@ router.get(
   rejectRestrictedCountries,
   rateLimiterMiddleware(getReqRateLimiter),
   ExportResponseCodeStats({ controllerName }),
-  async (_req: express.Request, res: express.Response) => {
+  async (req: express.Request, res: express.Response) => {
     const start: number = Date.now();
     try {
       const controller: HeightController = new HeightController();
@@ -49,6 +49,7 @@ router.get(
         'HeightController GET /',
         'Height error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/controllers/api/v4/historical-funding-controller.ts

@@ -116,6 +116,7 @@ router.get(
         'HistoricalFundingController GET /',
         'HistoricalFunding error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/controllers/api/v4/historical-pnl-controller.ts

@@ -128,6 +128,7 @@ router.get(
         'HistoricalPnlController GET /',
         'Historical Pnl error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/controllers/api/v4/orderbook-controller.ts

@@ -79,6 +79,7 @@ router.get(
         'OrderbooksController GET /perpetualMarket/:ticker',
         'Orderbooks error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/controllers/api/v4/orders-controller.ts

@@ -262,6 +262,7 @@ router.get(
         'OrdersController GET /',
         'Orders error',
         error,
+        req,
         res,
       );
     } finally {
@@ -300,6 +301,7 @@ router.get(
         'OrdersController GET /:orderId',
         'Orders error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/controllers/api/v4/perpetual-markets-controller.ts

@@ -142,6 +142,7 @@ router.get(
         'PerpetualMarketController GET /',
         'PerpetualMarket error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/controllers/api/v4/perpetual-positions-controller.ts

@@ -198,6 +198,7 @@ router.get(
         'PerpetualPositionsController GET /',
         'Perpetual positions error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/controllers/api/v4/sparklines-controller.ts

@@ -90,6 +90,7 @@ router.get(
         'SparklinesController GET /',
         'Sparklines error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/controllers/api/v4/trades-controller.ts

@@ -116,6 +116,7 @@ router.get(
         'TradesController GET /perpetualMarket/:ticker',
         'Trades error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/controllers/api/v4/transfers-controller.ts

@@ -163,6 +163,7 @@ router.get(
         'TransfersController GET /',
         'Transfers error',
         error,
+        req,
         res,
       );
     } finally {

indexer/services/comlink/src/lib/helpers.ts

@@ -48,6 +48,7 @@ export function handleControllerError(
   at: string,
   message: string,
   error: Error,
+  req: express.Request,
   res: express.Response,
 ): express.Response {
   if (error instanceof NotFoundError) {
@@ -57,6 +58,7 @@ export function handleControllerError(
     at,
     message,
     error,
+    req,
     res,
   );
 }
@@ -65,6 +67,7 @@ function handleInternalServerError(
   at: string,
   message: string,
   error: Error,
+  req: express.Request,
   res: express.Response,
 ): express.Response {
   if (config.isDevelopment()) {
@@ -76,6 +79,8 @@ function handleInternalServerError(
     at,
     message,
     error,
+    params: JSON.stringify(req.params),
+    query: JSON.stringify(req.query),
   });
   return createInternalServerErrorResponse(res);
 }