Skip to content

http: add complete property

Rodrigo Muino Tomonari requested to merge github/fork/nxtedition/feat-http-complete into master

This is a suggestion to add a complete property to http request and response to indicate that no further work can/should be performed on the instance.

I believe this is a better alternative to the current finished property which I believe is currently incorrectly used in a lot of scenarios.

Depends on https://github.com/nodejs/node/pull/28621 and https://github.com/nodejs/node/pull/28627 which should probably first be merged.

Checklist
  • 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