coreutils-wasm 1.1.1 → 1.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 201be27f8a44408474d8c47b84c78e1c5d40dc92d56586e8e34b12f94cf79c07
-  data.tar.gz: 98eafb045cf3f6c2d8ddbb343af22c445c01d59c246efa6ccc62b7c65e2606e3
+  metadata.gz: 65800d9d3ebb71f3402d59452228da0221924072355d28493541ea1ded51ebec
+  data.tar.gz: 56fada4b8e4987674349278d6bfe9c1e3beef9811e81b463ca13d4919763f4eb
 SHA512:
-  metadata.gz: 0f504a6f31c7bdec248ee226f6101766efabfdc5a887007b0b0c3be9b32ed26ccec7f95d0f4fdd3683a9859b8f4f30aec5ec1b55f8ee80c5d2e0f44ac71c8984
-  data.tar.gz: 446a198e51f921ea15ef580425d723e98a5e62f931fb51e86b624dd350426e6966671834605560b4ddd89351313049e8bef9ead3c6457341d9b7f56fab2e57d9
+  metadata.gz: 4c32c5f6118c5774df9308b738703876fcfd0764c80c8adfbe65f8a9dc0cfed0fbcecef3f8ce1dc5a748c094fa7c083d1cf574c2b2c93e4657079bf3ad33a802
+  data.tar.gz: e2a4d442e39f50751ba075e2a2d3286fc36fa8e43b66227255e2712b6d501c291d071abedcb573242037452b335c7bd81d6e098ac94f1204b2ab12fd91bcc3d0

data/lib/coreutils_wasm/downloader.rb

@@ -32,38 +32,43 @@ module CoreutilsWasm
 
       def get_latest_release_tag
         uri = URI("https://api.github.com/repos/#{REPO_OWNER}/#{REPO_NAME}/releases/latest")
-        response = Net::HTTP.get(uri)
-        data = JSON.parse(response)
+
+        response = Net::HTTP.start(uri.host, uri.port, use_ssl: true) do |http|
+          request = Net::HTTP::Get.new(uri)
+          request['Accept'] = 'application/vnd.github+json'
+          http.request(request)
+        end
+
+        unless response.is_a?(Net::HTTPSuccess)
+          raise "GitHub API returned #{response.code}: could not fetch latest release for #{REPO_OWNER}/#{REPO_NAME}"
+        end
+
+        data = JSON.parse(response.body)
         data['tag_name']
       end
 
       def download_asset(tag, target)
         uri = URI("https://github.com/#{REPO_OWNER}/#{REPO_NAME}/releases/download/#{tag}/#{ASSET_NAME}")
-
-        Net::HTTP.start(uri.host, uri.port, use_ssl: true) do |http|
-          request = Net::HTTP::Get.new(uri)
-          http.request(request) do |response|
-            if response.is_a?(Net::HTTPRedirection)
-              return download_from_uri(URI(response['location']), target)
-            end
-
-            File.open(target, 'wb') do |file|
-              response.read_body do |chunk|
-                file.write(chunk)
-              end
-            end
-          end
-        end
+        download_from_uri(uri, target)
       end
 
-      def download_from_uri(uri, target)
+      def download_from_uri(uri, target, redirect_limit: 5)
+        raise 'Too many redirects' if redirect_limit.zero?
+
         Net::HTTP.start(uri.host, uri.port, use_ssl: uri.scheme == 'https') do |http|
           request = Net::HTTP::Get.new(uri)
           http.request(request) do |response|
-            File.open(target, 'wb') do |file|
-              response.read_body do |chunk|
-                file.write(chunk)
+            case response
+            when Net::HTTPRedirection
+              return download_from_uri(URI(response['location']), target, redirect_limit: redirect_limit - 1)
+            when Net::HTTPSuccess
+              File.open(target, 'wb') do |file|
+                response.read_body do |chunk|
+                  file.write(chunk)
+                end
               end
+            else
+              raise "Download failed: server returned #{response.code} for #{uri}"
             end
           end
         end

data/lib/coreutils_wasm/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module CoreutilsWasm
-  VERSION = "1.1.1"
+  VERSION = "1.1.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: coreutils-wasm
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 1.1.2
 platform: ruby
 authors:
 - Nathan Himpens