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.

Sign up for GitHub

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

Jump to bottom

util: prevent leaking inspect internals #24971

Closed
wants to merge 2 commits into from
Closed

util: prevent leaking inspect internals #24971

wants to merge 2 commits into from

Commits on Feb 16, 2019

  1. lib: don't use util.inspect() internals 

    This makes sure the internal `stylize` function is not used to render
anything and instead just uses the regular inspect function in case
of reaching the maximum depth level.
    @BridgeAR
    BridgeAR committed Feb 16, 2019
    Copy the full SHA
    2867425 View commit details
    Browse the repository at this point in the history

  2. util: prevent leaking internal properties 

    This prevents leaking of the internal `inspect()` properties when
using a custom inspect function.

It also aligns the indentation to the way it was in v8.0.0 since
that changed unintentionally. All strings returned by the custom
inspect function will now be indented appropriately to the current
depth.
    @BridgeAR
    BridgeAR committed Feb 16, 2019
    Copy the full SHA
    dd1a712 View commit details
    Browse the repository at this point in the history