From e5442f4d3b2b69b899663a1491c1b1f0e2816074 Mon Sep 17 00:00:00 2001 From: Daniel Beard <5148750+DanBeard@users.noreply.github.com> Date: Mon, 3 Oct 2022 14:35:40 -0700 Subject: [PATCH 1/2] Removed logging check on releases that was causing dependency fetching to fail --- app/models/package_manager/pypi.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/models/package_manager/pypi.rb b/app/models/package_manager/pypi.rb index 155276a95..9f716d2f9 100644 --- a/app/models/package_manager/pypi.rb +++ b/app/models/package_manager/pypi.rb @@ -127,7 +127,7 @@ def self.dependencies(name, version, _mapped_project) api_response = get("https://pypi.org/pypi/#{name}/#{version}/json") deps = api_response.dig("info", "requires_dist") source_info = api_response.dig("releases", version) - Rails.logger.warn("Pypi sdist (no deps): #{name}") unless source_info.any? { |rel| rel["packagetype"] == "bdist_wheel" } + #Rails.logger.warn("Pypi sdist (no deps): #{name}") unless source_info.any? { |rel| rel["packagetype"] == "bdist_wheel" } deps.map do |dep| name, version = dep.split From 9b4cb0e763e0b0de44fa56dcf91f13e887357534 Mon Sep 17 00:00:00 2001 From: Daniel Beard <5148750+DanBeard@users.noreply.github.com> Date: Mon, 3 Oct 2022 16:33:54 -0700 Subject: [PATCH 2/2] Update app/models/package_manager/pypi.rb Co-authored-by: Tieg Zaharia --- app/models/package_manager/pypi.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/models/package_manager/pypi.rb b/app/models/package_manager/pypi.rb index 9f716d2f9..9eea4ebe3 100644 --- a/app/models/package_manager/pypi.rb +++ b/app/models/package_manager/pypi.rb @@ -127,7 +127,7 @@ def self.dependencies(name, version, _mapped_project) api_response = get("https://pypi.org/pypi/#{name}/#{version}/json") deps = api_response.dig("info", "requires_dist") source_info = api_response.dig("releases", version) - #Rails.logger.warn("Pypi sdist (no deps): #{name}") unless source_info.any? { |rel| rel["packagetype"] == "bdist_wheel" } + Rails.logger.warn("Pypi sdist (no deps): #{name}") unless source_info&.any? { |rel| rel["packagetype"] == "bdist_wheel" } deps.map do |dep| name, version = dep.split