doc: clarify http.request language#8990
doc: clarify http.request language#8990skeggse wants to merge 1 commit intonodejs:masterfrom skeggse:patch-1
Conversation
The http.request documentation doesn't clearly specify the significance of some of its options fields, `host` and `hostname`. Signed-off-by: Eli Skeggs <skeggse@gmail.com>
|
@cjihrig @chrisdickinson ... not convinced this one is necessary. Inclined to close but wanted to check with you first. |
|
Just to expand on this, the current language in the docs says that the |
|
@skeggse ... yeah, the description in general should be improved. Tell you what... can you close this here and open this as a new PR against http://github.com/nodejs/node master? Active development is moving there and I'd like to get others to weigh in on the change. |
|
I'm in favor of improving the documentation here. I also agree that we should move this over to nodejs/node. Going to close here. |
I can't even count the number of times I've looked at the http docs to see whether
hostorhostnamelets you include a port forhttp.request.