GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — vm: reject in importModuleDynamically without --experimental-vm-modules · nodejs/node@9853fd9

File tree

Expand file treeCollapse file tree
Open diff view settings
Filter options
  • doc/api
    • errors.md
    • vm.md
  • lib/internal
    • errors.js
    • modules/esm
      • utils.js
    • vm.js
  • src
    • env_properties.h
  • test/parallel
    • test-vm-dynamic-import-callback-missing-flag.js
Expand file treeCollapse file tree
Open diff view settings
Collapse file

‎doc/api/errors.md‎

Copy file name to clipboardExpand all lines: doc/api/errors.md
+6Lines changed: 6 additions & 0 deletions
  • Display the source diff
  • Display the rich diff
Original file line numberDiff line numberDiff line change
@@ -3004,6 +3004,12 @@ An attempt was made to use something that was already closed.
30043004
While using the Performance Timing API (`perf_hooks`), no valid performance
30053005
entry types are found.
30063006

3007+
<a id="ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING_FLAG"></a>
3008+
3009+
### `ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING_FLAG`
3010+
3011+
A dynamic import callback was invoked without `--experimental-vm-modules`.
3012+
30073013
<a id="ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING"></a>
30083014

30093015
### `ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING`
Collapse file

‎doc/api/vm.md‎

