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

net: do not manipulate potential user code #26751

Closed

BridgeAR wants to merge 1 commit into nodejs:master from BridgeAR:do-not-manipulate-errors

Commits on Mar 21, 2019

net: do not manipulate potential user code

The error provided in this function could come from user code. Thus
the error should not be manipulated in any way. The added properties
do not seem to provide any actual value either as can not be part
of the error. The `hostname` is already set on the error and adding
the `host` property with the identical value does not seem right in
this case.

BridgeAR committed Mar 21, 2019