simple-get
simple-get copied to clipboard
Remove require('url') in favor of URL
(Once we're ready to drop Node 8 support)
For: https://github.com/brave/brave-browser/issues/5490
It doesn't seem viable without support for relative URLs.
Relative URLs are currently used for the redirect logic and parsed from the Location header so we will have to wait for https://github.com/nodejs/node/pull/28482 to land but it requires Node 12.x — which is likely a leap too far.
https://github.com/nodejs/node/issues/12682 https://github.com/whatwg/url/issues/421
It's possible to do new URL('...', 'http://example.com') to parse a relative URL.