datatrans 4.0.1 → 5.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f0b065ed93895deb3f8b801fe1060246a21e597bf2b5d3373b502ea6ce6f1ead
-  data.tar.gz: 94f5cd63b06ee0af246adf8dd30a16790fde64e68d22ca53cf005852cfaa4263
+  metadata.gz: eb01c19e9afe3aecaa82ff08e7e9024fefb8c60d3cab86fbd3400e4598879dcd
+  data.tar.gz: 13a9c7a7068610907a8123c11cf50f60014f79b5de6db2b51f763a5509159fb3
 SHA512:
-  metadata.gz: 11c4e5483bbcc2d9833348934fc6b54c7af958a6c41d3cae7a183c4614df56c2e09bfd06b23746ade55e4482c5aa3493edd6d5a3c5eb8fa9fd0680f196f101d4
-  data.tar.gz: c286baa29ea0d26186a864fdd6b5a133a36ebb8c780bfe4d1484db83f5c36aa715ce434b247e93baa80964e3e9612522308f81ede13f7954da6c2e953970c93d
+  metadata.gz: b8d2cc670be21be5336b8856eca9603ef8351be8ebc9e8e19efca2486a2304d7a8701fcabeea629285434dcd4b6c99cf8f53b8382870cbd0a7dbf38db7b6ded2
+  data.tar.gz: a13b4a7a1934bfa437209b9eb53b2c8bba45d3e0c7e24cb699e8544c7d2af67d6497a0a1c885e3d6af034030b3b0b077d5928a4e8e6577a2b606824c1752be22

data/CHANGELOG.md

@@ -5,6 +5,14 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 
 ## [Unreleased]
 
+## 5.0.0 - 2022-09-21
+
+### Changed
+
+* [BREAKING CHANGE] Authenticate requests with HTTP Basic Auth (@crackofdusk [#41](https://github.com/simplificator/datatrans/pull/41))
+
+  Datatrans requires [HTTP Basic Auth for XML API calls](https://api-reference.datatrans.ch/xml/#authentication-tls) since 2022-09-14 ([announcement](https://mailchi.mp/datatrans/basic-authsign2_1-email_en))
+
 ## 4.0.1 - 2022-03-18
 ### Fixed
 * Check for successful transaction depends on `status` and not on `response_message` that can be diferent for different payment methods (@schmijos [#8](https://github.com/simplificator/datatrans/pull/8) and @TatianaPan [#39](https://github.com/simplificator/datatrans/pull/39))

data/README.markdown

@@ -11,6 +11,7 @@ Build your Datatrans Configuration like so:
     datatrans = Datatrans::Config.new(
       :merchant_id => '1234567',
       :sign_key => 'ab739fd5b7c2a1...',
+      :password => 'server to server request password',
       :environment => :production,
       :proxy => {
         :http_proxyaddr => "proxy.com",

data/lib/datatrans/config.rb

@@ -11,10 +11,11 @@ module Datatrans
     }
     DOMAIN = 'datatrans.com'
 
-    attr_reader :environment, :merchant_id, :sign_key, :proxy
+    attr_reader :environment, :merchant_id, :sign_key, :password, :proxy
 
     # Configure with following options
     # * :merchant_id (required)
+    # * :password (required)
     # * :sign_key (defaults to false)
     # * :environment (defaults to :development, available environments are defined in ENVIRONMENTS)
     # * :proxy (a hash containing :http_proxyaddr, :http_proxyport, :http_proxyuser, :http_proxypass)
@@ -22,6 +23,8 @@ module Datatrans
       @merchant_id = options[:merchant_id]
       raise ArgumentError.new(":merchant_id is required") unless self.merchant_id
       self.environment = options[:environment] || DEFAULT_ENVIRONMENT
+      @password = options[:password]
+      raise ArgumentError.new(":password is required") unless self.password
       @sign_key = options[:sign_key] || DEFAULT_SIGN_KEY
       @proxy = options[:proxy] || {}
     end

data/lib/datatrans/version.rb

@@ -1,3 +1,3 @@
 module Datatrans
-  VERSION = "4.0.1"
+  VERSION = "5.0.0"
 end

data/lib/datatrans/xml/transaction/request.rb

@@ -7,7 +7,9 @@ class Datatrans::XML::Transaction
     attr_accessor :params, :datatrans
 
     def post(url, options = {})
-      options = options.merge(self.datatrans.proxy)
+      options = options
+        .merge(self.datatrans.proxy)
+        .merge(:basic_auth => { :username => self.datatrans.merchant_id, :password => self.datatrans.password })
       HTTParty.post(url, **options)
     end
 
@@ -39,4 +41,4 @@ class Datatrans::XML::Transaction
       xml.target!
     end
   end
-end
+end

data/spec/config_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 describe Datatrans::Config do
   describe "Instance Methods" do
     before do
-      @datatrans = Datatrans::Config.new(:merchant_id => "xxx")
+      @datatrans = Datatrans::Config.new(:merchant_id => "xxx", :password => "yyy")
     end
 
     describe "web_transaction" do

data/spec/spec_helper.rb

@@ -15,6 +15,7 @@ RSpec.configure do |config|
     @datatrans = Datatrans::Config.new(
       :merchant_id => '1100000000',
       :sign_key => 'd777c17ba2010282c2d2350a68b441ca07a799d294bfaa630b7c8442207c0b69703cc55775b0ca5a4e455b818a9bb10a43669c0c20ce31f4a43f10e0cabb9525',
+      :password => 'basic_auth_password',
       :environment => :development
     )
   end

data/spec/xml/request_spec.rb

@@ -7,6 +7,7 @@ describe Datatrans::XML::Transaction::Request do
         @datatrans = Datatrans::Config.new(
           :merchant_id => '1100000000',
           :sign_key => 'd777c17ba2010282c2d2350a68b441ca07a799d294bfaa630b7c8442207c0b69703cc55775b0ca5a4e455b818a9bb10a43669c0c20ce31f4a43f10e0cabb9525',
+          :password => 'basic_auth_password',
           :key => "value",
           :proxy => {
             :http_proxyaddr => "proxy.com",
@@ -21,6 +22,7 @@ describe Datatrans::XML::Transaction::Request do
       it "forward those options to HTTParty" do
         request = Datatrans::XML::Transaction::Request.new(@datatrans, {})
         expect(HTTParty).to receive(:post).with('lirum',
+         :basic_auth => {:password => 'basic_auth_password', :username => '1100000000'},
          :params => {:foo => :bar},
            :http_proxpass => 'xxx',
            :http_proxyuser => 'hans',
@@ -33,7 +35,7 @@ describe Datatrans::XML::Transaction::Request do
     describe "not configured" do
       it "should not add any proxy settings" do
         request = Datatrans::XML::Transaction::Request.new(@datatrans, {})
-        expect(HTTParty).to receive(:post).with('lirum', :params => {:foo => :bar})
+        expect(HTTParty).to receive(:post).with('lirum', :basic_auth => {:password => 'basic_auth_password', :username => '1100000000'}, :params => {:foo => :bar})
         request.post('lirum', :params => {:foo => :bar})
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: datatrans
 version: !ruby/object:Gem::Version
-  version: 4.0.1
+  version: 5.0.0
 platform: ruby
 authors:
 - Tobias Miesel
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-03-18 00:00:00.000000000 Z
+date: 2022-09-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty