exact4r 1.4 → 1.5

data/CHANGELOG

@@ -1,3 +1,6 @@
+== v1.5
+	Added support for specifying authorization_num & reference_no in CR requests, to obtain decrypted CC numbers.
+
 == v1.4
 	Added support for specifying alternative SSL certificates
 

data/VERSION

@@ -1 +1 @@
-1.4
+1.5

data/after.log

@@ -0,0 +1,17 @@
+(in /Users/donch/Development/Exact/WebService/exact4r)
+Loaded suite /usr/local/rvm/gems/ruby-1.8.7-p249@global/gems/rake-0.8.7/lib/rake/rake_test_loader
+Started
+....................................................................................................................................................F..............
+Finished in 579.15838 seconds.
+
+  1) Failure:
+test_encodes_parameters(TransactionDetailsTest)
+    [./test/exhaustive/transaction_details_test.rb:93:in `test_encodes_parameters'
+     ./test/exhaustive/transaction_details_test.rb:91:in `each'
+     ./test/exhaustive/transaction_details_test.rb:91:in `test_encodes_parameters'
+     /usr/local/rvm/gems/ruby-1.8.7-p249@rpm/gems/mocha-0.9.8/lib/mocha/integration/test_unit/ruby_version_186_and_above.rb:19:in `__send__'
+     /usr/local/rvm/gems/ruby-1.8.7-p249@rpm/gems/mocha-0.9.8/lib/mocha/integration/test_unit/ruby_version_186_and_above.rb:19:in `run']:
+<"4111111111111111"> expected but was
+<nil>.
+
+163 tests, 2705 assertions, 1 failures, 0 errors

data/lib/ews/transporter.rb

@@ -1,4 +1,5 @@
 require 'net/https'
+require 'uri'
 require File.dirname(__FILE__) + '/certificate_helper'
 
 module EWS # :nodoc:
@@ -55,7 +56,7 @@ module EWS # :nodoc:
       request = build_http_request(transaction, transport_type, transport_details[:suffix])
       request.basic_auth(transaction.gateway_id, transaction.password)
       request.add_field "Accept", transport_details[:content_type]
-      request.add_field "User-Agent", "exact4r v1.4"
+      request.add_field "User-Agent", "exact4r v1.5"
       request.add_field "Content-type", "#{transport_details[:content_type]}; charset=UTF-8"
 
       response = get_connection.request(request)
@@ -94,7 +95,14 @@ private
         end
         req.body = EWS::Transaction::Mapping.send "request_to_#{transport_type.to_s}", transaction
       else
-        req = Net::HTTP::Get.new(@url.path + "/transaction/#{transaction.transaction_tag}.#{request_suffix}")
+        param_str = ""
+        escaping_regex = Regexp.new("[^#{URI::PATTERN::UNRESERVED}]")
+        params = [:authorization_num, :reference_no].collect do |attr_name|
+          value = transaction.send(attr_name)
+          value.blank? ? nil : "#{attr_name}=" + URI.escape(value, escaping_regex)
+        end.compact
+        param_str = "?"+params.join('&') unless params.empty?
+        req = Net::HTTP::Get.new(@url.path + "/transaction/#{transaction.transaction_tag}.#{request_suffix}"+param_str)
       end
       req
     end

data/test/exhaustive/referenced_void_test.rb

@@ -14,8 +14,8 @@ class ReferencedVoidTest < Test::Unit::TestCase
     request = EWS::Transaction::Request.new(cc_number_params.merge({
       :transaction_type => :purchase,
       :amount => 11.25,
-      :reference_no => "reference_no_#{rand(10)}",
-      :customer_ref => "customer_ref_#{rand(10)}"
+      :reference_no => "reference_no_#{rand(1000)}",
+      :customer_ref => "customer_ref_#{rand(1000)}"
     }))
     assert request.valid?, request.errors.inspect
     

data/test/exhaustive/transaction_details_test.rb

@@ -38,9 +38,63 @@ class TransactionDetailsTest < Test::Unit::TestCase
     }.merge(@@credentials.current_gateway))
     assert request.valid?, request.errors.inspect
     
-    assert_details_match_original_response pre_response, @transporter.submit(request, :json)
-    assert_details_match_original_response pre_response, @transporter.submit(request, :rest)
-    assert_details_match_original_response pre_response, @transporter.submit(request, :soap)
+    [:json, :rest, :soap].each do |encoding|
+      response = @transporter.submit(request, encoding)
+      assert_equal "############1111", response.cc_number
+      assert_details_match_original_response pre_response, response, encoding
+    end
+  end
+
+  def test_supplying_auth_num_decrypts_cc_number
+    # do initial purchase
+    pre_request = EWS::Transaction::Request.new(cc_number_params.merge(:transaction_type => :purchase))
+    pre_request.amount = 10.1
+    assert pre_request.valid?, pre_request.errors.inspect
+    
+    pre_response = @transporter.submit(pre_request, :json)
+    assert pre_response.approved?
+    
+    request = EWS::Transaction::Request.new({
+      :transaction_type => :transaction_details,
+      :transaction_tag => pre_response.transaction_tag,
+      :authorization_num => pre_response.authorization_num
+    }.merge(@@credentials.current_gateway))
+    assert request.valid?, request.errors.inspect
+    
+    [:json, :rest, :soap].each do |encoding|
+      response = @transporter.submit(request, encoding)
+      assert_equal "4111111111111111", response.cc_number
+      assert_details_match_original_response pre_response, response, encoding
+    end
+  end
+  
+  def test_encodes_parameters
+    # do initial purchase
+    pre_request = EWS::Transaction::Request.new(cc_number_params.merge({
+     :transaction_type => :purchase,
+     :reference_no => "barry&=x=jones"
+    }))
+    pre_request.amount = 10.1
+    assert pre_request.valid?, pre_request.errors.inspect
+    
+    pre_response = @transporter.submit(pre_request, :json)
+    assert pre_response.approved?
+    
+    request = EWS::Transaction::Request.new({
+      :transaction_type => :transaction_details,
+      :transaction_tag => pre_response.transaction_tag,
+      :authorization_num => pre_response.authorization_num,
+      :reference_no => "barry&=x=jones"
+    }.merge(@@credentials.current_gateway))
+    assert request.valid?, request.errors.inspect
+    
+    [:json, :rest, :soap].each do |encoding|
+      response = @transporter.submit(request, encoding)
+      assert_not_nil response, "should not be nil for #{encoding}"
+      assert response.approved?, "#{encoding}: #{response.error_number} / #{response.error_description} / #{response.exact_message}" 
+      assert_equal "4111111111111111", response.cc_number, "should match for #{encoding}"
+      assert_details_match_original_response pre_response, response, encoding
+    end
   end
   
   def test_debit_transaction_details
@@ -64,17 +118,19 @@ class TransactionDetailsTest < Test::Unit::TestCase
     }.merge(@@credentials.current_gateway))
     assert request.valid?, request.errors.inspect
     
-    assert_details_match_original_response pre_response, @transporter.submit(request, :json)
-    assert_details_match_original_response pre_response, @transporter.submit(request, :rest)
-    assert_details_match_original_response pre_response, @transporter.submit(request, :soap)
+    [:json, :rest, :soap].each do |encoding|
+      response = @transporter.submit(request, encoding)
+      assert_equal "############1111", response.cc_number
+      assert_details_match_original_response pre_response, response, encoding
+    end
   end
   
-  def assert_details_match_original_response(original_response, details_response)
-    # exclude: client_ip, pan
+  def assert_details_match_original_response(original_response, details_response, encoding)
+    # exclude: client_ip, pan, cc_number
     [:logon_message, :error_number, :error_description, :transaction_error, :transaction_approved, :exact_resp_code, :exact_message,
       :bank_resp_code, :bank_message, :bank_resp_code_2, :sequence_no, :avs, :cvv2, :retrieval_ref_no, :cavv_response, :merchant_name,
       :merchant_address, :merchant_city, :merchant_province, :merchant_country, :merchant_postal, :merchant_url, :gateway_id, :password,
-      :transaction_type, :amount, :surcharge_amount, :cc_number, :transaction_tag, :track1, :track2, :authorization_num, :cc_expiry,
+      :transaction_type, :amount, :surcharge_amount, :transaction_tag, :track1, :track2, :authorization_num, :cc_expiry,
       :cardholder_name, :cc_verification_str1, :cc_verification_str2, :cvd_presence_ind, :tax1_amount, :tax1_number, :tax2_amount,
       :tax2_number, :secure_auth_required, :secure_auth_result, :ecommerce_flag, :xid, :cavv, :cavv_algorithm, :reference_no, :customer_ref,
       :reference_3, :language, :client_email, :user_name, :zip_code].each do |attr_name|
@@ -82,7 +138,7 @@ class TransactionDetailsTest < Test::Unit::TestCase
         
       o_value = original_response.send(attr_name).to_s
       d_value = details_response.send(attr_name).to_s
-      assert_equal o_value, d_value, "#{attr_name}: #{o_value} / #{d_value}"
+      assert_equal o_value, d_value, "Encoding: #{encoding} - #{attr_name}: #{o_value} / #{d_value}"
     end
   end
   private :assert_details_match_original_response

metadata

@@ -4,8 +4,8 @@ version: !ruby/object:Gem::Version
   prerelease: false
   segments: 
   - 1
-  - 4
-  version: "1.4"
+  - 5
+  version: "1.5"
 platform: ruby
 authors: 
 - E-xact Transactions Ltd.
@@ -13,7 +13,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-07-09 00:00:00 +10:00
+date: 2010-11-02 00:00:00 +11:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -56,6 +56,7 @@ extra_rdoc_files:
 - README
 - VERSION
 files: 
+- ./after.log
 - ./certs/e-xact.com.crt
 - ./certs/equifax_ca.cer
 - ./certs/exact.cer
@@ -71,7 +72,6 @@ files:
 - ./lib/ews/transporter.rb
 - ./lib/exact4r.rb
 - ./LICENCE
-- ./pkg/exact4r-1.2.gem
 - ./Rakefile
 - ./README
 - ./test/credentials.rb