Skip to content

Add 'remoteHostHeader' option for node attach#1664

Merged
connor4312 merged 1 commit intomicrosoft:mainfrom
pilaoda:main
Apr 18, 2023
Merged

Add 'remoteHostHeader' option for node attach#1664
connor4312 merged 1 commit intomicrosoft:mainfrom
pilaoda:main

Conversation

@pilaoda
Copy link
Copy Markdown
Contributor

@pilaoda pilaoda commented Apr 18, 2023

Fixes #778
Related: https://github.com/microsoft/vscode-js-debug/pull/667/files#r468093566

The use case is very clear here:
#778 (comment)

I notice that there is already a PR for this but with too old code base to merge.
I think the reason to take this feature is good enough since it is requested more than three times.

Clear use case again:
The remote gateway rule that reject host:localhost header is under control of my serverless cloud provider and I have no way to modify it. I already know that nodejs will only accept ip address or localhost, but I can change the Host header back to localhost behind the cloud gateway with one more websocket proxy that in my control.
I think this is the only way to remote debug such strict server that not support customize firewall.

@connor4312 connor4312 enabled auto-merge April 18, 2023 23:00
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.

Cannot attach to debugger

3 participants