Copy file name to clipboardExpand all lines: doc/api/vm.md
+19-5Lines changed: 19 additions & 5 deletions
  • Display the source diff
  • Display the rich diff
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,9 @@ changes:
9898
when `import()` is called. If this option is not specified, calls to
9999
`import()` will reject with [`ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING`][].
100100
This option is part of the experimental modules API. We do not recommend
101-
using it in a production environment.
101+
using it in a production environment. If `--experimental-vm-modules` isn't
102+
set, this callback will be ignored and calls to `import()` will reject with
103+
[`ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING_FLAG`][].
102104
* `specifier` {string} specifier passed to `import()`
103105
* `script` {vm.Script}
104106
* `importAttributes` {Object} The `"with"` value passed to the
@@ -767,6 +769,9 @@ changes:
767769
* `importModuleDynamically` {Function} Called during evaluation of this module
768770
when `import()` is called. If this option is not specified, calls to
769771
`import()` will reject with [`ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING`][].
772+
If `--experimental-vm-modules` isn't set, this callback will be ignored
773+
and calls to `import()` will reject with
774+
[`ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING_FLAG`][].
770775
* `specifier` {string} specifier passed to `import()`
771776
* `module` {vm.Module}
772777
* `importAttributes` {Object} The `"with"` value passed to the
@@ -1025,7 +1030,9 @@ changes:
10251030
when `import()` is called. If this option is not specified, calls to
10261031
`import()` will reject with [`ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING`][].
10271032
This option is part of the experimental modules API, and should not be
1028-
considered stable.
1033+
considered stable. If `--experimental-vm-modules` isn't
1034+
set, this callback will be ignored and calls to `import()` will reject with
1035+
[`ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING_FLAG`][].
10291036
* `specifier` {string} specifier passed to `import()`
10301037
* `function` {Function}
10311038
* `importAttributes` {Object} The `"with"` value passed to the
@@ -1249,7 +1256,9 @@ changes:
12491256
when `import()` is called. If this option is not specified, calls to
12501257
`import()` will reject with [`ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING`][].
12511258
This option is part of the experimental modules API. We do not recommend
1252-
using it in a production environment.
1259+
using it in a production environment. If `--experimental-vm-modules` isn't
1260+
set, this callback will be ignored and calls to `import()` will reject with
1261+
[`ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING_FLAG`][].
12531262
* `specifier` {string} specifier passed to `import()`
12541263
* `script` {vm.Script}
12551264
* `importAttributes` {Object} The `"with"` value passed to the
@@ -1348,7 +1357,9 @@ changes:
13481357
when `import()` is called. If this option is not specified, calls to
13491358
`import()` will reject with [`ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING`][].
13501359
This option is part of the experimental modules API. We do not recommend
1351-
using it in a production environment.
1360+
using it in a production environment. If `--experimental-vm-modules` isn't
1361+
set, this callback will be ignored and calls to `import()` will reject with
1362+
[`ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING_FLAG`][].
13521363
* `specifier` {string} specifier passed to `import()`
13531364
* `script` {vm.Script}
13541365
* `importAttributes` {Object} The `"with"` value passed to the
@@ -1428,7 +1439,9 @@ changes:
14281439
when `import()` is called. If this option is not specified, calls to
14291440
`import()` will reject with [`ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING`][].
14301441
This option is part of the experimental modules API. We do not recommend
1431-
using it in a production environment.
1442+
using it in a production environment. If `--experimental-vm-modules` isn't
1443+
set, this callback will be ignored and calls to `import()` will reject with
1444+
[`ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING_FLAG`][].
14321445
* `specifier` {string} specifier passed to `import()`
14331446
* `script` {vm.Script}
14341447
* `importAttributes` {Object} The `"with"` value passed to the
@@ -1592,6 +1605,7 @@ are not controllable through the timeout either.
15921605
[Source Text Module Record]: https://tc39.es/ecma262/#sec-source-text-module-records
15931606
[Synthetic Module Record]: https://heycam.github.io/webidl/#synthetic-module-records
15941607
[V8 Embedder's Guide]: https://v8.dev/docs/embed#contexts
1608+
[`ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING_FLAG`]: errors.md#err_vm_dynamic_import_callback_missing_flag
15951609
[`ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING`]: errors.md#err_vm_dynamic_import_callback_missing
15961610
[`ERR_VM_MODULE_STATUS`]: errors.md#err_vm_module_status
15971611
[`Error`]: errors.md#class-error
Collapse file

‎lib/internal/errors.js‎

Copy file name to clipboardExpand all lines: lib/internal/errors.js
+3Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1821,6 +1821,9 @@ E('ERR_VALID_PERFORMANCE_ENTRY_TYPE',
18211821
'At least one valid performance entry type is required', Error);
18221822
E('ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING',
18231823
'A dynamic import callback was not specified.', TypeError);
1824+
E('ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING_FLAG',
1825+
'A dynamic import callback was invoked without --experimental-vm-modules',
1826+
TypeError);
18241827
E('ERR_VM_MODULE_ALREADY_LINKED', 'Module has already been linked', Error);
18251828
E('ERR_VM_MODULE_CANNOT_CREATE_CACHED_DATA',
18261829
'Cached data cannot be created for a module which has been evaluated', Error);
Collapse file

‎lib/internal/modules/esm/utils.js‎

Copy file name to clipboardExpand all lines: lib/internal/modules/esm/utils.js
+7-1Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,11 @@ const {
1414
} = internalBinding('util');
1515
const {
1616
default_host_defined_options,
17+
vm_dynamic_import_missing_flag,
1718
} = internalBinding('symbols');
1819

1920
const {
21+
ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING_FLAG,
2022
ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING,
2123
ERR_INVALID_ARG_VALUE,
2224
} = require('internal/errors').codes;
@@ -132,7 +134,8 @@ const moduleRegistries = new SafeWeakMap();
132134
*/
133135
function registerModule(referrer, registry) {
134136
const idSymbol = referrer[host_defined_option_symbol];
135-
if (idSymbol === default_host_defined_options) {
137+
if (idSymbol === default_host_defined_options ||
138+
idSymbol === vm_dynamic_import_missing_flag) {
136139
// The referrer is compiled without custom callbacks, so there is
137140
// no registry to hold on to. We'll throw
138141
// ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING when a callback is
@@ -173,6 +176,9 @@ async function importModuleDynamicallyCallback(symbol, specifier, attributes) {
173176
return importModuleDynamically(specifier, callbackReferrer, attributes);
174177
}
175178
}
179+
if (symbol === vm_dynamic_import_missing_flag) {
180+
throw new ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING_FLAG();
181+
}
176182
throw new ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING();
177183
}
178184

Collapse file

‎lib/internal/vm.js‎

Copy file name to clipboardExpand all lines: lib/internal/vm.js
+16Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,19 @@ const {
1515
} = ContextifyScript.prototype;
1616
const {
1717
default_host_defined_options,
18+
vm_dynamic_import_missing_flag,
1819
} = internalBinding('symbols');
1920
const {
2021
validateFunction,
2122
validateObject,
2223
kValidateObjectAllowArray,
2324
} = require('internal/validators');
2425

26+
const {
27+
getOptionValue,
28+
} = require('internal/options');
29+
30+
2531
function isContext(object) {
2632
validateObject(object, 'object', kValidateObjectAllowArray);
2733

@@ -41,6 +47,16 @@ function getHostDefinedOptionId(importModuleDynamically, filename) {
4147
// compilation cache can be hit.
4248
return default_host_defined_options;
4349
}
50+
// We should've thrown here immediately when we introduced
51+
// --experimental-vm-modules and importModuleDynamically, but since
52+
// users are already using this callback to throw a similar error,
53+
// we also defer the error to the time when an actual import() is called
54+
// to avoid breaking them. To ensure that the isolate compilation
55+
// cache can still be hit, use a constant sentinel symbol here.
56+
if (!getOptionValue('--experimental-vm-modules')) {
57+
return vm_dynamic_import_missing_flag;
58+
}
59+
4460
return Symbol(filename);
4561
}
4662

Collapse file

‎src/env_properties.h‎

Copy file name to clipboardExpand all lines: src/env_properties.h
+2-1Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,8 @@
4444
V(owner_symbol, "owner_symbol") \
4545
V(onpskexchange_symbol, "onpskexchange") \
4646
V(resource_symbol, "resource_symbol") \
47-
V(trigger_async_id_symbol, "trigger_async_id_symbol")
47+
V(trigger_async_id_symbol, "trigger_async_id_symbol") \
48+
V(vm_dynamic_import_missing_flag, "vm_dynamic_import_missing_flag")
4849

4950
// Strings are per-isolate primitives but Environment proxies them
5051
// for the sake of convenience. Strings should be ASCII-only.
Collapse file

‎test/parallel/test-vm-dynamic-import-callback-missing-flag.js‎

Copy file name to clipboard
+28Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
'use strict';
2+
3+
const common = require('../common');
4+
const { Script, compileFunction } = require('vm');
5+
const assert = require('assert');
6+
7+
assert(
8+
!process.execArgv.includes('--experimental-vm-modules'),
9+
'This test must be run without --experimental-vm-modules');
10+
11+
assert.rejects(async () => {
12+
const script = new Script('import("fs")', {
13+
importModuleDynamically: common.mustNotCall(),
14+
});
15+
const imported = script.runInThisContext();
16+
await imported;
17+
}, {
18+
code: 'ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING_FLAG'
19+
}).then(common.mustCall());
20+
21+
assert.rejects(async () => {
22+
const imported = compileFunction('return import("fs")', [], {
23+
importModuleDynamically: common.mustNotCall(),
24+
})();
25+
await imported;
26+
}, {
27+
code: 'ERR_VM_DYNAMIC_IMPORT_CALLBACK_MISSING_FLAG'
28+
}).then(common.mustCall());

0 commit comments

Comments
0 (0)
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.