serialize headers (renamed header to headers)

pinojs · Oct 11, 2018 · 017313f · 017313f
1 parent 6675674
commit 017313f
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/lib/res.js b/lib/res.js
@@ -35,7 +35,7 @@ Object.defineProperty(pinoResProto, rawSymbol, {
 function resSerializer (res) {
   const _res = Object.create(pinoResProto)
   _res.statusCode = res.statusCode
-  _res.header = res._header
+  _res.headers = res._headers
   _res.raw = res
   return _res
 }

diff --git a/test/res.test.js b/test/res.test.js
@@ -68,3 +68,22 @@ test('can wrap response serializers', function (t) {
     res.end()
   }
 })
+
+test('res.headers is serialized', function (t) {
+  t.plan(1)
+
+  var server = http.createServer(handler)
+  server.unref()
+  server.listen(0, () => {
+    http.get(server.address(), () => {})
+  })
+
+  t.tearDown(() => server.close())
+
+  function handler (req, res) {
+    res.setHeader('x-custom', 'y')
+    var serialized = serializers.resSerializer(res)
+    t.is(serialized.headers['x-custom'], 'y')
+    res.end()
+  }
+})