josdejong / workerpool

Offload tasks to a pool of workers on node.js and in the browser
Apache License 2.0
2.06k stars 147 forks source link

fix: properly timeout on worker termination #395

Closed Michsior14 closed 1 year ago

Michsior14 commented 1 year ago

Fixes the termination bug mentioned at #387.

Michsior14 commented 1 year ago

@josdejong What do you think about this? I've seen you couldn't tell the difference with or without this timeout adjustment.

josdejong commented 1 year ago

Thanks @Michsior14 this fix makes sense indeed!

josdejong commented 1 year ago

Fix published now in v6.4.2