dependabot-gradle 0.351.0 → 0.352.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 30dce402f6a1e1d5d480105632edb8e3d2bfeac81934edb44f31792ee613546a
-  data.tar.gz: db954fea128d5474b45d3b21996387d80818456cea82f563429872d8a9d2d8cc
+  metadata.gz: 9167450605c90c479bfbd32b2f4a0e68458e25a411cc58cee54a380870c3d073
+  data.tar.gz: 6162f15731d05b8d9f041d3101d39b47071caaac5e9b78e3b4ae1393d1fb0dfb
 SHA512:
-  metadata.gz: e5586d92c24df693f36ae79fadf7bf271d23acc62ff0e513b28e147c32e349b64593b2153355d6e995956d9b39f5430e91bbbb0991d3013e095a0dbdb6b88eaf
-  data.tar.gz: 71f5a0b55f4613257c37def935459077338c767a715fd246ffab3125661c2c709f725375a5a7972440b733bd23d865324bf306852eed6ca796272ddce62f4697
+  metadata.gz: 772120fb47bab993074d800657abff25723b064f4b4b15b776ed26acc41bb87292fbc8bf092d1793527d07295f80e9418fc4defa20ee9e21a356f8b4a2d949a2
+  data.tar.gz: 643749b490126dac40f3602a6d7bb428e9072897c0945b6985ddd9163a17460685dd1830ef69fb392343cea06d1e51dcfd15ed225a81b6e26b048e7e4e1f1f1c

data/lib/dependabot/gradle/file_updater/wrapper_updater.rb

@@ -37,6 +37,9 @@ module Dependabot
           )
         end
 
+        # rubocop:disable Metrics/AbcSize
+        # rubocop:disable Metrics/MethodLength
+        # rubocop:disable Metrics/PerceivedComplexity
         sig { params(build_file: Dependabot::DependencyFile).returns(T::Array[Dependabot::DependencyFile]) }
         def update_files(build_file)
           # We only run this updater if it's a distribution dependency
@@ -49,29 +52,57 @@ module Dependabot
           # If we don't have any files in the build files don't generate one
           return [] unless local_files.any?
 
+          # we only run this updater if the build file has a requirement for this dependency
+          target_requirements = dependency.requirements.select do |req|
+            T.let(req[:file], String) == build_file.name
+          end
+          return [] unless target_requirements.any?
+
           updated_files = dependency_files.dup
           SharedHelpers.in_a_temporary_directory do |temp_dir|
             populate_temp_directory(temp_dir)
             cwd = File.join(temp_dir, base_path(build_file))
 
-            # Create gradle.properties file with proxy settings
-            # Would prefer to use command line arguments, but they don't work.
-            properties_filename = File.join(temp_dir, build_file.directory, "gradle.properties")
-            write_properties_file(properties_filename)
-
-            command_parts = %w(gradle --no-daemon --stacktrace) + command_args
-            command = Shellwords.join(command_parts)
+            has_local_script = File.exist?(File.join(cwd, "./gradlew"))
+            command_parts = %w(--no-daemon --stacktrace) + command_args(target_requirements)
+            command = Shellwords.join([has_local_script ? "./gradlew" : "gradle"] + command_parts)
 
             Dir.chdir(cwd) do
-              SharedHelpers.run_shell_command(command, cwd: cwd)
+              FileUtils.chmod("+x", "./gradlew") if has_local_script
+
+              properties_file = File.join(cwd, "gradle/wrapper/gradle-wrapper.properties")
+              validate_option = get_validate_distribution_url_option(properties_file)
+              env = { "JAVA_OPTS" => proxy_args.join(" ") } # set proxy for gradle execution
+
+              begin
+                # first attempt: run the wrapper task via the local gradle wrapper (if present)
+                # `gradle-wrapper.jar` might be too old to run on host's Java version
+                SharedHelpers.run_shell_command(command, cwd: cwd, env: env)
+              rescue SharedHelpers::HelperSubprocessFailed => e
+                raise e unless has_local_script # already field with system one, there is no point to retry
+
+                Dependabot.logger.warn("Running #{command} failed, retrying first with system Gradle: #{e.message}")
+
+                # second attempt: run the wrapper task via system gradle and then retry via local wrapper
+                system_command = Shellwords.join(["gradle"] + command_parts)
+                SharedHelpers.run_shell_command(system_command, cwd: cwd, env: env) # run via system gradle
+                SharedHelpers.run_shell_command(command, cwd: cwd, env: env) # retry via local wrapper
+              end
+
+              # Restore previous validateDistributionUrl option if it existed
+              override_validate_distribution_url_option(properties_file, validate_option)
+
               update_files_content(temp_dir, local_files, updated_files)
             rescue SharedHelpers::HelperSubprocessFailed => e
