From e67901f23672519ba63755de7bbfc3e666c9c0a6 Mon Sep 17 00:00:00 2001
From: Mike McQuaid <mike@mikemcquaid.com>
Date: Thu, 8 Dec 2022 09:34:05 +0000
Subject: [PATCH] api/formula: handle JSON file corruption.

If we can't parse the file: it's corrupt. Try again up to 3 times before
bailing.
---
 Library/Homebrew/api/formula.rb | 25 +++++++++++++++++++------
 1 file changed, 19 insertions(+), 6 deletions(-)

diff --git a/Library/Homebrew/api/formula.rb b/Library/Homebrew/api/formula.rb
index 52f3d7018b..0e62566220 100644
--- a/Library/Homebrew/api/formula.rb
+++ b/Library/Homebrew/api/formula.rb
@@ -10,6 +10,8 @@ module Homebrew
       class << self
         extend T::Sig
 
+        MAX_RETRIES = 3
+
         sig { returns(String) }
         def formula_api_path
           "formula"
@@ -29,13 +31,24 @@ module Homebrew
         sig { returns(Hash) }
         def all_formulae
           @all_formulae ||= begin
-            curl_args = %w[--compressed --silent https://formulae.brew.sh/api/formula.json]
-            if cached_formula_json_file.exist? && !cached_formula_json_file.empty?
-              curl_args.prepend("--time-cond", cached_formula_json_file)
-            end
-            curl_download(*curl_args, to: cached_formula_json_file, max_time: 5)
+            retry_count = 0
 
-            json_formulae = JSON.parse(cached_formula_json_file.read)
+            url = "https://formulae.brew.sh/api/formula.json"
+            json_formulae = begin
+              curl_args = %W[--compressed --silent #{url}]
+              if cached_formula_json_file.exist? && !cached_formula_json_file.empty?
+                curl_args.prepend("--time-cond", cached_formula_json_file)
+              end
+              curl_download(*curl_args, to: cached_formula_json_file, max_time: 5)
+
+              JSON.parse(cached_formula_json_file.read)
+            rescue JSON::ParserError
+              cached_formula_json_file.unlink
+              retry_count += 1
+              odie "Cannot download non-corrupt #{url}!" if retry_count > MAX_RETRIES
+
+              retry
+            end
 
             @all_aliases = {}
             json_formulae.to_h do |json_formula|