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.

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

test: ability to use node debugger #56277

Merged
merged 4 commits into from
Dec 7, 2023
Merged

test: ability to use node debugger #56277

merged 4 commits into from
Dec 7, 2023

Conversation

tomfa
Copy link
Contributor

@tomfa tomfa commented Oct 1, 2023

Fixes ability to run NODE_OPTIONS='--inspect' next dev as described in docs, by removing inspect option from NODE_OPTIONS arg passed to worker process.

This bug seem to have been introduced in 7d93808 as a part of a some refactoring. See how getNodeOptionsWithoutInspect is no longer used.

Fixes #55862

@tomfa tomfa changed the title Test NODE_OPTION inspect option attaches debugger Fix ability to use node debugger Oct 1, 2023
@escherstair
Copy link

Hi, is there a roadmap for merging this PR?
Issue #55862 broke this PR

Thanks

@feedthejim
Copy link
Contributor

@escherstair I think I fixed it in #57159

@escherstair
Copy link

@escherstair I think I fixed it in #57159

And so this PR is not necessary anymore?

@escherstair
Copy link

Hi @feedthejim
it seems that the original issue #55862 is present in 14.0.0 #55862 (comment)

@kylemh
Copy link
Contributor

kylemh commented Dec 6, 2023

@feedthejim still not working in v14, right? so probably not fixed from #57159

@huozhi huozhi changed the title Fix ability to use node debugger test: ability to use node debugger Dec 7, 2023
@huozhi
Copy link
Member

huozhi commented Dec 7, 2023

Looks like it's already fixed on canary, I merged the canary branch and updated the PR title. Thanks for adding the test!

@huozhi huozhi enabled auto-merge (squash) December 7, 2023 19:19
@huozhi huozhi merged commit e1fe0c9 into vercel:canary Dec 7, 2023
65 of 69 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Node inspector stopped working on v13.5.x
6 participants