Bartek Iwańczuk
Bartek Iwańczuk
Thanks for flagging this @panva, this should be fixed by https://github.com/denoland/deno/pull/22312.
Opened https://github.com/denoland/deno_core/pull/354 that will make it easier to customize.
@jsejcksn do you still experience this problem with Deno v1.35? I believe this has been fixed.
@dsherret could you take a look here?
The gist of the problem is that `node:vm` creates a new V8 context, but it doesn't create related structures that would be set up for every single context - `JsRealm`,...
@mmastrac can you take a look?
I hit this problem as well working on Node compatibility - accessing `process.argv` (which uses `Deno.mainModule` internally) requires to open full FS read sandbox. I think it's hindering DX in...
Discussed with other team members, we are going to remove requirement for allow read permission on `Deno.mainModule`.
@mmastrac can this issue be closed given the sanitizer rewrite?
Seems reasonable as a non-recommended rule. Contributions are very welcome.