[argus] include response headers in http log

Joystream · Oct 10, 2023 · bce9585 · bce9585
1 parent df3ed68
commit bce9585
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/distributor-node/CHANGELOG.md b/distributor-node/CHANGELOG.md
@@ -4,6 +4,7 @@
 
 - Add 1s delay between fetching pages for QN state sync. This will allow QN to have some breathing space and process other requests that may have been stuck because of heavy processing.
 - Add `Access-Control-Expose-Headers` response header to allow web clients checking cache status of downloaded file.
+- Include response headers in `http` logs
 
 ### 1.3.1
 

diff --git a/distributor-node/src/services/httpApi/HttpApiBase.ts b/distributor-node/src/services/httpApi/HttpApiBase.ts
@@ -61,7 +61,10 @@ export abstract class HttpApiBase {
       winstonInstance: this.logger,
       level: 'http',
       dynamicMeta: (req, res) => {
-        return { prematurelyClosed: res.locals.prematurelyClosed ?? false }
+        return {
+          prematurelyClosed: res.locals.prematurelyClosed ?? false,
+          res: { headers: res.getHeaders(), statusCode: res.statusCode },
+        }
       },
     }
   }