pinojs / pino-http

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

fix: #308 include req.readableAborted flag to defaultSuccessfulRequestMessageProvider #309

Closed someden closed 8 months ago

someden commented 9 months ago

Closes #308

someden commented 9 months ago

@mcollina done

mcollina commented 8 months ago

A test is failing on node v14

someden commented 8 months ago

@mcollina, do you have some plans to drop node v14 support?

mcollina commented 8 months ago

Of course, might actually be time to drop v14 and v16.

mcollina commented 8 months ago

Could you remove them from CI? I will bump the major.