nodejs
diff --git a/Collapse file
‎src/node_contextify.cc‎
Copy file name to clipboardExpand all lines: src/node_contextify.cc
+6-3Lines changed: 6 additions & 3 deletions b/Collapse file
‎src/node_contextify.cc‎
Copy file name to clipboardExpand all lines: src/node_contextify.cc
+6-3Lines changed: 6 additions & 3 deletions
diff --git a/Collapse file
‎test/parallel/test-vm-global-configurable-properties.js‎
Copy file name to clipboard
+15Lines changed: 15 additions & 0 deletions b/Collapse file
‎test/parallel/test-vm-global-configurable-properties.js‎
Copy file name to clipboard
+15Lines changed: 15 additions & 0 deletions
@@ -609,11 +609,14 @@ void ContextifyContext::PropertyDefinerCallback(
   bool read_only =
       static_cast<int>(attributes) &
           static_cast<int>(PropertyAttribute::ReadOnly);
+  bool dont_delete = static_cast<int>(attributes) &
+                     static_cast<int>(PropertyAttribute::DontDelete);
 
-  // If the property is set on the global as read_only, don't change it on
-  // the global or sandbox.
-  if (is_declared && read_only)
+  // If the property is set on the global as neither writable nor
+  // configurable, don't change it on the global or sandbox.
+  if (is_declared && read_only && dont_delete) {
     return;
+  }
 
   Local<Object> sandbox = ctx->sandbox();
 
 
@@ -0,0 +1,15 @@
+'use strict';
+// https://github.com/nodejs/node/issues/47799
+
+require('../common');
+const assert = require('assert');
+const vm = require('vm');
+
+const ctx = vm.createContext();
+
+const window = vm.runInContext('this', ctx);
+
+Object.defineProperty(window, 'x', { value: '1', configurable: true });
+assert.strictEqual(window.x, '1');
+Object.defineProperty(window, 'x', { value: '2', configurable: true });
+assert.strictEqual(window.x, '2');