Domains created inside uncaughtException event handler don't allow catching errors thrown inside the domain.run() #22400

Activity

1. m0uneer

   changed the title [-]Domains created inside `uncaughtException` event don't allow catching errors thrown inside the `domain.run()`[/-] [+]Domains created inside `uncaughtException` event handler don't allow catching errors thrown inside the `domain.run()`[/+] on Aug 23, 2018
2. addaleax

   added

   domainIssues and PRs related to the domain subsystem.Issues and PRs related to the domain subsystem.

   on Aug 24, 2018
3. addaleax commented on Aug 24, 2018

   

   addaleax

   on Aug 24, 2018

   Member

   More actions

   The reason for this is that when the fatal exception handler throws (and all uncaughtException/domain handlers are invoked from there), Node.js always exits immediately.

   It might be possible to fix this, but I’m not sure whether we want to do so, given that domain has been deprecated for a long time and this is a very edge-case-y scenario.

   I think patches would be welcome in either case.
4. m0uneer commented on Aug 25, 2018

   

   m0uneer

   on Aug 25, 2018

   Last edited by m0uneer

   Author

   More actions

   Thanks, @addaleax for commenting on this.

   So, I think that domains internally depend on a special uncaughtException event to catch errors. They don't catch sync errors that are swallowed and not cause this special uncaughtException event to be emitted.

   I can also closely explain my thoughts on how a domain works as follows:

   function domainRun({runCb, onErrorCb}) {
     process.on('specialUncaughtException', onErrorCb);
     runCb();
   }
   
   domainRun({
     runCb: notExistingFn,
     onErrorCb: () => console.log('Caught error within the domain!'),
   });

   So and to wrap up, the cases (I personally met) that highlight the problem are:

   • Domains created inside uncaughtException event handler that throws a sync error (Explained in the issue description)
   • Domains created inside a try/catch that throws a sync error as the following:

   const domain = require('domain');
   
   function foo() {
     const createdDomain = domain.create();
     createdDomain.on('error', () => console.log('Caught error within the `foo()` domain!'));
     createdDomain.run(notExistingFn);
   }
   
   try {
     foo();
   } catch (err) {
     console.log('Caught error within the `foo()` try/catch!');
   }

   Workarounds

   1. Convert the logic to be async by using process.nextTick() or a timer method every time we use domain.run(). This guarantees that the error is always saved from being swallowed. Example:

   createdDomain.run(() => process.nextTick(notExistingFn));

   2. Using try/catch in conjunction with the domains

   function onError(err) { 
     //... 
   }
   
   createdDomain.on('error', onError).run(() => {
     try {
       notExistingFn();
     } catch(err) {
       onError(err);
     }
   });
5. avivkeller commented on Sep 30, 2024

   

   avivkeller

   on Sep 30, 2024

   Member

   More actions

   Hey, I've closed this issue, as the domain builtin has been deprecated. Feel free to re-open.
6. avivkeller

   closed this as not plannedon Sep 30, 2024