connect-sdk-ruby 5.0.0 → 5.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8f1cb58509ee13a6f901b29e5da28837958cfcb1d574b3ebee1b908d654a89bb
-  data.tar.gz: d063c020e3995486220c4d72303905b98b0b6ef3c5eb6cbed9392d86798f6628
+  metadata.gz: 58cbad6e878461552ac4206cf9951c00a9dcaf15224b58b75e8296217f6aba0c
+  data.tar.gz: 5669d32653bf1f26c277c873e60452bb4862e7e52cd0a94777bd1e9a45e0a06c
 SHA512:
-  metadata.gz: c3d49a2225746da3328bc7f544510d3f652981ae3606a46a94102ed6a51577ce7ba0abc7429e43dc9654fcbd08c52f1f9180ba85b595a178df95cc90f3297d7d
-  data.tar.gz: 8243a73cc4b957b860a04863e28f8093ae252e8ab029216d86ce4458c940184dc66a1239a9cd99eb57888a4d0bb8d4a4ab2902bcb1fe3c509be7a1685f08531e
+  metadata.gz: 0ffbd46cde3450c1b117ecadfbf6f63ca22a693b1d6f01f08b0c5ecf5a28a7e1de06284306f5e1bd9266be7734e3ed090d029bddee5091ca6ee18c5d0cf78d32
+  data.tar.gz: 6a98c7656ea0a68938d7b1490cb956ae892939f7507a90da316802fbb1ec47026b07f1b85dec58ce339cae2cf1554f8b724196904824b93ed3fb46c63b49b226

data/connect-sdk-ruby.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name           = 'connect-sdk-ruby'
-  spec.version        = '5.0.0'
+  spec.version        = '5.0.1'
   spec.authors        = ['Worldline Global Collect']
   spec.email          = ['github.connect@worldline.com']
   spec.summary        = %q{SDK to communicate with the Worldline Global Collect platform using the Worldline Connect Server API}

data/lib/worldline/connect/sdk/communication/metadata_provider.rb

@@ -14,7 +14,7 @@ module Worldline
         class MetadataProvider
           private
 
-          SDK_VERSION = '5.0.0'.freeze
+          SDK_VERSION = '5.0.1'.freeze
           SERVER_META_INFO_HEADER = 'X-GCS-ServerMetaInfo'.freeze
           PROHIBITED_HEADERS = [SERVER_META_INFO_HEADER, 'X-GCS-Idempotence-Key','Date', 'Content-Type', 'Authorization'].sort!.freeze
           CHARSET = 'utf-8'.freeze

data/lib/worldline/connect/sdk/communication/response_exception.rb

@@ -15,12 +15,9 @@ module Worldline
             super('the Worldline Global Collect platform returned an error response')
             @status_code = status_code
             @headers = if headers.nil? or headers.empty?
-                         {}
+                         []
                        else
-                         headers.inject({}) do |hash, header|
-                           hash[header.name.downcase.to_sym] = header.dup.freeze
-                           hash
-                         end
+                         headers.map { |h| h.dup.freeze }
                        end.freeze
             @body = body
           end

data/lib/worldline/connect/sdk/communication/response_header.rb

@@ -51,8 +51,8 @@ module Worldline
           # If this Response does not contain a header with the given name, return _nil_ instead
           def self.get_disposition_filename(headers)
             header_value = get_header_value(headers, "Content-Disposition")
-            if !header_value.nil? && header_value =~ /(?:^|;)\s*filename\s*=\s*(.*?)\s*(?:;|$)i/
-              return trim_quotes($2)
+            if !header_value.nil? && header_value =~ /(?:^|;)\s*filename\s*=\s*(.*?)\s*(?:;|$)/i
+              return trim_quotes($1)
             end
 
             return nil
@@ -62,9 +62,9 @@ module Worldline
 
           # Trims the single or double quotes at the beginning and end of parameter _filename_ if they exist
           # If they don't exist, it returns the original _filename_ instead
-          def trim_quotes(filename)
+          def self.trim_quotes(filename)
             if filename.length >= 2 &&
