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

Merge pull request #11412 from MikeMcQuaid/improve_github_packages_json_manifest_error

Browse Source 
software_spec: improve bad manifest error message.
This commit is contained in:
Mike McQuaid 2021-05-19 14:37:10 +01:00 committed by GitHub
commit e916f9316e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 15 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
Library/Homebrew/software_spec.rb
View File

@ -347,8 +347,20 @@ class Bottle
end
def fetch_tab
return if github_packages_manifest_resource.blank?
# a checksum is used later identifying the correct tab but we do not have the checksum for the manifest/tab
github_packages_manifest_resource&.fetch(verify_download_integrity: false)
github_packages_manifest_resource.fetch(verify_download_integrity: false)
begin
JSON.parse(github_packages_manifest_resource.cached_download.read)
rescue JSON::ParserError
raise DownloadError.new(
github_packages_manifest_resource,
RuntimeError.new("The downloaded GitHub Packages manifest was corrupted or modified (it is not valid JSON):"\
"\n#{github_packages_manifest_resource.cached_download}"),
)
end
rescue DownloadError
raise unless fallback_on_error
@ -363,7 +375,8 @@ class Bottle
json = begin
JSON.parse(manifest_json)
rescue JSON::ParserError
raise ArgumentError, "Couldn't parse manifest JSON."
raise "The downloaded GitHub Packages manifest was corrupted or modified (it is not valid JSON): "\
"\n#{github_packages_manifest_resource.cached_download}"
end
manifests = json["manifests"]