From 3761d7078547e0fd51542c66cb45b4f17cafbd93 Mon Sep 17 00:00:00 2001 From: Justin Rackliffe Date: Fri, 2 May 2025 11:33:48 -0400 Subject: [PATCH] Revised conditional logic to match CONTRIBUTING.md recommendations on adhering to brew style findings. --- Library/Homebrew/download_strategy.rb | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/Library/Homebrew/download_strategy.rb b/Library/Homebrew/download_strategy.rb index a593835c2e..19e03072a8 100644 --- a/Library/Homebrew/download_strategy.rb +++ b/Library/Homebrew/download_strategy.rb @@ -706,9 +706,11 @@ class CurlGitHubPackagesDownloadStrategy < CurlDownloadStrategy # GitHub Packages authorization header. # HOMEBREW_GITHUB_PACKAGES_AUTH set in brew.sh # If using a private GHCR mirror with no Authentication set than do not add the header. In all other cases add it. - if not (Homebrew::EnvConfig.artifact_domain.presence && !Homebrew::EnvConfig.docker_registry_basic_auth_token.presence && !Homebrew::EnvConfig.docker_registry_token.presence) - meta[:headers] << "Authorization: #{HOMEBREW_GITHUB_PACKAGES_AUTH}" - end + if !Homebrew::EnvConfig.artifact_domain.presence || + Homebrew::EnvConfig.docker_registry_basic_auth_token.presence || + Homebrew::EnvConfig.docker_registry_token.presence + meta[:headers] << "Authorization: #{HOMEBREW_GITHUB_PACKAGES_AUTH}" + end super end