my_target_api 2.0.4 → 2.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 31c5a7b6d6b7d6de79c2704db6b9bc3101b0162d
-  data.tar.gz: 9e8755caca634eba30406c48fb2045270124f7f8
+  metadata.gz: 0534d2ade6c5094fbd43f6564d1a9d14a3b7e19f
+  data.tar.gz: 76e109957b1ea31fbd6f35ce4e2323c665585a4a
 SHA512:
-  metadata.gz: 937f7bb1e42d43d6661bacd65300f9104974ed9460665ddfd5488b9f29dccbcd4cfba3f886283190b8c186939182162cd88e916196abfb2063ebab9119a9509b
-  data.tar.gz: 2de78c84e54ef5567f9a1eb1999e8004940d98f9876db64ace2bdab8fc0c838c5310773d7378512de03e406395cb8bc477f000fb6619cf4b28a5c9d7928fcb1e
+  metadata.gz: 4aafd4665a3b022f6b8ee3191432107005bfae18a75cc75b68a047964d914df16ce1014fca9d53e663273c74ce65382f8d1b9a6a887d4ae0ac23f3e90a00d683
+  data.tar.gz: f8e106020f78d3de052431d1b33d6f917b4298b2be46c722ed101654b250b6a427a1b2b9cde8192f756cb35f9d8ceff8026966ded5d0e606f3996ab333c26201

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    my_target_api (2.0.3)
+    my_target_api (2.0.5)
       json (~> 2.0, >= 2.0.0)
       rest-client (~> 2.0, >= 2.0.0)
 

data/README.md

@@ -7,7 +7,7 @@
 Add this line to your application's Gemfile:
 
 ```
-gem 'my_target_api', '~> 2.0.4'
+gem 'my_target_api', '~> 2.0.5'
 ```
 
 Or install from command line:

data/lib/my_target_api.rb

@@ -10,6 +10,8 @@ class MyTargetApi
   autoload :RequestError, 'my_target_api/request_error'
   autoload :NetClient, 'my_target_api/net_client'
   autoload :LogRequestParametersDecorator, 'my_target_api/log_request_parameters_decorator'
+  autoload :AddAccessTokenToRequestDecorator, 'my_target_api/add_access_token_to_request_decorator'
+  autoload :PreparePostParamsDecorator, 'my_target_api/prepare_post_params_decorator'
 
   def initialize(access_token, options = {})
     @access_token = access_token
@@ -24,19 +26,19 @@ class MyTargetApi
   end
 
   def get_request(url, params)
-    request_object.get(url, params)
+    request_object.get(url, params, headers)
   end
 
   def post_request(url, params)
-    request_object.post(url, params)
+    request_object.post(url, params, headers)
   end
 
   def delete_request(url, params)
-    request_object.delete(url, params)
+    request_object.delete(url, params, headers)
   end
 
   def upload_request(url, content, params)
-    request_object.upload(url, content, params)
+    request_object.upload(url, content, params, headers)
   end
 
   private
@@ -45,13 +47,16 @@ class MyTargetApi
 
   def request_object
     @_request_object ||= begin
-      request_options = {
-        logger: options[:logger],
-        access_token: access_token,
-        headers: options[:headers]
-      }
-      LogRequestParametersDecorator.new(Request.new(request_options), request_options)
+      logger = options[:logger]
+      request = Request.new(logger: logger)
+      request = LogRequestParametersDecorator.new(request, { logger: logger }.compact)
+      request = AddAccessTokenToRequestDecorator.new(request, { access_token: access_token })
+      PreparePostParamsDecorator.new(request)
     end
   end
 
+  def headers
+    options[:headers] || {}
+  end
+
 end

data/lib/my_target_api/add_access_token_to_request_decorator.rb

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+
+class MyTargetApi
+  # Add Authorization header
+  class AddAccessTokenToRequestDecorator
+
+    CONTENT_OUTPUT_LIMIT = 1000
+
+    def initialize(origin, access_token: nil)
+      @origin = origin
+      @access_token = access_token
+    end
+
+    def get(url, params = {}, headers = {})
+      origin.get(url, params, add_authorization_header(headers))
+    end
+
+    def post(url, params = {}, headers = {})
+      origin.post(url, params, add_authorization_header(headers))
+    end
+
+    def delete(url, params = {}, headers = {})
+      origin.delete(url, params, add_authorization_header(headers))
+    end
+
+    def upload(url, content, params = {}, headers = {})
+      origin.upload(url, content, params, add_authorization_header(headers))
+    end
+
+    private
+
+    attr_reader :origin, :access_token
+
+    def add_authorization_header(headers)
+      if access_token
+        { 'Authorization' => "Bearer #{access_token}" }.merge(headers)
+      else
+        headers
+      end
+    end
+
+  end
+end

