pinojs / pino-http

🌲 high-speed HTTP logger for Node.js
MIT License
539 stars 117 forks source link

Fix tests on Node 20 #301

Closed jsumners closed 1 year ago

jsumners commented 1 year ago

https://github.com/nodejs/node/pull/45597/files#diff-2464899ab38c61dd42654fdb9658c75989cb6883b9c9d76613872a12d3fe2780R1036 introduces a requirement that all incoming requests must have a Host header included. This is a breaking change in Node v20 that affects our test suite, and blocks #300. The change in this PR fixes it.