fastlane-plugin-firebase_app_distribution 0.7.1.pre.2 → 0.7.2.pre.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f18fcc30ef0dac6c2fc77d0813e51fc6b2c602e76e397c69aec31bbbdb69baf5
-  data.tar.gz: 00f4aa04fffd04807c931952908c2c885cb6b5b53a49a145e48b634225e130cc
+  metadata.gz: 01fc473b0dc2f5c03472c4d98f1748c6f8183d23dac7b11416d6c30f4d822862
+  data.tar.gz: 2d9942f8e9a28fc9ca5eb8af100ea2867560aef80558e53d47976b2bc4724a83
 SHA512:
-  metadata.gz: ac9ef34ece7fece99ee759c4163585b58a96208ce3470a067446bca2fe99e769d85e48cd321d532369b7f15cdcbcc284eeccfe7147f8a8f65aa24e760337ddb7
-  data.tar.gz: 762d50b9e45c3ec08920c0cd6c8031da03786d3e6a48293053d89f540c0ee4d441fc45ce4c323a37b3f99d641a458e6fcedc2e077490b819ecadabc8e23dd065
+  metadata.gz: e496e43baf4a81d3eb86ea734d2458a548f13a1e3f15ba969feb9cfd8fd3e004c51b7472a345f61b7767afeb0d4bd7d56612956b69b1e3b55bc93533d825a4e3
+  data.tar.gz: e0e5a877e7664d017981d623ed9f8d51a4dce186cb0b75bc17de1325e01ee6bd0c5a9b180014a98fc4817051d31502bdcb6a305c7263530c92396a82b38d9cbc

data/lib/fastlane/plugin/firebase_app_distribution/actions/firebase_app_distribution_action.rb

@@ -42,8 +42,7 @@ module Fastlane
                              params[:debug],
                              timeout)
 
-        # If binary is an AAB, get the AAB info for this app, which includes the integration state
-        # and certificate data
+        # If binary is an AAB, get the AAB info for this app, which includes the integration state and certificate data
         if binary_type == :AAB
           aab_info = get_aab_info(client, app_name)
           validate_aab_setup!(aab_info)
@@ -51,8 +50,21 @@ module Fastlane
 
         binary_type = binary_type_from_path(binary_path)
         UI.message("⌛ Uploading the #{binary_type}.")
-        operation = upload_binary(app_name, binary_path, client, timeout)
-        release = poll_upload_release_operation(client, operation, binary_type)
+
+        # For some reason calling the client.upload_medium returns nil when
+        # it should return a long running operation object
+        # (https://github.com/googleapis/google-api-ruby-client/blob/main/generated/google-apis-firebaseappdistribution_v1/lib/google/apis/firebaseappdistribution_v1/service.rb#L79).
+        # We could use client.http, but is much slower
+        # (https://github.com/firebase/fastlane-plugin-firebase_app_distribution/issues/330),
+        # so we still use the old client for now.
+        # TODO(kbolay) Prefer client.upload_medium, assuming it is sufficiently fast
+        fad_api_client = Client::FirebaseAppDistributionApiClient.new(client.authorization.access_token, params[:debug])
+        operation_name = fad_api_client.upload_binary(app_name,
+                                                      binary_path,
+                                                      platform.to_s,
+                                                      get_upload_timeout(params))
+
+        release = poll_upload_release_operation(client, operation_name, binary_type)
 
         if binary_type == :AAB && aab_info && !aab_certs_included?(aab_info.test_certificate)
           updated_aab_info = get_aab_info(client, app_name)
@@ -210,8 +222,8 @@ module Fastlane
         release_notes_param || Actions.lane_context[SharedValues::FL_CHANGELOG]
       end
 
-      def self.poll_upload_release_operation(client, operation, binary_type)
-        operation = client.get_project_app_release_operation(operation.name)
+      def self.poll_upload_release_operation(client, operation_name, binary_type)
+        operation = client.get_project_app_release_operation(operation_name)
         MAX_POLLING_RETRIES.times do
           if operation.done && operation.response && operation.response['release']
             release = extract_release(operation)
@@ -245,30 +257,6 @@ module Fastlane
         extract_release(operation)
       end
 
-      def self.upload_binary(app_name, binary_path, client, timeout)
-        options = Google::Apis::RequestOptions.new
-        options.max_elapsed_time = timeout # includes retries (default = no retries)
-        options.header = {
-          'Content-Type' => 'application/octet-stream',
-          'X-Goog-Upload-File-Name' => File.basename(binary_path),
-          'X-Goog-Upload-Protocol' => 'raw'
-        }
-
-        # For some reason calling the client.upload_medium returns nil when
-        # it should return a long running operation object, so we make a
-        # standard http call instead and convert it to a long running object
-        # https://github.com/googleapis/google-api-ruby-client/blob/main/generated/google-apis-firebaseappdistribution_v1/lib/google/apis/firebaseappdistribution_v1/service.rb#L79
-        # TODO(kbolay) Prefer client.upload_medium
-        response = client.http(
-          :post,
-          "https://firebaseappdistribution.googleapis.com/upload/v1/#{app_name}/releases:upload",
-          body: File.open(binary_path, 'rb').read,
-          options: options
-        )
-
-        Google::Apis::FirebaseappdistributionV1::GoogleLongrunningOperation.from_json(response)
-      end
-
       def self.extract_release(operation)
         Google::Apis::FirebaseappdistributionV1::GoogleFirebaseAppdistroV1Release.from_json(operation.response['release'].to_json)
       end

data/lib/fastlane/plugin/firebase_app_distribution/client/firebase_app_distribution_api_client.rb

@@ -7,8 +7,12 @@ module Fastlane
       include Helper::FirebaseAppDistributionHelper
 
       BASE_URL = "https://firebaseappdistribution.googleapis.com"
+      MAX_POLLING_RETRIES = 60
+      POLLING_INTERVAL_SECONDS = 5
 
       AUTHORIZATION = "Authorization"
+      CONTENT_TYPE = "Content-Type"
+      APPLICATION_OCTET_STREAM = "application/octet-stream"
       CLIENT_VERSION = "X-Client-Version"
 
       def initialize(auth_token, debug = false)
@@ -16,6 +20,33 @@ module Fastlane
         @debug = debug
       end
 
+      # Uploads the app binary to the Firebase API
+      #
+      # args
+      #   app_name - Firebase App resource name
+      #   binary_path - Absolute path to your app's aab/apk/ipa file
+      #   platform - 'android' or 'ios'
+      #   timeout - The amount of seconds before the upload will timeout, if not completed
+      #
+      # Returns the long-running operation name.
+      #
+      # Throws a user_error if the binary file does not exist
+      def upload_binary(app_name, binary_path, platform, timeout)
+        response = connection.post(binary_upload_url(app_name), read_binary(binary_path)) do |request|
+          request.options.timeout = timeout # seconds
+          request.headers[AUTHORIZATION] = "Bearer " + @auth_token
+          request.headers[CONTENT_TYPE] = APPLICATION_OCTET_STREAM
+          request.headers[CLIENT_VERSION] = client_version_header_value
+          request.headers["X-Goog-Upload-File-Name"] = File.basename(binary_path)
+          request.headers["X-Goog-Upload-Protocol"] = "raw"
+        end
+
+        response.body[:name] || ''
+      rescue Errno::ENOENT # Raised when binary_path file does not exist
+        binary_type = binary_type_from_path(binary_path)
+        UI.user_error!("#{ErrorMessage.binary_not_found(binary_type)}: #{binary_path}")
+      end
+
       # Get tester UDIDs
       #
       # args
@@ -40,6 +71,10 @@ module Fastlane
         "fastlane/#{Fastlane::FirebaseAppDistribution::VERSION}"
       end
 
+      def binary_upload_url(app_name)
+        "/upload/v1/#{app_name}/releases:upload"
+      end
+
       def get_udids_url(app_id)
         "/v1alpha/apps/#{app_id}/testers:getTesterUdids"
       end
@@ -52,6 +87,11 @@ module Fastlane
           conn.adapter(Faraday.default_adapter)
         end
       end
+
+      def read_binary(path)
+        # File must be read in binary mode to work on Windows
+        File.open(path, 'rb').read
+      end
     end
   end
 end

data/lib/fastlane/plugin/firebase_app_distribution/helper/firebase_app_distribution_auth_client.rb

@@ -38,9 +38,13 @@ module Fastlane
         elsif !ENV["FIREBASE_TOKEN"].nil? && !ENV["FIREBASE_TOKEN"].empty?
           UI.message("🔐 Authenticating with FIREBASE_TOKEN environment variable")
           firebase_token(ENV["FIREBASE_TOKEN"], debug)
-        elsif !application_default_creds.nil?
-          UI.message("🔐 Authenticating with Application Default Credentials")
-          application_default_creds
+        # TODO(lkellogg): Not using Google::Auth.get_application_default yet while we are still
+        # using the old client for uploads. ADC also does not work for the get_udids action:
+        # https://cloud.google.com/docs/authentication/troubleshoot-adc#user-creds-client-based
+        # For now go back to just using the environment variable:
+        elsif !ENV["GOOGLE_APPLICATION_CREDENTIALS"].nil? && !ENV["GOOGLE_APPLICATION_CREDENTIALS"].empty?
+          UI.message("🔐 Authenticating with GOOGLE_APPLICATION_CREDENTIALS environment variable: #{ENV['GOOGLE_APPLICATION_CREDENTIALS']}")
+          service_account(ENV["GOOGLE_APPLICATION_CREDENTIALS"], debug)
         elsif (refresh_token = refresh_token_from_firebase_tools)
           UI.message("🔐 No authentication method found. Using cached Firebase CLI credentials.")
           firebase_token(refresh_token, debug)
@@ -52,12 +56,6 @@ module Fastlane
 
       private
 
-      def application_default_creds
-        Google::Auth.get_application_default([SCOPE])
-      rescue
-        nil
-      end
-
       def refresh_token_from_firebase_tools
         config_path = format_config_path
         if File.exist?(config_path)

data/lib/fastlane/plugin/firebase_app_distribution/version.rb

@@ -1,5 +1,5 @@
 module Fastlane
   module FirebaseAppDistribution
-    VERSION = "0.7.1.pre.2"
+    VERSION = "0.7.2.pre.1"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fastlane-plugin-firebase_app_distribution
 version: !ruby/object:Gem::Version
-  version: 0.7.1.pre.2
+  version: 0.7.2.pre.1
 platform: ruby
 authors:
 - Stefan Natchev
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-08-02 00:00:00.000000000 Z
+date: 2023-08-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: google-apis-firebaseappdistribution_v1