data/lib/my_target_api/log_request_parameters_decorator.rb

@@ -1,54 +1,95 @@
 # frozen_string_literal: true
 
 require 'json'
-require_relative './response_formatter'
-require_relative './net_client'
+require_relative './nil_logger'
 
 class MyTargetApi
   # Requests
   class LogRequestParametersDecorator
 
-    def initialize(origin, options = {})
+    CONTENT_OUTPUT_LIMIT = 1000
+
+    def initialize(origin, logger: NilLogger)
       @origin = origin
-      @options = options
+      @logger = logger
     end
 
-    def get(url, params = {})
-      log_hash(method: 'Request#get', url: url, params: params)
+    def get(url, params = {}, headers = {})
+      log_request(method: 'GET', url: url, params: params, headers: headers)
 
-      origin.get(url, params)
+      origin.get(url, params, headers)
     end
 
-    def post(url, params = {})
-      log_hash(method: 'Request#post', url: url, params: params)
+    def post(url, params = {}, headers = {})
+      log_request(method: 'POST', url: url, params: params, headers: headers)
 
-      origin.post(url, params)
+      origin.post(url, params, headers)
     end
 
-    def delete(url, params = {})
-      log_hash(method: 'Request#delete', url: url, params: params)
+    def delete(url, params = {}, headers = {})
+      log_request(method: 'DELETE', url: url, params: params, headers: headers)
 
-      origin.delete(url, params)
+      origin.delete(url, params, headers)
     end
 
-    def upload(url, content, params = {})
-      log_hash(method: 'Request#upload', url: url, params: params, content: 'no logging')
+    def upload(url, content, params = {}, headers = {})
+      log_request(method: 'POST', url: url, params: params, headers: headers, content: content)
 
-      origin.upload(url, content, params)
+      origin.upload(url, content, params, headers)
     end
 
     private
 
-    attr_reader :origin, :options
+    attr_reader :origin, :logger
+
+    def log_request(method:, url:, params:, headers:, content: nil)
+      logger <<
+        "#{inspect_request(method: method, url: url, params: params, headers: headers)}"\
+        "#{inspect_content(content)}"
+    end
+
+    def inspect_request(method:, url:, params:, headers:)
+      <<~LOG
+        #{method} #{url}
+        #{inspect_headers(headers)}
+        #{inspect_params(params)}
+      LOG
+    end
+
+    def inspect_content(content)
+      return '' unless content
+
+      result = "Body content:\n"
+      result +=
+        if content.size > CONTENT_OUTPUT_LIMIT
+          <<~LOG
+            << first #{CONTENT_OUTPUT_LIMIT} symbols >>
+            #{content[0...CONTENT_OUTPUT_LIMIT]}
+          LOG
+        else
+          "#{content}"
+        end
+      result
+    end
+
+    def inspect_headers(params)
+      if params.empty?
+        'Headers: No headers'
+      else
+        "Headers:\n#{params.map { |name, value| "#{name}: #{value}" }.join("\n")}"
+      end
+    end
 
-    def log_hash(hash)
-      log(hash.map do |key, value|
-        "#{key}: #{value.is_a?(String) ? value : value.inspect}"
-      end.join("\n"))
+    def inspect_params(params)
+      if params.empty?
+        'Params: No params'
+      else
+        "Params:\n#{params.map { |name, value| "#{name}: #{inspect_param(value)}" }.join("\n")}"
+      end
     end
 
-    def log(message)
-      options[:logger] << message if options[:logger]
+    def inspect_param(value)
+      value.is_a?(String) ? value : value.inspect
     end
 
   end

data/lib/my_target_api/nil_logger.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+class MyTargetApi
+  # NULL object
+  class NilLogger
+
+    def self.<<(_message); end
+
+  end
+end

data/lib/my_target_api/prepare_post_params_decorator.rb