-              puts "Failed to update files: #{e.message}"
+              Dependabot.logger.error("Failed to update files: #{e.message}")
               return updated_files
             end
           end
           updated_files
         end
+        # rubocop:enable Metrics/AbcSize
+        # rubocop:enable Metrics/MethodLength
+        # rubocop:enable Metrics/PerceivedComplexity
 
         private
 
@@ -80,12 +111,20 @@ module Dependabot
           @target_files.any? { |r| "/#{file.name}".end_with?(r) }
         end
 
-        sig { returns(T::Array[String]) }
-        def command_args
-          version = T.let(dependency.requirements[0]&.[](:requirement), String)
-          checksum = T.let(dependency.requirements[1]&.[](:requirement), String) if dependency.requirements.size > 1
-
-          args = %W(wrapper --no-validate-url --gradle-version #{version})
+        sig { params(requirements: T::Array[T::Hash[Symbol, T.untyped]]).returns(T::Array[String]) }
+        def command_args(requirements)
+          version = T.let(requirements[0]&.[](:requirement), String)
+          checksum = T.let(requirements[1]&.[](:requirement), String) if dependency.requirements.size > 1
+          distribution_url = T.let(requirements[0]&.[](:source), T::Hash[Symbol, String])[:url]
+          distribution_type = distribution_url&.match(/\b(bin|all)\b/)&.captures&.first
+
+          # --no-validate-url is required to bypass HTTP proxy issues when running ./gradlew
+          # This prevents validation failures during the wrapper update process
+          # Note: This temporarily sets validateDistributionUrl=false in gradle-wrapper.properties
+          # The original value is restored after the wrapper task completes
+          # see method `get_validate_distribution_url_option` for more details
+          args = %W(wrapper --gradle-version #{version} --no-validate-url) # see
+          args += %W(--distribution-type #{distribution_type}) if distribution_type
           args += %W(--gradle-distribution-sha256-sum #{checksum}) if checksum
           args
         end
@@ -135,8 +174,35 @@ module Dependabot
           end
         end
 
-        sig { params(file_name: String).void }
-        def write_properties_file(file_name) # rubocop:disable Metrics/PerceivedComplexity
+        # This is a consequence of the lack of proper proxy support in Gradle Wrapper
+        # During the update process, Gradle Wrapper logic will try to validate the distribution URL
+        # by performing an HTTP request. If the environment requires a proxy, this validation will fail
+        # We need to add the `--no-validate-url` the commandline args to disable this validation
+        # However, this change is persistent in the `gradle-wrapper.properties` file
+        # To avoid side effects, we read the existing value before the update and restore it afterward
+        sig { params(properties_file: T.any(Pathname, String)).returns(T.nilable(String)) }
+        def get_validate_distribution_url_option(properties_file)
+          return nil unless File.exist?(properties_file)
+
+          properties_content = File.read(properties_file)
+          properties_content.match(/^validateDistributionUrl=(.*)$/)&.captures&.first
+        end
+
+        sig { params(properties_file: T.any(Pathname, String), value: T.nilable(String)).void }
+        def override_validate_distribution_url_option(properties_file, value)
+          return unless File.exist?(properties_file)
+
+          properties_content = File.read(properties_file)
+          updated_content = properties_content.gsub(
+            /^validateDistributionUrl=(.*)\n/,
+            value ? "validateDistributionUrl=#{value}\n" : ""
+          )
+          File.write(properties_file, updated_content)
+        end
+
+        # rubocop:disable Metrics/PerceivedComplexity
+        sig { returns(T::Array[String]) }
+        def proxy_args
           http_proxy = ENV.fetch("HTTP_PROXY", nil)
           https_proxy = ENV.fetch("HTTPS_PROXY", nil)
           http_split = http_proxy&.split(":")
@@ -145,13 +211,15 @@ module Dependabot
           https_proxy_host = https_split&.fetch(1, nil)&.gsub("//", "") || "host.docker.internal"
           http_proxy_port = http_split&.fetch(2) || "1080"
           https_proxy_port = https_split&.fetch(2) || "1080"
-          properties_content = "
-systemProp.http.proxyHost=#{http_proxy_host}
-systemProp.http.proxyPort=#{http_proxy_port}
-systemProp.https.proxyHost=#{https_proxy_host}
-systemProp.https.proxyPort=#{https_proxy_port}"
-          File.write(file_name, properties_content)
+
+          args = []
+          args += %W(-Dhttp.proxyHost=#{http_proxy_host}) if http_proxy_host
+          args += %W(-Dhttp.proxyPort=#{http_proxy_port}) if http_proxy_port
+          args += %W(-Dhttps.proxyHost=#{https_proxy_host}) if https_proxy_host
+          args += %W(-Dhttps.proxyPort=#{https_proxy_port}) if https_proxy_port
+          args
         end
+        # rubocop:enable Metrics/PerceivedComplexity
 
         sig { returns(T::Array[Dependabot::DependencyFile]) }
         attr_reader :dependency_files

data/lib/dependabot/gradle/file_updater.rb

@@ -56,9 +56,7 @@ module Dependabot
       end
 
       # rubocop:disable Metrics/AbcSize
-      # rubocop:disable Metrics/CyclomaticComplexity
       # rubocop:disable Metrics/PerceivedComplexity
-      # rubocop:disable Metrics/MethodLength
       sig do
         params(buildfiles: T::Array[Dependabot::DependencyFile], dependency: Dependabot::Dependency)
           .returns(T::Array[Dependabot::DependencyFile])
@@ -103,33 +101,42 @@ module Dependabot
         end
 
         # runs native updaters (e.g. wrapper, lockfile) on relevant build files updated
-        updated_files = T.let([], T::Array[Dependabot::DependencyFile])
-        buildfiles_processed.each do |_, buildfile|
-          if Dependabot::Experiments.enabled?(:gradle_wrapper_updater)
+        if Dependabot::Experiments.enabled?(:gradle_wrapper_updater)
+          buildfiles_processed.each_value do |buildfile|
             wrapper_updater = WrapperUpdater.new(dependency_files: files, dependency: dependency)
-            updated_files += wrapper_updater.update_files(buildfile)
+            updated_files = wrapper_updater.update_files(buildfile)
+            replace_updated_files(files, updated_files)
           end
-          if Dependabot::Experiments.enabled?(:gradle_lockfile_updater)
+        end
+        if Dependabot::Experiments.enabled?(:gradle_lockfile_updater)
+          buildfiles_processed.each_value do |buildfile|
             lockfile_updater = LockfileUpdater.new(dependency_files: files)
-            updated_files += lockfile_updater.update_lockfiles(buildfile)
+            updated_files = lockfile_updater.update_lockfiles(buildfile)
+            replace_updated_files(files, updated_files)
           end
         end
 
+        files
+      end
+      # rubocop:enable Metrics/PerceivedComplexity
+      # rubocop:enable Metrics/AbcSize
+      sig do
+        params(
+          files: T::Array[Dependabot::DependencyFile],
+          updated_files: T::Array[Dependabot::DependencyFile]
+        ).returns(T::Array[Dependabot::DependencyFile])
+      end
+      def replace_updated_files(files, updated_files)
         updated_files.each do |file|
-          existing_file = files.find { |f| f.name == file.name && f.directory == file.directory }
+          existing_file = files.find { |f| f.name == file.name }
           if existing_file.nil?
             files << file
           else
             files[T.must(files.index(existing_file))] = file
           end
         end
-
         files
       end
-      # rubocop:enable Metrics/PerceivedComplexity
-      # rubocop:enable Metrics/CyclomaticComplexity
-      # rubocop:enable Metrics/AbcSize
-      # rubocop:enable Metrics/MethodLength
 
       sig do
         params(

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dependabot-gradle
 version: !ruby/object:Gem::Version
-  version: 0.351.0
+  version: 0.352.0
 platform: ruby
 authors:
 - Dependabot
@@ -15,28 +15,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.351.0
+        version: 0.352.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.351.0
+        version: 0.352.0
 - !ruby/object:Gem::Dependency
   name: dependabot-maven
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.351.0
+        version: 0.352.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.351.0
+        version: 0.352.0
 - !ruby/object:Gem::Dependency
   name: debug
   requirement: !ruby/object:Gem::Requirement
@@ -284,7 +284,7 @@ licenses:
 - MIT
 metadata:
   bug_tracker_uri: https://github.com/dependabot/dependabot-core/issues
-  changelog_uri: https://github.com/dependabot/dependabot-core/releases/tag/v0.351.0
+  changelog_uri: https://github.com/dependabot/dependabot-core/releases/tag/v0.352.0
 rdoc_options: []
 require_paths:
 - lib