Commit b91d22c
repl: fix referrer for dynamic import
The ESM loader does not accept a directory as the referrer, it requires
a path within the directory. Add `/repl` to ensure relative dynamic
imports can succeed.
Fixes: #19570
PR-URL: #30609
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ben Coe <bencoe@gmail.com>1 parent 6257408 commit b91d22cCopy full SHA for b91d22c
File tree
Expand file treeCollapse file tree
2 files changed
+29
-3
lines changedOpen diff view settings
Filter options
- lib
- test/parallel
Expand file treeCollapse file tree
2 files changed
+29
-3
lines changedOpen diff view settings
Collapse file
+5-3Lines changed: 5 additions & 3 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
332 | 332 | |
333 | 333 | |
334 | 334 | |
335 | | - |
| 335 | + |
336 | 336 | |
337 | 337 | |
338 | | - |
| 338 | + |
| 339 | + |
| 340 | + |
339 | 341 | |
340 | 342 | |
341 | 343 | |
| ||
350 | 352 | |
351 | 353 | |
352 | 354 | |
353 | | - |
| 355 | + |
354 | 356 | |
355 | 357 | |
356 | 358 | |
|
Collapse file
test/parallel/test-repl-import-referrer.js
Copy file name to clipboard+24Lines changed: 24 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
| 18 | + |
| 19 | + |
| 20 | + |
| 21 | + |
| 22 | + |
| 23 | + |
| 24 | + |
0 commit comments