Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

lavamoat does not recognize core modules #652

Closed
boneskull opened this issue Aug 15, 2023 · 0 comments · Fixed by #653, #665 or #720
Closed

lavamoat does not recognize core modules #652

boneskull opened this issue Aug 15, 2023 · 0 comments · Fixed by #653, #665 or #720
Labels
bug Something isn't working

Comments

@boneskull
Copy link
Contributor

Since v14.18.0 and v16.0.0, Node.js supports the require of builtins using the node: prefix; e.g., node:fs (docs).

It doesn't seem that lavamoat understands these:

lavamoat - unable to resolve "node:events" from "/some/script.js"
@boneskull boneskull added the bug Something isn't working label Aug 15, 2023
boneskull added a commit that referenced this issue Aug 15, 2023
@boneskull
fix(lavamoat): allow prefixed builtins
db04a4e 
Closes #652
@boneskull boneskull mentioned this issue Aug 15, 2023
Merged
boneskull added a commit that referenced this issue Aug 15, 2023
@boneskull
fix(lavamoat): allow prefixed builtins
8dc7910 
Closes #652
boneskull added a commit that referenced this issue Aug 15, 2023
@boneskull
fix(lavamoat): allow prefixed builtins
753953d 
Closes #652
boneskull added a commit that referenced this issue Aug 16, 2023
@boneskull
fix(lavamoat): allow prefixed builtins
55613cd 
Closes #652
boneskull added a commit that referenced this issue Aug 16, 2023
@boneskull
fix(lavamoat): allow prefixed builtins
9c35742 
Closes #652
boneskull added a commit that referenced this issue Aug 16, 2023
@boneskull
fix(lavamoat): allow prefixed builtins
1a962d4 
Closes #652
boneskull added a commit that referenced this issue Aug 17, 2023
@boneskull
fix(lavamoat): allow prefixed builtins
2e369ab 
Closes #652
boneskull added a commit that referenced this issue Aug 21, 2023
@boneskull
fix(lavamoat): allow prefixed builtins
6ef1593 
Closes #652
@github-actions github-actions bot mentioned this issue Aug 21, 2023
Merged
@github-actions github-actions bot mentioned this issue Sep 14, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
1 participant
@boneskull