1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

domain: do not add domain to promise from other context

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/39135
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
This commit is contained in:
Stephen Belanger 2021-07-05 15:15:30 -07:00 committed by Node.js GitHub Bot
parent 37d5739e0c
commit f88da657f1
1 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
lib/domain.js
View File

@ -36,6 +36,7 @@ const {
Error, Error,
FunctionPrototypeCall, FunctionPrototypeCall,
ObjectDefineProperty, ObjectDefineProperty,
Promise,
ReflectApply, ReflectApply,
SafeMap, SafeMap,
Symbol, Symbol,
@ -73,12 +74,14 @@ const asyncHook = createHook({
if (process.domain !== null && process.domain !== undefined) { if (process.domain !== null && process.domain !== undefined) {
// If this operation is created while in a domain, let's mark it // If this operation is created while in a domain, let's mark it
pairing.set(asyncId, process.domain[kWeak]); pairing.set(asyncId, process.domain[kWeak]);
ObjectDefineProperty(resource, 'domain', { if (type !== 'PROMISE' || resource instanceof Promise) {
configurable: true, ObjectDefineProperty(resource, 'domain', {
enumerable: false, configurable: true,
value: process.domain, enumerable: false,
writable: true value: process.domain,
}); writable: true
});
}
} }
}, },
before(asyncId) { before(asyncId) {