ladjs / superagent

Ajax for Node.js and browsers (JS HTTP client). Maintained for @forwardemail, @ladjs, @spamscanner, @breejs, @cabinjs, and @lassjs.
https://ladjs.github.io/superagent/
MIT License
16.58k stars 1.33k forks source link

feat: add lookup method #1595

Closed yunnysunny closed 2 years ago

yunnysunny commented 3 years ago

Add the feature of custom DNS resolve in node. Sometime we use custom DNS to do service discovery, but superagent doesn't have the ability of changing the lookup method when call node's http.request method. So I pull this request to solve the problem.

niftylettuce commented 2 years ago

Released in v7.1.0 to npm

https://github.com/visionmedia/superagent/releases/tag/v7.1.0

niftylettuce commented 2 years ago

This caused a break in v7.1.0 and I had to revert in v7.1.1

https://github.com/visionmedia/superagent/issues/1673

Please submit a new PR with the fix when you can, thank you