Skip to content

Commit

Permalink
fix: handle relative urls in redirects (Brooooooklyn#865)
Browse files Browse the repository at this point in the history
  • Loading branch information
lostfictions authored Aug 9, 2024
1 parent 267b682 commit da07e12
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion load-image.js
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ function makeRequest(url, resolve, reject, redirectCount, requestOptions) {
try {
const shouldRedirect = REDIRECT_STATUSES.has(res.statusCode) && typeof res.headers.location === 'string'
if (shouldRedirect && redirectCount > 0)
return makeRequest(new URL(res.headers.location), resolve, reject, redirectCount - 1, requestOptions)
return makeRequest(new URL(res.headers.location, url.origin), resolve, reject, redirectCount - 1, requestOptions)
if (typeof res.statusCode === 'number' && (res.statusCode < 200 || res.statusCode >= 300)) {
return reject(new Error(`remote source rejected with status code ${res.statusCode}`))
}
Expand Down

0 comments on commit da07e12

Please sign in to comment.