paypalhttp 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6e1d7486774ce1d1a9a52d214a00f3212196d0d1da58b599861056a0367753db
-  data.tar.gz: 18f0971f48707107899ed7cbd397384605b334e761ef33176c9b642742e8bf31
+  metadata.gz: 21343a9b1bd5b019013244665c45fa5a88eb32d6e4735e92ae4d0f61a8902d61
+  data.tar.gz: 7372fafb251adf5c7571dfa9659f161742de0afdcd2eb822cdd10a1525fd3430
 SHA512:
-  metadata.gz: 219e4ffe88eeabfde09731075a2dfd92394568c14d64c0ac4d6f9ac2d2cfe7e24e36353150555fe922077b761e5b01678ae1d3df0435cb10a377bae99d223870
-  data.tar.gz: ed6ee01c152150c9df2d34b47cb1f620271e549c1f8e8953cdbc19eded34d83c7604c97ca6929db8ab9feb972ea03d8221c62758151a615de65b73ac915171e0
+  metadata.gz: ad0f5323b3da8d9650d0fc2d41a68050e80cefd9b37e0a84b0a021c015b40c647aeb127f62291189b1e25d0c104ed9ba495b112435aa087cbad5089548481f35
+  data.tar.gz: d5930250391a51720f17a275e0ba4151389194c095b0381ee2d705c1f6d42003a2d6733ce049f540668662b8a6890584c69fa23f15d8b2d387aca9537bad9429

data/lib/paypalhttp/encoder.rb

@@ -41,6 +41,7 @@ module PayPalHttp
       raise UnsupportedEncodingError.new('HttpResponse did not have Content-Type header set') unless headers && (headers['content-type'])
 
       content_type = _extract_header(headers, 'content-type')
+      content_type.downcase!
 
       enc = _encoder(content_type)
       raise UnsupportedEncodingError.new("Unable to deserialize response with Content-Type #{content_type}. Supported decodings are #{supported_encodings}") unless enc

data/lib/paypalhttp/http_client.rb

@@ -28,7 +28,13 @@ module PayPalHttp
     def format_headers(headers)
       formatted_headers = {}
       headers.each do |key, value|
-        formatted_headers[key.downcase] = value
+        # TODO: Since header is treated as a hash, val is in an array.
+        # Will this cause an issue when accessing and modifying val
+        # Ensure this is the case and will not propegate access issues/errors
+        if key.casecmp("content-type") == 0
+          value[0].downcase!
+        end
+          formatted_headers[key.downcase] = value
       end
       formatted_headers
     end

data/lib/paypalhttp/version.rb

@@ -1 +1 @@
-VERSION = "1.0.0"
+VERSION = "1.0.1"

data/spec/paypalhttp/encoder_spec.rb

@@ -205,6 +205,26 @@ describe Encoder do
       expect(deserialized).to eq(expected)
     end
 
+    it 'deserializes the response when content-type == application/json: case insensitive' do
+      expected = {
+        "string" => "value",
+        "number" => 1.23,
+        "bool" => true,
+        "array" => ["one", "two", "three"],
+        "nested" => {
+          "nested_string" => "nested_value",
+          "nested_array" => [1,2,3]
+        }
+      }
+
+      headers = {"content-type" => ["application/JSON; charset=utf8"]}
+      body = '{"string":"value","number":1.23,"bool":true,"array":["one","two","three"],"nested":{"nested_string":"nested_value","nested_array":[1,2,3]}}'
+
+      deserialized = Encoder.new.deserialize_response(body, headers)
+
+      expect(deserialized).to eq(expected)
+    end
+
     it 'deserializes the response when content-type == text/*' do
       headers = {"content-type" => ["text/plain; charset=utf8"]}
       body = 'some text'
@@ -212,6 +232,13 @@ describe Encoder do
       expect(Encoder.new.deserialize_response(body, headers)).to eq('some text')
     end
 
+    it 'deserializes the response when content-type == text/*: case insensitive' do
+      headers = {"content-type" => ["TEXT/plain; charset=utf8"]}
+      body = 'some text'
+
+      expect(Encoder.new.deserialize_response(body, headers)).to eq('some text')
+    end
+
     it 'throws when attempting to deserialize multipart/*' do
       headers = {"content-type" => ["multipart/form-data"]}
       body = 'some multipart encoded data here'

data/spec/paypalhttp/http_client_spec.rb

@@ -226,6 +226,57 @@ describe HttpClient do
     expect(resp.result).to eq(return_data)
   end
 
+  it 'handles json array result: case insensitive' do
+    WebMock.enable!
+
+    return_data = ["one", "two"]
+
+    http_client = HttpClient.new(@environment)
+
+    stub_request(:get, @environment.base_url + "/v1/api")
+      .to_return(body: JSON.generate(return_data), status: 200, headers: {"Content-Type" => "application/JSON"})
+
+    req = OpenStruct.new({:verb => "GET", :path => "/v1/api"})
+
+    resp = http_client.execute(req)
+
+    expect(resp.result).to eq(return_data)
+  end
+
+  it 'handles plain text result' do
+    WebMock.enable!
+
+    return_data = "value"
+
+    http_client = HttpClient.new(@environment)
+
+    stub_request(:get, @environment.base_url + "/v1/api")
+      .to_return(body: return_data, status: 200, headers: {"Content-Type" => "text/plain; charset=utf8"})
+
+    req = OpenStruct.new({:verb => "GET", :path => "/v1/api"})
+
+    resp = http_client.execute(req)
+
+    expect(resp.result).to eq(return_data)
+  end
+
+  it 'handles plain text result: case insensitive' do
+    WebMock.enable!
+
+    return_data = "value"
+
+    http_client = HttpClient.new(@environment)
+
+    stub_request(:get, @environment.base_url + "/v1/api")
+      .to_return(body: return_data, status: 200, headers: {"Content-Type" => "TEXT/plain; charset=utf8"})
+
+    req = OpenStruct.new({:verb => "GET", :path => "/v1/api"})
+
+    resp = http_client.execute(req)
+
+    expect(resp.result).to eq(return_data)
+  end
+
   it 'deserializes nested response object into nested openstruct response' do
     WebMock.enable!
 

metadata

@@ -1,17 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: paypalhttp
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - PayPal
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-11-11 00:00:00.000000000 Z
+date: 2021-09-14 00:00:00.000000000 Z
 dependencies: []
 description: Used for generated API clients
-email: 
+email:
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -33,11 +33,11 @@ files:
 - spec/paypalhttp/serializers/form_part_spec.rb
 - spec/paypalhttp/serializers/multipart_spec.rb
 - spec/spec_helper.rb
-homepage: 
+homepage:
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -52,8 +52,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.0.9
+signing_key:
 specification_version: 4
 summary: PayPalHttp Client Library
 test_files: []