-               (filename.chars.first == '\'' && filename.chars.last == '\'') || (filename.chars.first == '"' && filename.chars.last == '"')
+               ((filename.chars.first == '\'' && filename.chars.last == '\'') || (filename.chars.first == '"' && filename.chars.last == '"'))
               return filename[1...-1]
             end
             filename

data/lib/worldline/connect/sdk/communicator.rb

@@ -41,12 +41,12 @@ module Worldline
           raise ArgumentError.new('connection is required') unless connection
           raise ArgumentError.new('authenticator is required') unless authenticator
           raise ArgumentError.new('metadata_provider is required') unless metadata_provider
-          raise ArgumentError('marshaller is required') if marshaller.nil?
+          raise ArgumentError.new('marshaller is required') if marshaller.nil?
 
           uri = URI(api_endpoint)
-          raise RuntimeError('api_endpoint should not contain a path') unless uri.path.nil? || uri.path.empty?
+          raise RuntimeError.new('api_endpoint should not contain a path') unless uri.path.nil? || uri.path.empty?
           unless uri.userinfo.nil? && uri.query.nil? && uri.fragment.nil?
-            raise RuntimeError('api_endpoint should not contain user info, query or fragment')
+            raise RuntimeError.new('api_endpoint should not contain user info, query or fragment')
           end
           @api_endpoint = uri
           @connection = connection
@@ -483,7 +483,7 @@ module Worldline
 
         def throw_exception_if_necessary(body, status_code, headers, request_path)
           if status_code < 200 || status_code >= 300
