Skip to content

https: remove usage of public require('util') - #26772

Closed
dnlup wants to merge 1 commit into
nodejs:masterfrom
dnlup:util_https
Closed

https: remove usage of public require('util')#26772
dnlup wants to merge 1 commit into
nodejs:masterfrom
dnlup:util_https

Conversation

@dnlup

@dnlup dnlup commented Mar 19, 2019

Copy link
Copy Markdown
Contributor

Use require('internal/util/debuglog').debuglog
and Object.setPrototypeOf instead of require('util').debuglog
and require('util').inherits.

Refs: #26546

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. https Issues or PRs related to the https subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants