Allow omitting token if api_base_url is file://

Author: AliSoftware

lib/fastlane/plugin/wpmreleasetoolkit/actions/ios/ios_send_app_size_metrics.rb

@@ -5,13 +5,22 @@ module Fastlane
   module Actions
     class IosSendAppSizeMetricsAction < Action
       def self.run(params)
+        # Check input parameters
+        base_url = URI(params[:api_base_url])
+        api_token = params[:api_token]
+        if (api_token.nil? || api_token.empty?) && !base_url.is_a?(URI::File)
+          UI.user_error!('An API token is required when using an `api_base_url` with a scheme other than `file://`')
+        end
+
+        # Build the payload base
         payload = Fastlane::WPMRT::AppSizeMetricsPayload.new(
           'App Name': params[:app_name],
           'Build Type': params[:build_type],
           'App Version': params[:app_version]
         )
         payload.add_metric(name: 'File Size', value: File.size(params[:ipa_path]))
 
+        # Add app-thinning metrics to the payload if a `.plist` is provided
         app_thinning_plist_path = params[:app_thinning_plist_path] || File.join(File.dirname(params[:ipa_path]), 'app-thinning.plist')
         if File.exist?(app_thinning_plist_path)
           plist = Plist.parse_xml(app_thinning_plist_path)
@@ -25,9 +34,10 @@ def self.run(params)
           end
         end
 
+        # Send the payload
         payload.send_metrics(
-          base_url: params[:api_base_url],
-          api_token: params[:api_token]
+          base_url: base_url,
+          api_token: api_token
         )
       end
 
@@ -57,7 +67,7 @@ def self.available_options
             env_name: 'FL_IOS_SEND_APP_SIZE_METRICS_API_TOKEN',
             description: 'The bearer token to call to API',
             type: String,
-            optional: false
+            optional: true # Required unless `api_base_url` is a `file://` URL
           ),
           FastlaneCore::ConfigItem.new(
             key: :app_name,

lib/fastlane/plugin/wpmreleasetoolkit/models/app_size_metrics_payload.rb

@@ -43,12 +43,12 @@ def to_h
       #
       # Must conform to the API described in https://github.com/Automattic/apps-metrics/wiki/Queue-Group-of-Metrics
       #
-      # @param [String] base_url The base URL of the App Metrics service
+      # @param [String,URI] base_url The base URL of the App Metrics service
       # @param [String] api_token The API bearer token to use to register the metric.
       # @return [Integer] the HTTP response code
       #
       def send_metrics(base_url:, api_token:)
-        uri = URI.parse(base_url)
+        uri = URI(base_url)
         # Allow using a `file:` URI for debugging
         if uri.is_a?(URI::File)
           UI.message("Writing metrics payload to file #{uri.path} (instead of sending it to a server)")