aspera-cli 4.26.2 → 4.27.0

data/lib/aspera/transfer/parameters.rb

@@ -2,6 +2,7 @@
 
 require 'aspera/log'
 require 'aspera/assert'
+require 'aspera/exec_spec'
 require 'aspera/command_line_builder'
 require 'aspera/temp_file_manager'
 require 'aspera/transfer/error'
@@ -38,8 +39,9 @@ module Aspera
           @file_list_folder ||= TempFileManager.instance.new_file_path_global('asession_filelists')
         end
 
-        # File list is provided directly with `ascp` arguments
-        # @columns ascp_args [Array,NilClass] `ascp` arguments
+        # Check whether a file list option is already present in the `ascp` argument list
+        # @param ascp_args [Array, nil] `ascp` arguments
+        # @return [Boolean] true if a file list option is present
         def ascp_args_file_list?(ascp_args)
           ascp_args&.any?{ |i| FILE_LIST_OPTIONS.include?(i)}
         end
@@ -47,7 +49,9 @@ module Aspera
 
       # @param job_spec        [Hash]          Transfer specification
       # @param ascp_args       [Array]         Other `ascp` args
+      # @param wss             [Boolean]       `true`: if both SSH and wss in ts: prefer wss
       # @param quiet           [Boolean]       Remove `ascp` progress bar if `true`
+      # @param file_list       [Boolean]       `true`: provide source list via a temp file, `false`: place directly on command line
       # @param client_ssh_key  [:rsa,:dsa]     Type of Aspera Client SSH key to use.
       # @param trusted_certs   [Array<String>] List of path to trusted certificates stores when using WSS
       # @param check_ignore_cb [Proc]          Callback to check if WSS connection shall ignore certificate validity
@@ -56,6 +60,7 @@ module Aspera
         ascp_args:       nil,
         wss:             true,
         quiet:           true,
+        file_list:       true,
         trusted_certs:   nil,
         client_ssh_key:  nil,
         check_ignore_cb: nil
@@ -66,6 +71,7 @@ module Aspera
         Aspera.assert_array_all(@ascp_args, String){'ascp_args'}
         @wss = wss
         @quiet = quiet
+        @file_list = file_list
         @trusted_certs = trusted_certs.nil? ? [] : trusted_certs
         Aspera.assert_type(@trusted_certs, Array){'trusted_certs'}
         @client_ssh_key = client_ssh_key.nil? ? :rsa : client_ssh_key.to_sym
@@ -74,7 +80,7 @@ module Aspera
         @builder = CommandLineBuilder.new(@job_spec, Spec::SCHEMA, CommandLineConverter)
       end
 
-      # either place source files on command line, or add file list file
+      # Place source files on command line or in a temp file list, controlled by @file_list and file_list_folder
       def process_file_list
         # is the file list provided through ascp parameters?
         ascp_file_list_provided = self.class.ascp_args_file_list?(@ascp_args)
@@ -88,8 +94,8 @@ module Aspera
           Aspera.assert(!ascp_file_list_provided, 'file list provided both in transfer spec and ascp file list. Remove one of them.')
           Aspera.assert(ts_paths_array.all?{ |i| i.key?('source')}, "All elements of paths must have a 'source' key")
           is_pair_list = ts_paths_array.any?{ |i| i.key?('destination')}
-          raise "All elements of paths must be consistent with 'destination' key" if is_pair_list && !ts_paths_array.all?{ |i| i.key?('destination')}
-          if self.class.file_list_folder.nil?
+          Aspera.assert(!(is_pair_list && !ts_paths_array.all?{ |i| i.key?('destination')})){"All elements of paths must be consistent with 'destination' key"}
+          if !@file_list || self.class.file_list_folder.nil?
             Aspera.assert(!is_pair_list, 'file pair list is not supported when file list folder is not set')
             # not safe for special characters ? (maybe not, depends on OS)
             Log.log.debug('placing source file list on command line (no file list file)')
@@ -112,7 +118,7 @@ module Aspera
         @builder.add_command_line_options("#{file_list_option}=#{file_list_file}") unless file_list_option.nil?
       end
 
-      # @return the list of certificates (option `-i`) to use when token/ssh or wss are used
+      # @return [Array<String>] the list of certificates (option `-i`) to use when token/ssh or wss are used
       def remote_certificates
         certificates_to_use = []
         # use web socket secure for session ?
