aspera-cli 4.11.0 → 4.12.0

data/bin/asession

@@ -15,9 +15,9 @@ TS_TMP_FILELIST_FOLDER = 'EX_file_list_folder'
 
 SAMPLE_DEMO = '"remote_host":"demo.asperasoft.com","remote_user":"asperaweb","ssh_port":33001,"remote_password":"demoaspera"'
 SAMPLE_DEMO2 = '"direction":"receive","destination_root":"./test.dir"'
-def assert_usage(assertion, errmsg)
+def assert_usage(assertion, error_message)
   return if assertion
-  $stderr.puts('ERROR: '.red.blink + errmsg) if errmsg
+  $stderr.puts('ERROR: '.red.blink + error_message) if error_message
   $stderr.puts('USAGE')
   $stderr.puts('    asession')
   $stderr.puts('    asession -h|--help')

data/docs/test_env.conf

@@ -139,6 +139,7 @@ misc:
   faspex_dbx: your value here
   faspex_wkg: your value here
   faspex_src: your value here
+  faspex5_shinbox: your value here
   faspex5_meta: your value here
   shares_upload: your value here
   console_smart_id: your value here

data/examples/aoc.rb

@@ -17,7 +17,7 @@ aoc_url = ARGV[0]
 aoc_user = ARGV[1]
 aoc_key_value = ARGV[2]
 
