Matteo Collina
Matteo Collina
Go for it! I suspect it's not supported but please verify.
Not a blocker for V8.
It's very hard to help you given that we cannot run the code. Could you create an example that is just calling an http endpoint to simulate slack?
Unfortunately not yet, it's in the queue after Fastify v4. things to verify, in this order: 1. never use console.log in workers. It does not work the way you think....
The error is catched in: https://github.com/pinojs/thread-stream/blob/f19ac8dbd602837d2851e17fbc7dfc5bbc51083f/lib/worker.js#L59-L70 and https://github.com/pinojs/thread-stream/blob/f19ac8dbd602837d2851e17fbc7dfc5bbc51083f/lib/worker.js#L135-L149 Then, it's forwarded to the transport `'error'` event in https://github.com/pinojs/thread-stream/blob/f19ac8dbd602837d2851e17fbc7dfc5bbc51083f/index.js#L163-L164 I hope it's clearer.
It will be good to document this.
Why are you are double-casing a wrong parameter? cc @kibertoad
Do you know which version supported it? I'm mot convinced we ever did. Anyway, would you like to send a Pull Request to address this issue? Remember to add unit...
This is implementable without having any breaking change.
What do you mean by > but is not exportable