domain: do not add domain to promise from other context

nodejs · Jul 5, 2021 · ad65c19 · ad65c19
1 parent ccf6eef
commit ad65c19
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/lib/domain.js b/lib/domain.js
@@ -73,12 +73,14 @@ const asyncHook = createHook({
     if (process.domain !== null && process.domain !== undefined) {
       // If this operation is created while in a domain, let's mark it
       pairing.set(asyncId, process.domain[kWeak]);
-      ObjectDefineProperty(resource, 'domain', {
-        configurable: true,
-        enumerable: false,
-        value: process.domain,
-        writable: true
-      });
+      if (type !== 'PROMISE' || resource instanceof Promise) {
+        ObjectDefineProperty(resource, 'domain', {
+          configurable: true,
+          enumerable: false,
+          value: process.domain,
+          writable: true
+        });
+      }
     }
   },
   before(asyncId) {