-aocapi = Aspera::AoC.new(
+aoc_api = Aspera::AoC.new(
   url: aoc_url,
   auth: :jwt,
   private_key: aoc_key_value,
@@ -25,6 +25,6 @@ aocapi = Aspera::AoC.new(
   scope: 'user:all',
   subpath: 'api/v1')
 
-self_user_data = aocapi.read('self')
+self_user_data = aoc_api.read('self')
 
 Aspera::Log.dump('self', self_user_data)

data/examples/dascli

@@ -1,20 +1,20 @@
 #!/usr/bin/env bash
 # set env var image to specify another docker image
-: ${image:=martinlaurent/ascli}
+: "${image:=martinlaurent/ascli}"
 # set env var version to specify another image version (default: latest version)
-: ${version:=latest}
+: "${version:=latest}"
 # set env var imgtag to specify a specific image/version
-: ${imgtag=$image:$version}
+: "${imgtag=$image:$version}"
 # set env var `docker` to podman, to use podman
-: ${docker:=docker}
-# set env var docker_args to add options to docker run (transform var into array)
-add_dock_args=( $docker_args )
+: "${docker:=docker}"
+# set env var docker_args to add options to docker run (transform var into array) # spellcheck disable=SC2086
+read -a add_dock_args <<< $docker_args
 # set env var ASCLI_HOME to set the config folder on host
-: ${ASCLI_HOME:=$HOME/.aspera/ascli}
+: "${ASCLI_HOME:=$HOME/.aspera/ascli}"
 # main config folder for ascli in container (same value as in `Dockerfile`)
 ascli_home_container=/home/cliuser/.aspera/ascli
-if test ! -d $ASCLI_HOME;then
-	echo "creating foder: $ASCLI_HOME"
+if test ! -d "$ASCLI_HOME";then
+	echo "creating folder: $ASCLI_HOME"
 	# create it if necessary to allow mounting the volume in container
 	mkdir -p "$ASCLI_HOME"
 fi
@@ -23,8 +23,8 @@ exec $docker run \
 --tty \
 --interactive \
 --user root \
---env ASCLI_HOME=$ascli_home_container \
+--env ASCLI_HOME="$ascli_home_container" \
 --volume "$ASCLI_HOME:$ascli_home_container" \
 "${add_dock_args[@]}" \
-$imgtag \
+"$imgtag" \
 "$@"

data/examples/faspex4.rb

@@ -82,13 +82,13 @@ end
 api_v4 = Aspera::Rest.new({
   base_url: faspex_url + '/api',
   auth:     {
-    type:     :oauth2,
-    base_url: faspex_url + '/auth/oauth2',
-    auth:     {type: :basic, username: faspex_user, password: faspex_pass},
-    crtype:   :generic,
-    generic:  {grant_type: 'password'},
-    scope:    'admin'
+    type:         :oauth2,
+    base_url:     faspex_url + '/auth/oauth2',
+    auth:         {type: :basic, username: faspex_user, password: faspex_pass},
+    grant_method: :generic,
+    generic:      {grant_type: 'password'},
+    scope:        'admin'
   }})
 
-# Use it. Note that Faspex 4 API v4 is totally different from Faspex 4 v3 APIs, see ref on line 2
+# Use it. Note that Faspex 4 API v4 is totally different from Faspex 4 v3 APIs, see ref in header
 Aspera::Log.dump('users', api_v4.read('users')[:data])

data/examples/node.rb

@@ -23,7 +23,7 @@ node_user = ARGV.shift
 node_pass = ENV['PASSWORD']
 
 ##############################################################
-# generic initialisation : configuration of FaspManager
+# generic initialization : configuration of FaspManager
 
 # set trace level for sample, set to :debug to see complete list of debug information
 Aspera::Log.instance.level = :debug

data/examples/proxy.pac

@@ -4,7 +4,7 @@ function FindProxyForURL(url, host) {
 	url = url.toLowerCase();
 	host = host.toLowerCase();
 
-	/* Don't proxy local hostnames */
+	/* Don't proxy local host names */
 	if (isPlainHostName(host)) {
 		return 'DIRECT';
 	}
@@ -31,7 +31,7 @@ function FindProxyForURL(url, host) {
 	if (isResolvable(host)) {
 		var hostIP = dnsResolve(host);
 
-		/* Don't proxy non-routable addresses (RFC 3330) */
+		/* Don't proxy private addresses (RFC 3330) */
 		if (isInNet(hostIP, '0.0.0.0', '255.0.0.0')
 				|| isInNet(hostIP, '10.0.0.0', '255.0.0.0')
 				|| isInNet(hostIP, '127.0.0.0', '255.0.0.0')

data/examples/server.rb

@@ -23,7 +23,7 @@ server_user = ARGV.shift
 server_pass = ENV['PASSWORD']
 
 ##############################################################
-# generic initialisation : configuration of FaspManager
+# generic initialization : configuration of FaspManager
 
 # set trace level for sample, set to :debug to see complete list of debug information
 Aspera::Log.instance.level = :debug
@@ -66,7 +66,7 @@ transfer_agent.add_listener(MyListener.new)
 ##############################################################
 # first example: download by SSH credentials
 
-# manually build teansfer spec
+# manually build transfer spec
 transfer_spec = {
   'remote_host'      => server_uri.host,
   'ssh_port'         => server_uri.port,
@@ -82,7 +82,7 @@ transfer_spec = {
 transfer_agent.start_transfer(transfer_spec)
 
 # optional: helper method: wait for completion of transfers
-# here we started a single transfer session (no multisession parameter)
+# here we started a single transfer session (no multi session parameter)
 # get array of status, one for each session (so, a single value array)
 # each status is either :success or "error message"
 transfer_result = transfer_agent.wait_for_transfers_completion

data/lib/aspera/aoc.rb

@@ -13,14 +13,14 @@ Aspera::Oauth.register_token_creator(
   lambda{|o|
     o.api.call({
       operation:   'POST',
-      subpath:     o.gparams[:path_token],
+      subpath:     o.generic_parameters[:path_token],
       headers:     {'Accept' => 'application/json'},
-      json_params: o.sparams[:json],
-      url_params:  o.sparams[:url].merge(scope: o.gparams[:scope]) # scope is here because it changes over time (node)
+      json_params: o.specific_parameters[:json],
+      url_params:  o.specific_parameters[:url].merge(scope: o.generic_parameters[:scope]) # scope is here because it changes over time (node)
     })
   },
   lambda { |oauth|
-    return [oauth.sparams.dig(:json, :url_token)]
+    return [oauth.specific_parameters.dig(:json, :url_token)]
   })
 
 module Aspera
@@ -122,7 +122,7 @@ module Aspera
             raise ArgumentError, 'link option must be URL with "token" parameter' if url_param_token_pair.nil?
             # ok we get it !
             a_opt[:url] = 'https://' + uri.host
-            a_auth[:crtype] = :aoc_pub_link
+            a_auth[:grant_method] = :aoc_pub_link
             a_auth[:aoc_pub_link] = {
               url:  {grant_type: 'url_token'}, # URL args
               json: {url_token: url_param_token_pair.last} # JSON body
@@ -162,7 +162,7 @@ module Aspera
       # shortcut to auth section
       aoc_auth_p = aoc_rest_p[:auth]
 
-      # sets opt[:url], aoc_rest_p[:auth][:crtype], [:auth][:aoc_pub_link] if there is a link
+      # sets opt[:url], aoc_rest_p[:auth][:grant_method], [:auth][:aoc_pub_link] if there is a link
       self.class.resolve_pub_link(aoc_auth_p, opt)
 
       # test here because link may set url
@@ -181,9 +181,9 @@ module Aspera
       aoc_auth_p[:scope] = opt[:scope]
 
       # filled if pub link
-      if !aoc_auth_p.key?(:crtype)
+      if !aoc_auth_p.key?(:grant_method)
         raise ArgumentError, 'Missing mandatory option: auth' if opt[:auth].nil?
-        aoc_auth_p[:crtype] = opt[:auth]
+        aoc_auth_p[:grant_method] = opt[:auth]
       end
 
       if aoc_auth_p[:client_id].nil?
@@ -191,7 +191,7 @@ module Aspera
       end
 
       # fill other auth parameters based on Oauth method
-      case aoc_auth_p[:crtype]
+      case aoc_auth_p[:grant_method]
       when :web
         raise ArgumentError, 'Missing mandatory option: redirect_uri' if opt[:redirect_uri].nil?
         aoc_auth_p[:web] = {redirect_uri: opt[:redirect_uri]}
@@ -211,13 +211,13 @@ module Aspera
       when :aoc_pub_link
         # basic auth required for /token
         aoc_auth_p[:auth] = {type: :basic, username: aoc_auth_p[:client_id], password: aoc_auth_p[:client_secret]}
-      else raise "ERROR: unsupported auth method: #{aoc_auth_p[:crtype]}"
+      else raise "ERROR: unsupported auth method: #{aoc_auth_p[:grant_method]}"
       end
       super(aoc_rest_p)
     end
 
     def url_token_data
-      return nil unless params[:auth][:crtype].eql?(:aoc_pub_link)
+      return nil unless params[:auth][:grant_method].eql?(:aoc_pub_link)
       return @cache_url_token_info unless @cache_url_token_info.nil?
       # TODO: can there be several in list ?
       @cache_url_token_info = read('url_tokens')[:data].first
@@ -225,7 +225,7 @@ module Aspera
     end
 
     def additional_persistence_ids
-      return [user_info['id']] if url_token_data.nil?
+      return [current_user_info['id']] if url_token_data.nil?
       return [] # TODO : url_token_data['id'] ?
     end
 
@@ -236,7 +236,7 @@ module Aspera
     end
 
     # cached user information
-    def user_info(exception: false)
+    def current_user_info(exception: false)
       if @cache_user_info.nil?
         # get our user's default information
         @cache_user_info =
@@ -255,7 +255,21 @@ module Aspera
     # @returns [Aspera::Node] a node API for access key
     # @param node_id [String] identifier of node in AoC
     # @param scope e.g. SCOPE_NODE_USER, or nil (requires secret)
-    def node_id_to_api(node_id:, plugin:, scope: nil, package_info: nil)
+    def node_api_from(node_id: nil, workspace_info: nil, package_info: nil, scope: nil)
+      if node_id.nil?
+        if package_info.nil?
+          raise 'INTERNAL ERROR: either node_id or package_info is required'
+        else
+          node_id = package_info['node_id']
+        end
+      end
+      if workspace_info.nil?
+        if package_info.nil?
+          raise 'INTERNAL ERROR: either workspace_info or package_info is required'
+        else
+          workspace_info = package_info['workspace_id']
+        end
+      end
       node_info = read("nodes/#{node_id}")[:data]
       node_rest_params = {base_url: node_info['url']}
       # if secret is available
@@ -270,20 +284,20 @@ module Aspera
         node_rest_params[:auth] = params[:auth].clone
         node_rest_params[:auth][:scope] = self.class.node_scope(node_info['access_key'], scope)
         # special header required for bearer token only
-        node_rest_params[:headers] = {Aspera::Node::X_ASPERA_ACCESSKEY => node_info['access_key']}
+        node_rest_params[:headers] = {Aspera::Node::HEADER_X_ASPERA_ACCESS_KEY => node_info['access_key']}
       end
       app_info = {
-        plugin:    plugin,
-        node_info: node_info,
-        app:       package_info.nil? ? FILES_APP : PACKAGES_APP,
-        api:       self # for callback
+        node_info:      node_info,
+        workspace_info: workspace_info,
+        app:            package_info.nil? ? FILES_APP : PACKAGES_APP,
+        api:            self # for callback
       }
       app_info[:package_info] = package_info unless package_info.nil?
       return Node.new(params: node_rest_params, app_info: app_info)
     end
 
     # Query entity type by name and returns the id if a single entry only
-    # @param entity_type path of entuty in API
+    # @param entity_type path of entity in API
     # @param entity_name name of searched entity
     # @param options additional search options
     def lookup_entity_by_name(entity_type, entity_name, options={})
@@ -295,11 +309,11 @@ module Aspera
       else
         # multiple case insensitive partial matches, try case insensitive full match
         # (anyway AoC does not allow creation of 2 entities with same case insensitive name)
-        icase_matches = matching_items.select{|i|i['name'].casecmp?(entity_name)}
-        case icase_matches.length
-        when 1 then return icase_matches.first
+        name_matches = matching_items.select{|i|i['name'].casecmp?(entity_name)}
+        case name_matches.length
+        when 1 then return name_matches.first
         when 0 then raise %Q(#{entity_type}: multiple case insensitive partial match for: "#{entity_name}": #{matching_items.map{|i|i['name']}} but no case insensitive full match. Please be more specific or give exact name.) # rubocop:disable Layout/LineLength
-        else raise "Two entities cannot have the same case insensitive name: #{icase_matches.map{|i|i['name']}}"
+        else raise "Two entities cannot have the same case insensitive name: #{name_matches.map{|i|i['name']}}"
         end
       end
     end
@@ -311,9 +325,7 @@ module Aspera
         pkg_data['recipients'].first.is_a?(Hash) &&
         pkg_data['recipients'].first.key?('type') &&
         pkg_data['recipients'].first['type'].eql?('dropbox')
-
-      shbx_kid = pkg_data['recipients'].first['id']
-      meta_schema = read("dropboxes/#{shbx_kid}")[:data]['metadata_schema']
+      meta_schema = read("dropboxes/#{pkg_data['recipients'].first['id']}")[:data]['metadata_schema']
       if meta_schema.nil? || meta_schema.empty?
         Log.log.debug('no metadata in shared inbox')
         return
@@ -344,8 +356,9 @@ module Aspera
     # @return nil package_data is modified
     def resolve_package_recipients(package_data, ws_id, recipient_list_field, new_user_option)
       return unless package_data.key?(recipient_list_field)
-      raise CliBadArgument, "#{recipient_list_field} must be an Array" unless package_data[recipient_list_field].is_a?(Array)
+      raise "#{recipient_list_field} must be an Array" unless package_data[recipient_list_field].is_a?(Array)
       new_user_option = {'package_contact' => true} if new_user_option.nil?
+      raise 'new_user_option must be a Hash' unless new_user_option.is_a?(Hash)
       # list with resolved elements
       resolved_list = []
       package_data[recipient_list_field].each do |short_recipient_info|
@@ -382,6 +395,58 @@ module Aspera
       return nil
     end
 
+    # CLI allows simplified format for metadata: transform if necessary for API
+    def update_package_metadata_for_api(pkg_data)
+      case pkg_data['metadata']
+      when Array, NilClass # no action
+      when Hash
+        api_meta = []
+        pkg_data['metadata'].each do |k, v|
+          api_meta.push({
+            # 'input_type' => 'single-dropdown',
+            'name'   => k,
+            'values' => v.is_a?(Array) ? v : [v]
+          })
+        end
+        pkg_data['metadata'] = api_meta
+      else raise "metadata field if not of expected type: #{pkg_meta.class}"
+      end
+      return nil
+    end
+
+    # create a package
+    # @param package_data [Hash] package creation (with extensions...)
+    # @param validate_meta [TrueClass,FalseClass] true to validate parameters locally
+    # @param new_user_option [Hash] options if an unknown user is specified
+    # @return transfer spec, node api and package information
+    def create_package_simple(package_data, validate_meta, new_user_option)
+      update_package_metadata_for_api(package_data)
+      # list of files to include in package, optional
+      # package_data['file_names']||=[..list of filenames to transfer...]
+
+      # lookup users
+      resolve_package_recipients(package_data, package_data['workspace_id'], 'recipients', new_user_option)
+      resolve_package_recipients(package_data, package_data['workspace_id'], 'bcc_recipients', new_user_option)
+
+      validate_metadata(package_data) if validate_meta
+
+      #  create a new package container
+      created_package = create('packages', package_data)[:data]
+
+      package_node_api = node_api_from(package_info: created_package, scope: AoC::SCOPE_NODE_USER)
+
+      # tell AoC what to expect in package: 1 transfer (can also be done after transfer)
+      # TODO: if multi session was used we should probably tell
+      # also, currently no "multi-source" , i.e. only from client-side files, unless "node" agent is used
+      update("packages/#{created_package['id']}", {'sent' => true, 'transfers_expected' => 1})[:data]
+
+      return {
+        spec: package_node_api.transfer_spec_gen4(created_package['contents_file_id'], Fasp::TransferSpec::DIRECTION_SEND),
+        node: package_node_api,
+        info: created_package
+      }
+    end
+
     # Add transferspec
     # callback in Aspera::Node (transfer_spec_gen4)
     def add_ts_tags(transfer_spec:, app_info:)
@@ -389,7 +454,7 @@ module Aspera
       transfer_type = Fasp::TransferSpec.action(transfer_spec)
       # Analytics tags
       ################
-      ws_info = app_info[:plugin].current_workspace_info
+      ws_info = app_info[:workspace_info]
       transfer_spec.deep_merge!({
         'tags' => {
           'aspera' => {
@@ -405,7 +470,7 @@ module Aspera
       # Console cookie
       ################
       # we are sure that fields are not nil
-      cookie_elements = [app_info[:app], user_info['name'], user_info['email']].map{|e|Base64.strict_encode64(e)}
+      cookie_elements = [app_info[:app], current_user_info['name'], current_user_info['email']].map{|e|Base64.strict_encode64(e)}
       cookie_elements.unshift(COOKIE_PREFIX_CONSOLE_AOC)
       transfer_spec['cookie'] = cookie_elements.join(':')
       # Application tags
@@ -433,7 +498,7 @@ module Aspera
     # Callback from Plugins::Node
     def permissions_create_params(create_param:, app_info:)
       # workspace shared folder:
-      # access_id = "#{ID_AK_ADMIN}_WS_#{ app_info[:plugin].current_workspace_info['id']}"
+      # access_id = "#{ID_AK_ADMIN}_WS_#{ app_info[:workspace_info]['id']}"
       default_params = {
         # 'access_type'   => 'user', # mandatory: user or group
         # 'access_id'     => access_id, # id of user or group
@@ -441,12 +506,12 @@ module Aspera
           'aspera' => {
             'files' => {
               'workspace' => {
-                'id'                => app_info[:plugin].current_workspace_info['id'],
-                'workspace_name'    => app_info[:plugin].current_workspace_info['name'],
-                'user_name'         => user_info['name'],
-                'shared_by_user_id' => user_info['id'],
-                'shared_by_name'    => user_info['name'],
-                'shared_by_email'   => user_info['email'],
+                'id'                => app_info[:workspace_info]['id'],
+                'workspace_name'    => app_info[:workspace_info]['name'],
+                'user_name'         => current_user_info['name'],
+                'shared_by_user_id' => current_user_info['id'],
+                'shared_by_name'    => current_user_info['name'],
+                'shared_by_email'   => current_user_info['email'],
                 # 'shared_with_name'  => access_id,
                 'access_key'        => app_info[:node_info]['access_key'],
                 'node'              => app_info[:node_info]['name']}}}}}
@@ -455,13 +520,13 @@ module Aspera
         contact_info = lookup_entity_by_name(
           'contacts',
           create_param['with'],
-          {'current_workspace_id' => app_info[:plugin].current_workspace_info['id'], 'context' => 'share_folder'})
+          {'current_workspace_id' => app_info[:workspace_info]['id'], 'context' => 'share_folder'})
         create_param.delete('with')
         create_param['access_type'] = contact_info['source_type']
         create_param['access_id'] = contact_info['source_id']
         create_param['tags']['aspera']['files']['workspace']['shared_with_name'] = contact_info['email']
       end
-      # optionnal
+      # optional
       app_info[:opt_link_name] = create_param.delete('link_name')
     end
 
@@ -470,7 +535,7 @@ module Aspera
       event_creation = {
         'types'        => ['permission.created'],
         'node_id'      => app_info[:node_info]['id'],
-        'workspace_id' => app_info[:plugin].current_workspace_info['id'],
+        'workspace_id' => app_info[:workspace_info]['id'],
         'data'         => created_data # Response from previous step
       }
       # (optional). The name of the folder to be displayed to the destination user. Use it if its value is different from the "share_as" field.

data/lib/aspera/cli/extended_value.rb

@@ -19,7 +19,7 @@ module Aspera
         # decode comma separated table text
         def decode_csvt(value)
           col_titles = nil
-          hasharray = []
+          hash_array = []
           CSV.parse(value).each do |values|
             next if values.empty?
             if col_titles.nil?
@@ -27,11 +27,11 @@ module Aspera
             else
               entry = {}
               col_titles.each{|title|entry[title] = values.shift}
-              hasharray.push(entry)
+              hash_array.push(entry)
             end
           end
-          Log.log.warn('Titled CSV file without any line') if hasharray.empty?
-          return hasharray
+          Log.log.warn('Titled CSV file without any line') if hash_array.empty?
+          return hash_array
         end
       end
 

data/lib/aspera/cli/{formater.rb → formatter.rb}

@@ -8,7 +8,7 @@ require 'pp'
 module Aspera
   module Cli
     # Take care of output
-    class Formater
+    class Formatter
       FIELDS_ALL = 'ALL'
       FIELDS_DEFAULT = 'DEF'
       CSV_RECORD_SEPARATOR = "\n"
@@ -36,11 +36,11 @@ module Aspera
           end
         end
 
-        def flatten_config_overview(conffile)
+        def flatten_config_overview(hash_array_conf)
           r = []
-          conffile.each do |config, preset|
+          hash_array_conf.each do |config, preset|
             preset.each do |parameter, value|
-              r.push(CONF_OVERVIEW_KEYS.zip([config, parameter, value]).to_h)
+              r.push(CONF_OVERVIEW_KEYS.zip([config, parameter, SecretHider.deep_remove_secret(value).to_s]).to_h)
             end
           end
           return r
@@ -52,7 +52,7 @@ module Aspera
 
         # recursive function to modify a hash
         # @param source [Hash] to be modified
-        # @param expand_last [bool] truer if last level is not
+        # @param expand_last [TrueClass,FalseClass] true if last level is not
         # @param result [Hash] new hash flattened
         # @param prefix [String] true if last level is not
         def flattened_object(source, result: {}, prefix: '', expand_last: false)
@@ -131,7 +131,7 @@ module Aspera
 
       def result_all_fields(_results, table_rows_hash_val)
         raise 'internal error: must be array' unless table_rows_hash_val.is_a?(Array)
-        # get the list of all column names used in all lines, not just frst one, as all lines may have different columns
+        # get the list of all column names used in all lines, not just first one, as all lines may have different columns
         return table_rows_hash_val.each_with_object({}){|v, m|v.each_key{|c|m[c] = true}; }.keys
       end
 
@@ -141,7 +141,7 @@ module Aspera
         raise 'INTERNAL ERROR, result must have type' unless results.key?(:type)
         raise 'INTERNAL ERROR, result must have data' unless results.key?(:data) || %i[empty nothing].include?(results[:type])
         res_data = results[:data]
-        # for config overvuew, it is name and value
+        # for config overview, it is name and value
         is_config_overview = res_data.is_a?(Array) && !res_data.empty? && res_data.first.is_a?(Hash) && res_data.first.keys.sort.eql?(CONF_OVERVIEW_KEYS)
         SecretHider.deep_remove_secret(res_data, is_name_value: is_config_overview) unless @option_show_secrets || @option_display.eql?(:data)
         # comma separated list in string format

data/lib/aspera/cli/listener/progress.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
 require 'aspera/fasp/listener'
-require 'ruby-progressbar'
 require 'aspera/environment'
+require 'ruby-progressbar'
 
 module Aspera
   module Cli

data/lib/aspera/cli/listener/progress_multi.rb

@@ -45,8 +45,8 @@ module Aspera
             Log.log.error{"Internal error: no #{Fasp::AgentBase::LISTENER_SESSION_ID_S} in event: #{data}"}
             return
           end
-          newtitle = @sessions.length < 2 ? '' : "multi=#{@sessions.length}"
-          @progress_bar.title = newtitle unless @progress_bar.title.eql?(newtitle)
+          new_title = @sessions.length < 2 ? '' : "multi=#{@sessions.length}"
+          @progress_bar.title = new_title unless @progress_bar.title.eql?(new_title)
           session = @sessions[data[Fasp::AgentBase::LISTENER_SESSION_ID_S]] ||= {
             cumulative: 0,
             job_size:   0,