http: add complete property

[ronag]

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 #28621 and #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

[mscdex]

Missing documentation?

[ronag]

@mscdex I'd like to ask for some feedback on whether this is a valid suggestion before I spend time on docs and tests.

[BridgeAR]

@nodejs/http PTAL

[ronag]

@benjamingr since you approved the closed PR do you think this PR is something worth spending further work on?

[benjamingr]

I am really not sure but I am game going on a zoom or chat to understand all the changes you want to make to HTTP at once so I can understand the motivation and make a better decision.

[ronag]

@benjamingr sounds good to me, how do we go about setting that up?

[benjamingr]

Ping relevant parties and maybe someone from the TSC with access to the zoom? Maybe Matteo (though he probably won't have time in the coming week - probably later)

[ronag]

replaced by #28968