-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
proposal: programmatically expose the V8 inspector URL #11496
Comments
Should be easy enough to do but let's see what @nodejs/v8-inspector thinks. |
Sounds like a good idea to me. I think it should probably not be on the V8 module though? |
We have plans of introducing bindings for inspector API (it is currently blocked on inspector becoming the default debug API, e.g. by installing the debug signal handler). I believe we could expose this url as a part of that API. API details are still TBD (e.g. I did not even open a pull request)... |
FWIW you can get the URI and other metadata like the UUID from the JSON returned from @ofrobots @cristiancavalli @eugeneo are working on providing a JS API to Inspector in general, perhaps access to the /cc @nodejs/diagnostics |
This seems like a good idea. Perhaps even one step further ... an API that simply returns the same data as the As for where it should live, perhaps when running with |
WIP JS bindings version I have (
https://github.com/eugeneo/node/tree/js-bindings) has a project.inspector
object so it will be easy to expose targets there. But I am not sure what
it will take to land such an API change.
…On Wed, Feb 22, 2017 at 8:48 AM James M Snell ***@***.***> wrote:
This seems like a good idea. Perhaps even one step further ... an API that
simply returns the same data as the /json/list endpoint.
As for where it should live, perhaps when running with --inspect, this
can be exposed via process.inspectInfo() API. When running without
--inspect, it would simply return undefined
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#11496 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AARkrYsCgBAIOA_y0MiGiuGqHrlwBzZxks5rfGbagaJpZM4MIT8_>
.
|
I think this URI should be also predictable and partially configurable from command line. As I mentioned in #11701 I'm working with remote debug and automated tools doesn't allow to extract this URI from node output. |
Is this in progress? Stalled? Blocked? Help wanted? I guess /cc @eugeneo but really anyone that knows? |
The requested feature is now provided in inspector.url(), so closing. |
PR-URL: #13228 Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This is a feature request to add a method to the
v8
module that exposes the V8 inspector URL if the process has been started with--inspect
. The motivation for this feature is to conveniently add links or buttons to a website that the developer can click to open the V8 inspector.This is a hypothetical API:
The text was updated successfully, but these errors were encountered: