Skip to content

Add a configurable node_modules allow-list#47

Merged
gsquared94 merged 1 commit intoGoogleContainerTools:duct-tapefrom
briandealwis:i45
Jul 31, 2020
Merged

Add a configurable node_modules allow-list#47
gsquared94 merged 1 commit intoGoogleContainerTools:duct-tapefrom
briandealwis:i45

Conversation

@briandealwis
Copy link
Copy Markdown
Member

Fixes #45

Adds an allow-list for node_modules that directly load user code and so are suitable scripts to use with --inspect. Additional
modules can be included by adding them to the WRAPPER_ALLOWED environment variable, separated by spaces.

Adds an allow-list for node_modules that directly load user code
and so are suitable scripts to use with --inspect.  Additional
modules can be included by adding them to the WRAPPER_ALLOWED
environment variable, separated by spaces.
@briandealwis briandealwis requested a review from gsquared94 July 30, 2020 01:40
@briandealwis
Copy link
Copy Markdown
Member Author

@vincentjocodes with this PR, I've marked the next.js launcher and so it should work with skaffold debug.

@gsquared94 gsquared94 merged commit b15f809 into GoogleContainerTools:duct-tape Jul 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

node wrapper should have a list of acceptable entrypoints from node_modules

2 participants