Bartek Iwańczuk

Results 361 comments of 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.

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`,...

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.