- 
          
 - 
                Notifications
    
You must be signed in to change notification settings  - Fork 5
 
Open
Description
I tested this module - it appears to be significantly more performant than using the equivalent functionality with the old domain module. It also seems to capture all the random async errors I threw at it (take that with a grain of salt, but so far so good).
So congrats to node core for implementing async-hooks, and thank you for creating this user-land library which re-creates the domain functionality, which is very important for my use cases, and I hope serves a strong purpose to other users going forward.
Not only is it more performant, but it should fix problems in both core and userland, so that users don't have to do this in userland:
if(process.domain){
   fn = process.domain.bind(fn);
}(I hope!)
Nice work!
Metadata
Metadata
Assignees
Labels
No labels