feross / simple-get

Simplest way to make http get requests. Supports HTTPS, redirects, gzip/deflate, streams in < 100 lines
MIT License
401 stars 50 forks source link

Redirect bug #50

Open eithan1231 opened 5 years ago

eithan1231 commented 5 years ago

Hello again, sorry for spam.

I have noticed the redirection system is somewhat faulty. When redirecting, it does not call url.resolve, so redirects will entirely malfunction if you got a relative path.

To solve this, navigate to the line within index.js, containing the comment // Follow 3xx redirects and update it to opts.url = url.resolve(url.format(opts), res.headers.location); // Follow 3xx redirects

Again, sorry for spamming the issues section.

Regards.