-            if !body.nil? && !is_json(headers)
+            if !body.nil? && !body.empty? && !is_json(headers)
               cause = Communication::ResponseException.new(status_code, headers, body)
               if status_code == 404
                 raise Communication::NotFoundException.new(cause, 'The requested resource was not found; invalid path: ' +

data/lib/worldline/connect/sdk/domain/uploadable_file.rb

@@ -14,9 +14,9 @@ module Worldline
         class UploadableFile
 
           def initialize(file_name, content, content_type, content_length = -1)
-            raise ArgumentError.new("file_name is required") if file_name.nil? or !file_name.strip
+            raise ArgumentError.new("file_name is required") if file_name.nil? or file_name.strip.empty?
             raise ArgumentError.new("content is required") if content.nil?
-            raise ArgumentError.new("content_type is required") if content_type.nil? or !content_type.strip
+            raise ArgumentError.new("content_type is required") if content_type.nil? or content_type.strip.empty?
 
             @file_name = file_name
             @content = content

data/lib/worldline/connect/sdk/json/default_marshaller.rb

@@ -13,15 +13,14 @@ module Worldline
 
           # Marshals the _request_object_ to a JSON string using request_object#to_h
           def marshal(request_object)
+            return 'null' if request_object.nil?
             ::JSON.pretty_generate(request_object.to_h)
           end
 
           # Unmarshals a JSON string into an object of type _klass_ using klass.new_from_hash
           def unmarshal(json_string, klass)
-            if json_string.nil?
+            if json_string.nil? or json_string.empty?
               return nil
-            elsif json_string.length == 0
-              return ''
             end
             if klass.respond_to?(:new_from_hash)
               klass.new_from_hash(::JSON.load(json_string))

data/lib/worldline/connect/sdk/logging/request_log_message_builder.rb

@@ -37,13 +37,12 @@ module Worldline
           private
 
           def format_uri
-            '' unless @uri && @uri.path
+            return '' unless @uri && @uri.path
             if @uri.query.nil?
               @uri.path
             else
-              "#{@uri.path}?#{@uri.query}" unless @uri.query.nil?
+              "#{@uri.path}?#{@uri.query}"
             end
-            # @uri.path + '?' + empty_if_null(@uri.query)
           end
         end
       end

data/lib/worldline/connect/sdk/v1/exception_factory.rb

@@ -34,6 +34,7 @@ module Worldline
             return DeclinedRefundException.new(status_code, response_body, error_object) unless error_object.refund_result.nil?
             return create_exception_from_response_fields(status_code, response_body, error_object.error_id, error_object.errors, context)
           end
+          return create_exception_from_response_fields(status_code, response_body, nil, [], context) if error_object.nil?
           raise ArgumentError.new("unsupported error object type: " + error_object.class.name) unless error_object.is_a?(Worldline::Connect::SDK::V1::Domain::ErrorResponse)
           create_exception_from_response_fields(status_code, response_body, error_object.error_id, error_object.errors, context)
         end

data/spec/lib/communication/default_connection_logger_spec.rb

@@ -303,7 +303,7 @@ describe 'DefaultConnectionLogging' do
     CLIENT.enable_logging(logger)
     response = CLIENT.v1.merchant('1234').tokens.delete('5678', nil)
 
-    expect(response).to_not be_nil
+    expect(response).to be_nil
 
     validate_request_and_response(logger.entries[0], logger.entries[1], 'deleteToken')
   end

data/spec/lib/communication/response_header_spec.rb

@@ -0,0 +1,47 @@
+require 'spec_helper'
+
+describe 'ResponseHeader' do
+  ResponseHeader ||= Worldline::Connect::SDK::Communication::ResponseHeader
+
+  test_data = {
+    "attachment; filename=testfile" => "testfile",
+    "attachment; filename=\"testfile\"" => "testfile",
+    "attachment; filename=\"testfile" => "\"testfile",
+    "attachment; filename=testfile\"" => "testfile\"",
+    "attachment; filename='testfile'" => "testfile",
+    "attachment; filename='testfile" => "'testfile",
+    "attachment; filename=testfile'" => "testfile'",
+
+    "filename=testfile" => "testfile",
+    "filename=\"testfile\"" => "testfile",
+    "filename=\"testfile" => "\"testfile",
+    "filename=testfile\"" => "testfile\"",
+    "filename='testfile'" => "testfile",
+    "filename='testfile" => "'testfile",
+    "filename=testfile'" => "testfile'",
+
+    "attachment; filename=testfile; x=y" => "testfile",
+    "attachment; filename=\"testfile\"; x=y" => "testfile",
+    "attachment; filename=\"testfile; x=y" => "\"testfile",
+    "attachment; filename=testfile\"; x=y" => "testfile\"",
+    "attachment; filename='testfile'; x=y" => "testfile",
+    "attachment; filename='testfile; x=y" => "'testfile",
+    "attachment; filename=testfile'; x=y" => "testfile'",
+
+    "attachment" => nil,
+
+    "filename=\"" => "\"",
+    "filename='" => "'",
+  }
+
+  it 'extracts the correct disposition filename from an array of ResponseHeader' do
+    test_data.each do |value, expected|
+      headers = [ResponseHeader.new('Content-Disposition', value)]
+      filename = ResponseHeader.get_disposition_filename(headers)
+      expect(filename).to eq(expected)
+    end
+    headers = []
+    filename = ResponseHeader.get_disposition_filename(headers)
+    expect(filename).to be_nil
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: connect-sdk-ruby
 version: !ruby/object:Gem::Version
-  version: 5.0.0
+  version: 5.0.1
 platform: ruby
 authors:
 - Worldline Global Collect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-06-03 00:00:00.000000000 Z
+date: 2026-06-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httpclient
@@ -781,6 +781,7 @@ files:
 - spec/lib/communication/default_connection_idempotence_spec.rb
 - spec/lib/communication/default_connection_logger_spec.rb
 - spec/lib/communication/default_connection_spec.rb
+- spec/lib/communication/response_header_spec.rb
 - spec/lib/communicator_configuration_spec.rb
 - spec/lib/communicator_spec.rb
 - spec/lib/factory_spec.rb
@@ -872,6 +873,7 @@ test_files:
 - spec/lib/communication/default_connection_idempotence_spec.rb
 - spec/lib/communication/default_connection_logger_spec.rb
 - spec/lib/communication/default_connection_spec.rb
+- spec/lib/communication/response_header_spec.rb
 - spec/lib/communicator_configuration_spec.rb
 - spec/lib/communicator_spec.rb
 - spec/lib/factory_spec.rb