patriksimek / vm2

Advanced vm/sandbox for Node.js
MIT License
3.86k stars 294 forks source link

NodeVM `require` config key cannot be false #456

Closed takamonowhite closed 2 years ago

takamonowhite commented 2 years ago

For my projects I am generally more comfortable to explicitly set default config values. But seems like I cannot pass false to require config key when creating new NodeVM.

image

XmiliaH commented 2 years ago

Thanks for the feedback. Changed the allowed values to boolean | VMRequire | undefined.