@@ -150,11 +156,7 @@ module Aspera
 
       # Translate transfer spec to env vars and command line arguments for `ascp`
       def ascp_args
-        env_args = {
-          args: [],
-          env:  {},
-          name: :ascp
-        }
+        exec_spec = Aspera::ExecSpec.new(exec: :ascp)
 
         # Special cases
         @job_spec.delete('source_root') if @job_spec.key?('source_root') && @job_spec['source_root'].empty?
@@ -165,7 +167,7 @@ module Aspera
         # Add ssh or wss certificates
         # (reverse, to keep order, as we unshift)
         remote_certificates&.reverse_each do |cert|
-          env_args[:args].unshift('-i', cert)
+          exec_spec.args.unshift('-i', cert)
         end
 
         case (delete_source = @builder.read_param('delete_source'))
@@ -183,9 +185,9 @@ module Aspera
         base64_destination = false
         # symbol must be index of Ascp::Installation.paths
         if @builder.read_param('use_ascp4')
-          env_args[:name] = :ascp4
+          exec_spec.exec = :ascp4
         else
-          env_args[:name] = :ascp
+          exec_spec.exec = :ascp
           base64_destination = true
         end
         # destination will be base64 encoded, put this before source path arguments
@@ -200,17 +202,17 @@ module Aspera
         destination_folder = Base64.strict_encode64(destination_folder) if base64_destination
         # destination MUST be last command line argument to ascp
         @builder.add_command_line_options(destination_folder)
-        @builder.add_env_args(env_args)
-        env_args[:args].unshift('-q') if @quiet
+        @builder.add_to_exec_spec(exec_spec)
+        exec_spec.args.unshift('-q') if @quiet
         # add fallback cert and key as arguments if needed
         if HTTP_FALLBACK_ACTIVATION_VALUES.include?(@job_spec['http_fallback'])
-          env_args[:args].unshift('-Y', Ascp::Installation.instance.path(:fallback_private_key))
-          env_args[:args].unshift('-I', Ascp::Installation.instance.path(:fallback_certificate))
+          exec_spec.args.unshift('-Y', Ascp::Installation.instance.path(:fallback_private_key))
+          exec_spec.args.unshift('-I', Ascp::Installation.instance.path(:fallback_certificate))
         end
         # disable redis in client, only for ascp, this makes ascp4 fail
-        env_args[:env]['ASPERA_TEST_REDIS_DISABLE'] = 'true' if env_args[:name].eql?(:ascp)
-        Log.log.debug{"ascp args: #{env_args}"}
-        return env_args
+        exec_spec.env['ASPERA_TEST_REDIS_DISABLE'] = 'true' if exec_spec.exec.eql?(:ascp)
+        Log.log.debug{"ascp args: #{exec_spec}"}
+        return exec_spec
       end
       DELETE_EQUIV = %w[remove_after_transfer remove_empty_directories remove_empty_source_directory]
       # `ascp` options to provide a file list

data/lib/aspera/transfer/result.rb

@@ -0,0 +1,74 @@
+# frozen_string_literal: true
+
+module Aspera
+  module Transfer
+    # Typed result of a transfer operation.
+    # Replaces the ad-hoc Array polymorphism previously returned by agents.
+    #
+    # Agents return one of the three concrete sub-classes:
+    #   Transfer::Result::Success  — all sessions completed successfully
+    #   Transfer::Result::Error    — at least one session failed
+    #   Transfer::Result::Async    — transfer submitted to a daemon; not yet complete
+    class Result
+      # All sessions completed successfully.
+      class Success < Result
+        def to_s
+          'success'
+        end
+      end
+
+      # At least one session failed.
+      class Error < Result
+        # @return [StandardError]
+        attr_reader :exception
+
+        def initialize(exception)
+          super()
+          @exception = exception
+        end
+
+        def to_s
+          "error: #{@exception.message}"
+        end
+      end
+
+      # Transfer submitted to an external daemon; status not yet known.
+      class Async < Result
+        # @return [String] UUID assigned to this background job
+        attr_reader :job_id
+        # @return [String] initial status string (e.g. 'running')
+        attr_reader :status
+
+        def initialize(job_id:, status: 'running')
+          super()
+          @job_id = job_id
+          @status = status
+        end
+
+        # Convenience: export as a plain Hash for serialization / display
+        def to_h
+          {'job_id' => @job_id, 'status' => @status}
+        end
+
+        def to_s
+          "async job_id=#{@job_id} status=#{@status}"
+        end
+      end
+
+      # Factory helpers
+      class << self
+        def success
+          Success.new
+        end
+
+        def error(exception)
+          Error.new(exception)
+        end
+
+        def async(job_id:, status: 'running')
+          Async.new(job_id: job_id, status: status)
+        end
+      end
+    end
+  end
+end

data/lib/aspera/transfer/resumer.rb

@@ -12,20 +12,20 @@ module Aspera
       # @param iter_max      [Integer] Maximum number of executions
       # @param sleep_initial [Integer] Initial wait to re-execute
       # @param sleep_factor  [Integer] Multiplier
-      # @param sleep_max.    [Integer] Max iterations
+      # @param sleep_max     [Integer] Max iterations
       def initialize(
         iter_max: 7,
         sleep_initial: 2,
         sleep_factor:  2,
         sleep_max:     60
       )
-        Aspera.assert_type(iter_max, Integer){k}
+        Aspera.assert_type(iter_max, Integer){'iter_max'}
         @iter_max = iter_max
-        Aspera.assert_type(sleep_initial, Integer){k}
+        Aspera.assert_type(sleep_initial, Integer){'sleep_initial'}
         @sleep_initial = sleep_initial
-        Aspera.assert_type(sleep_factor, Integer){k}
+        Aspera.assert_type(sleep_factor, Integer){'sleep_factor'}
         @sleep_factor = sleep_factor
-        Aspera.assert_type(sleep_max, Integer){k}
+        Aspera.assert_type(sleep_max, Integer){'sleep_max'}
         @sleep_max = sleep_max
       end
 
@@ -53,7 +53,7 @@ module Aspera
             # do not retry non-retryable
             raise unless e.retryable?
             # exit if we exceed the max number of retry
-            raise Error, "Maximum number of retry reached: #{@iter_max}" if remaining_resumes <= 0
+            Aspera.assert(remaining_resumes > 0, type: Error){"Maximum number of retry reached: #{@iter_max}"}
           end
 
           # take this retry in account

data/lib/aspera/transfer/spec.rb

@@ -34,7 +34,13 @@ module Aspera
           'sparse_csum' => 'sparse_checksum',
           'full_csum'   => 'full_checksum'
         }
-        private_constant :POLICY_FIX
+        # Multipliers for target_rate suffix (result is in kbps)
+        RATE_SUFFIX_KBPS = {
+          'k' => 1,
+          'm' => 1_000,
+          'g' => 1_000_000
+        }.freeze
+        private_constant :POLICY_FIX, :RATE_SUFFIX_KBPS
         # translate upload/download to send/receive
         def transfer_type_to_direction(transfer_type)
           XFER_TYPE_TO_DIR.fetch(transfer_type)
@@ -45,6 +51,17 @@ module Aspera
           XFER_DIR_TO_TYPE.fetch(direction)
         end
 
+        # Parse a human-readable rate string (as accepted by ascp -l) into an integer kbps value.
+        # Accepted: plain integer (kbps), or integer + suffix k/K (kbps), m/M (x1000 kbps), g/G (x1000000 kbps).
+        # @param value [String] e.g. "100m", "500000", "1g"
+        # @return [Integer] value in kbps
+        def rate_string_to_kbps(value)
+          m = value.to_s.strip.match(/\A(\d+)([kKmMgG])?\z/)
+          Aspera.assert(m){"Invalid rate value: #{value.inspect}. Expected integer with optional suffix k/K, m/M or g/G."}
+          multiplier = m[2] ? RATE_SUFFIX_KBPS.fetch(m[2].downcase) : 1
+          return m[1].to_i * multiplier
+        end
+
         def fix_transferd_resume_policy(transfer_spec)
           # Fix discrepancy in transfer spec
           transfer_spec['resume_policy'] = POLICY_FIX[transfer_spec['resume_policy']] if transfer_spec.key?('resume_policy')

data/lib/aspera/transfer/spec.schema.yaml

@@ -377,6 +377,7 @@ properties:
     description: Specifies whether to precalculate the job size.
     type: boolean
     x-cli-switch: true
+    x-cli-false: --precalculate-disable
   preserve_access_time:
     description: >-
       Preserve the source-file access timestamps at the destination.
@@ -711,11 +712,11 @@ properties:
       Refer to HSTS manual for more details.
 
       - `follow`: Follow a symbolic link and transfer the contents of the linked file or directory when the link target is in the user's docroot.
-      
+
       - `copy`: Copy only the symbolic link. If a file with the same name exists at the destination, the symbolic link does not replace the file.
-      
+
       - `skip`: Skip-symbolic links. The link or the file to which it points are not transferred.
-      
+
       - `copy+force`: Copy only the symbolic link. If a file with the same name exists at the destination, the symbolic link replaces the file. If the file of the same name at the destination is a symbolic link to a directory, it is not replaced.
     type: string
     enum:
@@ -753,6 +754,14 @@ properties:
     description: Specifies desired speed for the transfer.
     type: integer
     x-cli-option: -l
+  target_rate:
+    description: >-
+      Pseudo-parameter: desired transfer rate as a human-readable string with an optional unit suffix.
+      Converted to `target_rate_kbps` before the transfer is started, overriding it if both are present.
+      Accepted formats: plain integer (kbps), or integer followed by `k`/`K` (kbps), `m`/`M` (x1000 kbps), `g`/`G` (x1000000 kbps).
+      Examples: `100000`, `100000k`, `100m`, `1g`.
+    type: string
+    x-cli-special: true
   title:
     description: Title of the transfer.
     type: string

data/lib/aspera/uri_reader.rb

@@ -45,14 +45,14 @@ module Aspera
           end
         when SCHEME_FILE, NilClass
           local_file_path = uri.path
-          raise Error, 'URL shall have a path, check syntax' if local_file_path.nil?
+          Aspera.assert(!local_file_path.nil?, type: Error){'URL shall have a path, check syntax'}
           local_file_path = File.expand_path(local_file_path.gsub(%r{^/}, '')) if %r{^/(~|.|..)/}.match?(local_file_path)
           return File.read(local_file_path)
         else Aspera.error_unexpected_value(uri.scheme){"scheme for [#{uri_to_read}]"}
         end
       end
 
-      # @return Path to file with content at URL
+      # @return [String] path to file with content at URL
       def read_as_file(url)
         if url.start_with?(SCHEME_FILE_PFX1)
           # for file scheme, return directly the path

data/lib/aspera/web_auth.rb

@@ -6,8 +6,8 @@ require 'aspera/assert'
 module Aspera
   # servlet called on callback: it records the callback request
   class WebAuthServlet < WEBrick::HTTPServlet::AbstractServlet
-    # @param server [WEBrick::HTTPServer]
-    # @param web_auth [WebAuth]
+    # @param server   [WEBrick::HTTPServer] the HTTP server instance
+    # @param web_auth [WebAuth]             the WebAuth instance to record the callback
     def initialize(server, web_auth)
       Log.log.debug('WebAuthServlet initialize')
       super(server)

data/lib/aspera/web_server_simple.rb

@@ -1,11 +1,9 @@
 # frozen_string_literal: true
 
 require 'webrick'
-require 'webrick/https'
 require 'aspera/log'
 require 'aspera/assert'
 require 'aspera/hash_ext'
-require 'openssl'
 
 module Aspera
   # Simple WEBrick server with HTTPS support
@@ -42,7 +40,7 @@ module Aspera
         cert
       end
 
-      # @return a list of Certificates from chain file
+      # @return [Array<OpenSSL::X509::Certificate>] list of Certificates from chain file
       def read_chain_file(chain)
         File.read(chain).scan(/-----BEGIN CERTIFICATE-----.*?-----END CERTIFICATE-----/m).map{ |i| OpenSSL::X509::Certificate.new(i)}
       end
@@ -66,13 +64,18 @@ module Aspera
       when 'http'
         Log.log.debug('HTTP mode')
       when 'https'
+        # Required lazily: webrick/ssl.rb reads OpenSSL::OPENSSL_VERSION at load time,
+        # a constant CosmoRuby's OpenSSL shim does not define, so requiring it
+        # unconditionally at file load would break any command in a Cosmo build
+        # even when no HTTPS server is ever started.
+        require 'webrick/https'
         webrick_options[:SSLEnable] = true
         if cert.nil? && key.nil?
           webrick_options[:SSLCertName] = [['CN', WEBrick::Utils.getservername]]
         elsif cert && PKCS12_EXT.include?(File.extname(cert).downcase)
           # PKCS12
           Log.log.debug('Using PKCS12 certificate')
-          raise Error, 'PKCS12 requires a key (password)' if key.nil?
+          Aspera.assert(!key.nil?, type: Error){'PKCS12 requires a key (password)'}
           pkcs12 = OpenSSL::PKCS12.new(File.read(cert), key)
           webrick_options[:SSLCertificate] = pkcs12.certificate
           webrick_options[:SSLPrivateKey] = pkcs12.key
@@ -90,7 +93,7 @@ module Aspera
             OpenSSL::X509::Certificate.new(File.read(cert))
           end
           webrick_options[:SSLExtraChainCert] = read_chain_file(chain) unless chain.nil?
-          raise Error, 'key and cert do not match' unless webrick_options[:SSLCertificate].public_key.to_der == webrick_options[:SSLPrivateKey].public_key.to_der
+          Aspera.assert(webrick_options[:SSLCertificate].public_key.to_der == webrick_options[:SSLPrivateKey].public_key.to_der, type: Error){'key and cert do not match'}
         end
       end
       # call constructor of parent class, but capture STDERR

data/lib/aspera/yaml.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require 'aspera/assert'
 require 'yaml'
 
 module Aspera
@@ -42,7 +43,7 @@ module Aspera
     # @raise [RuntimeError] If duplicate keys are found
     def safe_load(yaml)
       duplicate_keys = find_duplicate_keys(Psych.parse_stream(yaml))
-      raise "Duplicate keys: #{duplicate_keys.join('; ')}" unless duplicate_keys.empty?
+      Aspera.assert(duplicate_keys.empty?){"Duplicate keys: #{duplicate_keys.join('; ')}"}
       YAML.safe_load(yaml, permitted_classes: [Time, Date, Symbol])
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aspera-cli
 version: !ruby/object:Gem::Version
-  version: 4.26.2
+  version: 4.27.0
 platform: ruby
 authors:
 - Laurent Martin
@@ -319,6 +319,8 @@ files:
 - README.md
 - bin/ascli
 - bin/asession
+- docs/README.md
+- docs/test-mcp-with-ai.md
 - lib/aspera/agent/base.rb
 - lib/aspera/agent/connect.rb
 - lib/aspera/agent/desktop.rb
@@ -334,19 +336,29 @@ files:
 - lib/aspera/api/faspex.rb
 - lib/aspera/api/httpgw.rb
 - lib/aspera/api/node.rb
+- lib/aspera/api/queries/bss_subscription_account.graphql
+- lib/aspera/api/queries/bss_subscription_usage.graphql
 - lib/aspera/ascmd.rb
 - lib/aspera/ascp/installation.rb
 - lib/aspera/ascp/management.rb
 - lib/aspera/assert.rb
+- lib/aspera/cli/ascp_actions.rb
+- lib/aspera/cli/async_transfer_store.rb
+- lib/aspera/cli/bootstrapper.rb
+- lib/aspera/cli/command_registry.rb
+- lib/aspera/cli/command_spec.rb
 - lib/aspera/cli/context.rb
 - lib/aspera/cli/error.rb
 - lib/aspera/cli/extended_value.rb
 - lib/aspera/cli/formatter.rb
+- lib/aspera/cli/gem_checker.rb
 - lib/aspera/cli/hints.rb
 - lib/aspera/cli/http.rb
 - lib/aspera/cli/info.rb
-- lib/aspera/cli/manager.rb
+- lib/aspera/cli/mailer.rb
+- lib/aspera/cli/mcp_tool.rb
 - lib/aspera/cli/options.schema.yaml
+- lib/aspera/cli/parser.rb
 - lib/aspera/cli/plugins/alee.rb
 - lib/aspera/cli/plugins/aoc.rb
 - lib/aspera/cli/plugins/ats.rb
@@ -360,20 +372,25 @@ files:
 - lib/aspera/cli/plugins/faspex5.rb
 - lib/aspera/cli/plugins/faspio.rb
 - lib/aspera/cli/plugins/httpgw.rb
+- lib/aspera/cli/plugins/mcp.rb
 - lib/aspera/cli/plugins/node.rb
 - lib/aspera/cli/plugins/oauth.rb
 - lib/aspera/cli/plugins/orchestrator.rb
 - lib/aspera/cli/plugins/preview.rb
 - lib/aspera/cli/plugins/server.rb
 - lib/aspera/cli/plugins/shares.rb
+- lib/aspera/cli/preset_actions.rb
 - lib/aspera/cli/preset_manager.rb
 - lib/aspera/cli/result.rb
 - lib/aspera/cli/runner.rb
+- lib/aspera/cli/secret_finder.rb
 - lib/aspera/cli/special_values.rb
 - lib/aspera/cli/sync_actions.rb
 - lib/aspera/cli/terminal_formatter.rb
+- lib/aspera/cli/transfer_actions.rb
 - lib/aspera/cli/transfer_agent.rb
 - lib/aspera/cli/transfer_progress.rb
+- lib/aspera/cli/vault_manager.rb
 - lib/aspera/cli/version.rb
 - lib/aspera/cli/wizard.rb
 - lib/aspera/colors.rb
@@ -389,18 +406,22 @@ files:
 - lib/aspera/data_repository.rb
 - lib/aspera/dot_container.rb
 - lib/aspera/environment.rb
+- lib/aspera/exec_spec.rb
 - lib/aspera/faspex_gw.rb
 - lib/aspera/faspex_postproc.rb
 - lib/aspera/formatter_interface.rb
+- lib/aspera/graphql.rb
 - lib/aspera/hash_ext.rb
 - lib/aspera/id_generator.rb
-- lib/aspera/json_rpc.rb
+- lib/aspera/json_rpc/client.rb
+- lib/aspera/json_rpc/version.rb
 - lib/aspera/keychain/base.rb
 - lib/aspera/keychain/encrypted_hash.rb
 - lib/aspera/keychain/factory.rb
 - lib/aspera/keychain/hashicorp_vault.rb
 - lib/aspera/keychain/macos_security.rb
 - lib/aspera/line_logger.rb
+- lib/aspera/link_header.rb
 - lib/aspera/log.rb
 - lib/aspera/markdown.rb
 - lib/aspera/nagios.rb
@@ -451,6 +472,7 @@ files:
 - lib/aspera/transfer/error.rb
 - lib/aspera/transfer/faux_file.rb
 - lib/aspera/transfer/parameters.rb
+- lib/aspera/transfer/result.rb
 - lib/aspera/transfer/resumer.rb
 - lib/aspera/transfer/spec.rb
 - lib/aspera/transfer/spec.schema.yaml

data/lib/aspera/json_rpc.rb

@@ -1,52 +0,0 @@
-# frozen_string_literal: true
-
-# cspell:ignore blankslate
-
-require 'aspera/rest_error_analyzer'
-require 'aspera/assert'
-require 'blankslate'
-
-Aspera::RestErrorAnalyzer.instance.add_simple_handler(name: 'JSON RPC', path: %w[error message], always: true)
-
-module Aspera
-  # a very simple JSON RPC client
-  class JsonRpcClient < BlankSlate
-    JSON_RPC_VERSION = '2.0'
-    reveal :instance_variable_get
-    reveal :inspect
-    reveal :to_s
-
-    def initialize(api, namespace = nil)
-      super()
-      @api = api
-      @namespace = namespace
-      @request_id = 0
-    end
-
-    def respond_to_missing?(_sym, _include_private = false)
-      true
-    end
-
-    def method_missing(method, *args, &block)
-      args = args.first if args.size == 1 && args.first.is_a?(Hash)
-      data = @api.create('', {
-        jsonrpc: JSON_RPC_VERSION,
-        method:  "#{@namespace}#{method}",
-        params:  args,
-        id:      @request_id += 1
-      })
-      Aspera.assert_type(data, Hash){'response'}
-      Aspera.assert(data['jsonrpc'] == JSON_RPC_VERSION, 'bad version in response')
-      Aspera.assert(data.key?('id'), 'missing id in response')
-      Aspera.assert(!(data.key?('error') && data.key?('result')), 'both error and response')
-      Aspera.assert(
-        !data.key?('error') ||
-        data['error'].is_a?(Hash) &&
-        data['error']['code'].is_a?(Integer) &&
-        data['error']['message'].is_a?(String),
-        'bad error response'
-      )
-      return data['result']
-    end
-  end
-end