mangopay 3.23.0 → 3.24.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3fe8c22cc128558db9e859283f7e723af2e7bc54e1b2b98cb82afc23f03de814
-  data.tar.gz: 2747bafde96b9a150cdd9505b2348da658d098746ec5fc1d6e5ffd0eafd4637c
+  metadata.gz: c227169098718116d4d2473a9e9f5373c5556a89a1bc851b304093fd6e8ff511
+  data.tar.gz: b925a267ad094eb69801beafb04904c95852820837d158298d2c8b49506be8fb
 SHA512:
-  metadata.gz: f530fd2a85bdeb7ef143048d9ce12f12d2e51a6ddbec4254c316235248add77f52ddf50e890906ca1ea4b0ac43720a397a37e37df5dc8051d5ecc4f5769e801a
-  data.tar.gz: 8044741c0cbb7650780e340321c8e5035f2ed817b355d22fb0f265e81fb1f5afd411c0df0fa2c8b5ca061fa898213cf593f3409a3c7e0fc5dba727804d3fdbc3
+  metadata.gz: 88c78792b4aff5d84ebc7c0687cf11b12f974507e09bec368a28101ca169520e3269353536af528669f37305fc0030c414e61754ea1c8f0fb4c7130b09639efd
+  data.tar.gz: c122b794f79e18718d833359d58d0ca6dd4fce57a21195eaae2130a275a0279d138d21939fe02ca754073bfc249f22676212cb7e435553c0cb6a21bfe9edef3a

data/CHANGELOG.md

@@ -1,3 +1,14 @@
+## [3.24.1] - 2024-04-10
+### Fixed
+
+- #249 Improve error handling when logging is enabled
+- #226 MultiJson::ParseError is thrown on HTTP error 5XX
+
+## [3.24.0] - 2024-04-02
+### Added
+
+- New endpoint [Add tracking to Paypal payin](https://mangopay.com/docs/endpoints/paypal#add-tracking-paypal-payin)
+
 ## [3.23.0] - 2024-03-08
 ### Fixed
 

data/lib/mangopay/pay_in.rb

@@ -21,6 +21,7 @@ module MangoPay
       # See http://docs.mangopay.com/api-references/payins/payins-card-web/
       class Web < Resource
         include HTTPCalls::Create
+
         def self.url(*)
           "#{MangoPay.api_path}/payins/card/#{CGI.escape(class_name.downcase)}"
         end
@@ -40,6 +41,7 @@ module MangoPay
       # See http://docs.mangopay.com/api-references/payins/payindirectcard/
       class Direct < Resource
         include HTTPCalls::Create
+
         def self.url(*)
           "#{MangoPay.api_path}/payins/card/#{CGI.escape(class_name.downcase)}"
         end
@@ -52,6 +54,7 @@ module MangoPay
       # See http://docs.mangopay.com/api-references/payins/preauthorized-payin/
       class Direct < Resource
         include HTTPCalls::Create
+
         def self.url(*)
           "#{MangoPay.api_path}/payins/preauthorized/direct"
         end
@@ -68,6 +71,7 @@ module MangoPay
       # See http://docs.mangopay.com/api-references/payins/payinbankwire/
       class Direct < Resource
         include HTTPCalls::Create
+
         def self.url(*)
           "#{MangoPay.api_path}/payins/bankwire/direct"
         end
@@ -83,6 +87,7 @@ module MangoPay
       # See http://docs.mangopay.com/api-references/payins/direct-debit-pay-in-web/
       class Web < Resource
         include HTTPCalls::Create
+
         def self.url(*)
           "#{MangoPay.api_path}/payins/directdebit/#{CGI.escape(class_name.downcase)}"
         end
@@ -91,6 +96,7 @@ module MangoPay
       # See https://docs.mangopay.com/api-references/payins/direct-debit-pay-in-direct/
       class Direct < Resource
         include HTTPCalls::Create
+
         def self.url(*)
           "#{MangoPay.api_path}/payins/directdebit/#{CGI.escape(class_name.downcase)}"
         end
@@ -105,6 +111,7 @@ module MangoPay
       # Please use the 'create_v2' function - MangoPay::PayIn::PayPal::Web.create_new(params)
       class Web < Resource
         include HTTPCalls::Create
+
         def self.url(*)
           "#{MangoPay.api_path}/payins/paypal/#{CGI.escape(class_name.downcase)}"
         end
@@ -112,6 +119,14 @@ module MangoPay
         def self.create_v2(params, idempotency_key = nil)
           MangoPay.request(:post, "#{MangoPay.api_path}/payins/payment-methods/paypal", params, {}, idempotency_key)
         end
+
+        # Add tracking information to a PayPal PayIn (add the tracking number and carrier for LineItems shipments.)
+        # Caution – Tracking information cannot be edited
+        # You can’t modify the TrackingNumber, Carrier, or NotifyBuyer once added.
+        # You can only send a unique tracking number once.
+        def self.add_paypal_tracking_information(pay_in_id, params, idempotency_key = nil)
+          MangoPay.request(:put, "#{MangoPay.api_path}/payins/#{pay_in_id}/trackings", params, {}, idempotency_key)
+        end
       end
 
     end
@@ -120,6 +135,7 @@ module MangoPay
 
       class Web < Resource
         include HTTPCalls::Create
+
         def self.url(*)
           "#{MangoPay.api_path}/payins/payconiq/#{CGI.escape(class_name.downcase)}"
         end

data/lib/mangopay/version.rb

@@ -1,3 +1,3 @@
 module MangoPay
-  VERSION = '3.23.0'
+  VERSION = '3.24.1'
 end

data/lib/mangopay.rb

@@ -214,7 +214,13 @@ module MangoPay
       raise MangoPay::ResponseError.new(uri, '408', {'Message' => 'Request Timeout'}) if res.nil?
 
       # decode json data
-      data = res.body.to_s.empty? ? {} : JSON.load(res.body.to_s)
+      begin
+        data = res.body.to_s.empty? ? {} : JSON.load(res.body.to_s)
+      rescue MultiJson::ParseError
+        details = {}
+        details['Message'] = res.body
+        raise MangoPay::ResponseError.new(uri, res.code, details)
+      end
 
       unless res.is_a?(Net::HTTPOK)
         raise MangoPay::ResponseError.new(uri, res.code, data)
@@ -298,12 +304,17 @@ module MangoPay
         res
       ensure
         line = "#{log_severity(res)} #{line}"
+        if time.nil?
+          time_log = "[Unknown ms]"
+        else
+          time_log = "[#{(time * 1000).round(1)}ms]"
+        end
         if res.nil?
           params = ''
-          line += "\n  [#{(time * 1000).round(1)}ms] 408 Request Timeout #{params}\n"
+          line += "\n  #{time_log} 408 Request Timeout #{params}\n"
         else
           params = FilterParameters.response(res.body)
-          line += "\n  [#{(time * 1000).round(1)}ms] #{res.code} #{params}\n"
+          line += "\n  #{time_log} #{res.code} #{params}\n"
         end
         logger.info { line }
       end

data/spec/mangopay/card_registration_spec.rb

@@ -6,7 +6,6 @@ describe MangoPay::CardRegistration do
     it 'creates a new card registration' do
       created = new_card_registration
       expect(created['Id']).not_to be_nil
-      expect(created['Id'].to_i).to be > 0
       expect(created['AccessKey']).not_to be_nil
       expect(created['PreregistrationData']).not_to be_nil
       expect(created['CardRegistrationURL']).not_to be_nil
@@ -51,7 +50,6 @@ describe MangoPay::CardRegistration do
 
       # card id filled-in...
       expect(card_id).not_to be_nil
-      expect(card_id.to_i).to be > 0
 
       # ...and points to existing (newly created) card
       card = MangoPay::Card.fetch(card_id)

data/spec/mangopay/preauthorization_spec.rb

@@ -7,7 +7,6 @@ describe MangoPay::PreAuthorization do
       cardreg = new_card_registration_completed
       created = new_card_preauthorization
       expect(created['Id']).not_to be_nil
-      expect(created['Id'].to_i).to be > 0
       expect(created['CardId']).to eq(cardreg['CardId'])
       expect(created['AuthorId']).to eq(new_natural_user["Id"])
       expect(created['PayInId']).to be_nil

data/spec/mangopay/user_spec.rb

@@ -144,7 +144,6 @@ describe MangoPay::User do
         fetched = MangoPay::Card.fetch(card['CardId'])
 
         expect(fetched['Id']).not_to be_nil
-        expect(fetched['Id'].to_i).to be > 0
         expect(fetched['UserId']).to eq(new_natural_user["Id"])
         expect(fetched['Currency']).to eq('EUR')
       end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mangopay
 version: !ruby/object:Gem::Version
-  version: 3.23.0
+  version: 3.24.1
 platform: ruby
 authors:
 - Geoffroy Lorieux
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-03-08 00:00:00.000000000 Z
+date: 2024-04-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json