v8eval icon indicating copy to clipboard operation
v8eval copied to clipboard

Proposal: Add option to set bind IP for debugger

Open thieman opened this issue 9 years ago • 4 comments

My reading of this line is that currently the debugger only ever binds itself to 127.0.0.1. This is a sensible default, but it makes attaching a remote debugger to a running v8eval instance impossible without layering on additional tooling.

Would you be open to making this configurable, probably as a constructor to EnableDebugger? If so I might look into providing a patch.

thieman avatar Jan 06 '17 18:01 thieman

As an alternative for anyone else having this problem, you can run socat on the same machine as v8eval to forward a remote request through the local interface.

thieman avatar Jan 06 '17 18:01 thieman

Certainly there is no need to reject remote debugger connections. I think it's another simple solution to replace 127.0.0.1 with 0.0.0.0. What do you think?

YoshiyukiMineo avatar Jan 11 '17 14:01 YoshiyukiMineo

There is nothing in v8eval which makes the debugger run by default, is there? As long as that's the case, I think we could get away with defaulting to 0.0.0.0, but we would want to make it very explicit in the docs that we bind on all addresses.

thieman avatar Jan 11 '17 14:01 thieman

v8eval never run the debug server until EnableDebugger is called. I will replace 127.0.0.1 with 0.0.0.0 and write it explicitly in the docs.

YoshiyukiMineo avatar Jan 13 '17 04:01 YoshiyukiMineo