admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

download_strategy: fix error when using custom headers with a redirect

Browse Source
This commit is contained in:
Bo Anderson 2021-08-18 03:02:13 +01:00
parent f026dd21c1
commit de15cd6420
No known key found for this signature in database
GPG Key ID: 3DB94E204E137D65
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Library/Homebrew/download_strategy.rb
View File

@ -391,7 +391,7 @@ class CurlDownloadStrategy < AbstractFileDownloadStrategy
resolved_url, _, url_time, _, is_redirection =
resolve_url_basename_time_file_size(url, timeout: end_time&.remaining!)
# Authorization is no longer valid after redirects
meta[:headers]&.delete_if { |header| header.first&.start_with?("Authorization") } if is_redirection
meta[:headers]&.delete_if { |header| header.start_with?("Authorization") } if is_redirection
fresh = if cached_location.exist? && url_time
url_time <= cached_location.mtime
@ -569,7 +569,7 @@ class CurlGitHubPackagesDownloadStrategy < CurlDownloadStrategy
meta ||= {}
meta[:headers] ||= []
token = Homebrew::EnvConfig.artifact_domain ? Homebrew::EnvConfig.docker_registry_token : "QQ=="
meta[:headers] << ["Authorization: Bearer #{token}"] if token.present?
meta[:headers] << "Authorization: Bearer #{token}" if token.present?
super(url, name, version, meta)
end