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

inspector: split the HostPort being used and the one parsed from CLI #24772

Closed
wants to merge 1 commit into from

Commits on Dec 6, 2018

  1. inspector: split the HostPort being used and the one parsed from CLI

    Instead of using a shared pointer of the entire debug option set,
    pass the parsed debug option to inspector classes by value because
    they are set once the CLI argument parsing is done. Add another shared
    pointer to HostPort being used by the inspector server, which is copied
    from the one in the debug options initially. The port of the shared
    HostPort is 9229 by default and can be specified as 0 initially but
    will be set to the actual port of the server once it starts listening.
    
    This makes the shared state clearer and makes it possible to use
    `require('internal/options')` in JS land to query the CLI options
    instead of using `process._breakFirstLine` and other underscored
    properties of `process` since we are now certain that these
    values should not be altered once the parsing is done and can be
    passed around in copies without locks.
    joyeecheung committed Dec 6, 2018
    Configuration menu
    Copy the full SHA
    c631a1a View commit details
    Browse the repository at this point in the history