undocumented "lookup" option in http.request / http.get

[davalapar]

Hi there, is the lookup option for http.request intentionally undocumented? Sindre's got uses it but I'd want to know if it's stable or not since it's not documented?

• node/test/parallel/test-http-client-timeout-option-with-agent.js

  Line 12 in b361f95

  lookup: () => {},

• node/test/parallel/test-http-client-req-error-dont-double-fire.js

  Line 18 in dcc5e51

  lookup: common.mustCall(errorLookupMock())

• https://github.com/szmarczak/cacheable-lookup
• https://nodejs.org/api/http.html#http_http_request_options_callback

[lpinca]

Yes it's stable, it is forwarded to https://nodejs.org/api/net.html#net_socket_connect_options_connectlistener

[davalapar]

awesome thank you!

i think it also defaults to https://nodejs.org/api/dns.html#dns_dns_lookup_hostname_options_callback, correct?

this is amazing 🎉 🎉