@@ -0,0 +1,56 @@
+# frozen_string_literal: true
+
+require 'json'
+
+class MyTargetApi
+  # Prepare post params
+  class PreparePostParamsDecorator
+
+    def initialize(origin)
+      @origin = origin
+    end
+
+    def get(url, params = {}, headers = {})
+      origin.get(url, params, headers)
+    end
+
+    def post(url, params = {}, headers = {})
+      prepared = body_parameters(params)
+
+      if prepared.is_a?(String)
+        origin.upload(url, prepared, {}, { 'Content-Type' => 'application/json' }.merge(headers))
+      else
+        origin.post(url, prepared, headers)
+      end
+    end
+
+    def delete(url, params = {}, headers = {})
+      origin.delete(url, params, headers)
+    end
+
+    def upload(url, content, params = {}, headers = {})
+      origin.upload(url, content, params, headers)
+    end
+
+    private
+
+    attr_reader :origin
+
+    def body_parameters(params)
+      result_params = params.dup
+
+      if result_params.values.any? { |param| param.is_a? IO } || result_params[:grant_type]
+        individual_body_parameters(result_params)
+      else
+        result_params.to_json
+      end
+    end
+
+    def individual_body_parameters(params)
+      params.transform_values do |value|
+        value.is_a?(Array) || value.is_a?(Hash) ? value.to_json : value
+      end
+    end
+
+  end
+end

data/lib/my_target_api/request.rb

@@ -3,16 +3,17 @@
 require 'json'
 require_relative './response_formatter'
 require_relative './net_client'
+require_relative './nil_logger'
 
 class MyTargetApi
   # Requests
   class Request
 
-    def initialize(options = {})
-      @options = options
+    def initialize(logger: NilLogger)
+      @logger = logger
     end
 
-    def get(url, params = {})
+    def get(url, params = {}, headers = {})
       response = with_exception_handling(params) do
         NetClient.get(url, headers.merge(query(params)))
       end
@@ -20,15 +21,15 @@ class MyTargetApi
       process_response(response)
     end
 
-    def post(url, params = {})
+    def post(url, params = {}, headers = {})
       response = with_exception_handling(params) do
-        NetClient.post(url, body_parameters(params), headers)
+        NetClient.post(url, params, headers)
       end
 
       process_response(response)
     end
 
-    def delete(url, params = {})
+    def delete(url, params = {}, headers = {})
       response = with_exception_handling(params) do
         NetClient.delete(url, headers.merge(query(params)))
       end
@@ -36,12 +37,14 @@ class MyTargetApi
       process_response(response)
     end
 
-    def upload(url, content, params = {})
+    def upload(url, content, params = {}, headers = {})
       response = with_exception_handling(params) do
         NetClient.post(
           url,
           content,
-          { 'Content-Type' => 'application/octet-stream' }.merge(headers).merge(query(params))
+          { 'Content-Type' => 'application/octet-stream' }
+            .merge(headers)
+            .merge(query(params))
         )
       end
 
@@ -50,36 +53,12 @@ class MyTargetApi
 
     private
 
-    attr_reader :options
-
-    def body_parameters(params)
-      result_params = params.dup
-
-      if result_params.values.any? { |param| param.is_a? IO } || result_params[:grant_type]
-        individual_body_parameters(result_params)
-      else
-        result_params.to_json
-      end
-    end
-
-    def individual_body_parameters(params)
-      params.transform_values do |value|
-        value.is_a?(Array) || value.is_a?(Hash) ? value.to_json : value
-      end
-    end
+    attr_reader :logger
 
     def query(params)
       { params: params }
     end
 
-    def headers
-      if access_token
-        optional_headers.merge('Authorization' => "Bearer #{access_token}")
-      else
-        optional_headers
-      end
-    end
-
     def process_response(response)
       JSON.parse(response.body)
     rescue JSON::ParserError
@@ -93,18 +72,12 @@ class MyTargetApi
       response
     rescue NetClient::Exception => e
       original_exception = e.original_exception
-      log("#{original_exception.class.name} #{original_exception.message}")
+      logger << "#{original_exception.class.name} #{original_exception.message}"
       raise_with_params(params: params, original_exception: original_exception)
     end
 
     def log_response(response)
-      log(ResponseFormatter.new(response).format)
-    end
-
-    def headers_to_string(headers)
-      headers.map do |name, value|
-        "#{name}: #{value}"
-      end.join("\n")
+      logger << ResponseFormatter.new(response).format
     end
 
     def raise_with_params(params:, response: nil, original_exception: nil)
@@ -115,17 +88,5 @@ class MyTargetApi
       raise result
     end
 
-    def log(message)
-      options[:logger] << message if options[:logger]
-    end
-
-    def access_token
-      options[:access_token]
-    end
-
-    def optional_headers
-      options[:headers] || {}
-    end
-
   end
 end

data/lib/my_target_api/version.rb

@@ -2,6 +2,6 @@
 
 class MyTargetApi
 
-  VERSION = '2.0.4'
+  VERSION = '2.0.5'
 
 end

data/spec/request_spec.rb

@@ -21,7 +21,7 @@ describe MyTargetApi::Request do
       result = subject.post('https://target.my.com/api/v1/campaigns.json')
 
       expect(MyTargetApi::NetClient).to(
-        have_received(:post).with('https://target.my.com/api/v1/campaigns.json', '{}', {})
+        have_received(:post).with('https://target.my.com/api/v1/campaigns.json', {}, {})
       )
 
       expect(result).to eq('name' => 'Campaign 1')
@@ -96,7 +96,10 @@ describe MyTargetApi::Request do
                                         headers: {}))
       )
 
-      request = MyTargetApi::Request.new(access_token: 'my_target_token')
+      request = MyTargetApi::AddAccessTokenToRequestDecorator.new(
+        MyTargetApi::Request.new,
+        access_token: 'my_target_token'
+      )
       request.get('https://target.my.com/api/v1/some_path.json')
 
       expect(MyTargetApi::NetClient).to(
@@ -117,7 +120,7 @@ describe MyTargetApi::Request do
       logger = double('Logger double', '<<': nil)
       options = { logger: logger }
       request = MyTargetApi::LogRequestParametersDecorator.new(
-        MyTargetApi::Request.new(options), options
+        MyTargetApi::Request.new(options), logger: logger
       )
       allow(MyTargetApi::NetClient).to(
         receive(:get).and_return(double(code: 200,
@@ -127,9 +130,14 @@ describe MyTargetApi::Request do
 
       request.get('https://target.my.com/api/v1/request.json')
 
-      expect(logger).to(have_received(:<<).with("method: Request#get\n"\
-        "url: https://target.my.com/api/v1/request.json\n"\
-        'params: {}'))
+      expect(logger).to(
+        have_received(:<<)
+          .with(<<~LOG)
+            GET https://target.my.com/api/v1/request.json
+            Headers: No headers
+            Params: No params
+          LOG
+      )
       expect(logger).to(have_received(:<<).with(<<~LOG))
         HTTP Code: 200
         HTTP Body:
@@ -142,7 +150,7 @@ describe MyTargetApi::Request do
       logger = double('Logger double', '<<': nil)
       options = { logger: logger }
       request = MyTargetApi::LogRequestParametersDecorator.new(
-        MyTargetApi::Request.new(options), options
+        MyTargetApi::Request.new(options), logger: logger
       )
       allow(MyTargetApi::NetClient).to(
         receive(:get).and_return(
@@ -159,9 +167,12 @@ describe MyTargetApi::Request do
                         '404: Unknown resource. Inspect #params, #response and #original_exception'\
                         ' for more details')
 
-      expect(logger).to(have_received(:<<).with("method: Request#get\n"\
-        "url: https://target.my.com/api/v1/request.json\n"\
-        'params: {}'))
+      expect(logger).to(have_received(:<<).with(<<~LOG))
+        GET https://target.my.com/api/v1/request.json
+        Headers: No headers
+        Params: No params
+      LOG
+
       expect(logger).to(have_received(:<<).with(<<~LOG))
         HTTP Code: 404
         HTTP Body:

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: my_target_api
 version: !ruby/object:Gem::Version
-  version: 2.0.4
+  version: 2.0.5
 platform: ruby
 authors:
 - OneRetarget.com
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-09-01 00:00:00.000000000 Z
+date: 2020-09-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -153,8 +153,11 @@ files:
 - LICENSE.txt
 - README.md
 - lib/my_target_api.rb
+- lib/my_target_api/add_access_token_to_request_decorator.rb
 - lib/my_target_api/log_request_parameters_decorator.rb
 - lib/my_target_api/net_client.rb
+- lib/my_target_api/nil_logger.rb
+- lib/my_target_api/prepare_post_params_decorator.rb
 - lib/my_target_api/request.rb
 - lib/my_target_api/request_error.rb
 - lib/my_target_api/resource.rb