Skip to content

doc: Fix the `ServerResponse` methods explainations to make it clear and not puzzled

Rodrigo Muino Tomonari requested to merge HttpUpdate into master

Created by: ghost

Ref: https://github.com/nodejs/node/issues/14146.

In short: ServerResponse acutally inherits from OutgoingMessage, with a series of methods like those in Stream.Writable. So we cannot use implements(this has made poeple feel puzzled because there are still many methods we don't need or have), so inherits from Stream is enough, due to some core reasons and performance told by mcollina from the ref (See some latest discussions at Ref).

  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows [commit guidelines]

Merge request reports