-
-
Notifications
You must be signed in to change notification settings - Fork 280
Closed
Description
Hi! I noticed bcrypt.js uses process.nextTick in node to delay the calculation.
Unfortunately, repeated calls to nextTick will still starve the event loop: https://nodejs.org/en/docs/guides/event-loop-timers-and-nexttick/#process-nexttick
nextTick is unfortunately badly named, it should've been called 'endOfThisTick`.
Can we switch to setImmediate instead?
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels