nxt_http_client 0.2.8 → 0.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5245a6f9449699cdffdac8c33cc39dab56cbcd3f492144595729bf6533c9e852
-  data.tar.gz: 6c09b7f3ef57eb1df6234516f7da6a9917924a43b89280efbeea791e0d9c1cef
+  metadata.gz: 0d9e310fbc3ec17ab20473c3794ad16abf9c4aca7c91efbab31a04542dff3884
+  data.tar.gz: 608cc184fc9e990cd3d2a2e1ca868f800f67f1e3210c4bf475edb22e923bc4b8
 SHA512:
-  metadata.gz: c86f043555c8e493c3be467da02ab7fcb4935cc48a1b2e637c7ae6c424de3c6b02327baed9d50e329ae8b509b699b02dfb868f4f7f24bcbc9056fb4a610e576b
-  data.tar.gz: 5077dd7b17869a9803d8ebd7cc1e4b14546ea45b682588ca0b16e2612e52f3ac4933353f2fc30e811e21c8ca418a2c370bc397f9c71aab5918143fd904cc8c3c
+  metadata.gz: 15e202d9f9d55cbc2cf7bf3746c97899f4c199e9e2ca46ea94cbd271e6e359a5022a611ef00f188252e82b995d925402ab1d8df92df75f08b171bc7d1fce4fbf
+  data.tar.gz: 23d855a852bf15986b02ae9a7bc923f062679fa3ea11c92e309834292a89e4d6b79400418905aeadb9a7d73c153ea7553abbb2ba519e81c72cf75f8aa672f94e

data/.circleci/config.yml

@@ -0,0 +1,57 @@
+# Ruby CircleCI 2.0 configuration file
+#
+# Check https://circleci.com/docs/2.0/language-ruby/ for more details
+#
+version: 2
+jobs:
+  build:
+    docker:
+      - image: circleci/ruby:2.7.0-node
+      - image: circleci/redis:5.0.4
+    environment:
+      BUNDLER_VERSION: 2.1.4
+
+    working_directory: ~/repo
+
+    steps:
+      - checkout
+
+      # Download and cache dependencies
+      - restore_cache:
+          keys:
+            - v1-dependencies-{{ checksum "Gemfile.lock" }}
+
+      - run: gem install bundler --version $BUNDLER_VERSION
+
+      - run:
+          name: install dependencies
+          command: |
+            bundle install --jobs=4 --retry=3 --path vendor/bundle
+
+      - save_cache:
+          paths:
+            - ./vendor/bundle
+          key: v1-dependencies-{{ checksum "Gemfile.lock" }}
+
+      # run tests!
+      - run:
+          name: run tests
+          command: |
+            mkdir /tmp/test-results
+            TEST_FILES="$(circleci tests glob "spec/**/*_spec.rb" | \
+              circleci tests split --split-by=timings)"
+
+            bundle exec rspec \
+              --format progress \
+              --format RspecJunitFormatter \
+              --out /tmp/test-results/rspec.xml \
+              --format progress \
+              $TEST_FILES
+
+      # collect reports
+      - store_artifacts:
+          path: /tmp/rspec/
+          destination: rspec
+
+      - store_test_results:
+          path: /tmp/rspec/

data/CHANGELOG.md

@@ -0,0 +1,14 @@
+# v0.3.3 2020-09-30
+
+### Updated NxtRegistry
+
+[Compare v0.3.2...v0.3.3](https://github.com/nxt-insurance/nxt_http_client/compare/v0.3.2...v0.3.3)
+
+# v0.2.10 2020-03-10
+
+### Refactored
+
+- [internal] Added CHANGELOG.MD
+- Refactored a bit
+
+[Compare v0.2.9...v0.3.0](https://github.com/nxt-insurance/nxt_http_client/compare/v0.2.9...v0.2.10)

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    nxt_http_client (0.2.8)
+    nxt_http_client (0.3.3)
       activesupport (~> 6.0.0)
       nxt_registry
       typhoeus
@@ -9,38 +9,38 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (6.0.2.1)
+    activesupport (6.0.3.3)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 0.7, < 2)
       minitest (~> 5.1)
       tzinfo (~> 1.1)
-      zeitwerk (~> 2.2)
+      zeitwerk (~> 2.2, >= 2.2.2)
     addressable (2.7.0)
       public_suffix (>= 2.0.2, < 5.0)
     coderay (1.1.2)
-    concurrent-ruby (1.1.5)
+    concurrent-ruby (1.1.7)
     crack (0.4.3)
       safe_yaml (~> 1.0.0)
     diff-lcs (1.3)
     ethon (0.12.0)
       ffi (>= 1.3.0)
-    ffi (1.11.3)
-    hashdiff (1.0.0)
-    i18n (1.8.1)
+    ffi (1.13.1)
+    hashdiff (1.0.1)
+    i18n (1.8.5)
       concurrent-ruby (~> 1.0)
-    method_source (0.9.2)
-    minitest (5.13.0)
-    nxt_registry (0.1.3)
+    method_source (1.0.0)
+    minitest (5.14.2)
+    nxt_registry (0.3.2)
       activesupport
     nxt_vcr_harness (0.1.3)
       rspec (~> 3.0)
       vcr (~> 5.0)
-    pry (0.12.2)
-      coderay (~> 1.1.0)
-      method_source (~> 0.9.0)
-    public_suffix (4.0.1)
-    rake (10.5.0)
-    redis (4.1.3)
+    pry (0.13.1)
+      coderay (~> 1.1)
+      method_source (~> 1.0)
+    public_suffix (4.0.6)
+    rake (13.0.1)
+    redis (4.2.2)
     rspec (3.9.0)
       rspec-core (~> 3.9.0)
       rspec-expectations (~> 3.9.0)
@@ -54,18 +54,20 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.9.0)
     rspec-support (3.9.0)
+    rspec_junit_formatter (0.4.1)
+      rspec-core (>= 2, < 4, != 2.12.0)
     safe_yaml (1.0.5)
     thread_safe (0.3.6)
-    typhoeus (1.3.1)
+    typhoeus (1.4.0)
       ethon (>= 0.9.0)
-    tzinfo (1.2.6)
+    tzinfo (1.2.7)
       thread_safe (~> 0.1)
-    vcr (5.0.0)
-    webmock (3.7.6)
+    vcr (5.1.0)
+    webmock (3.9.1)
       addressable (>= 2.3.6)
       crack (>= 0.3.2)
       hashdiff (>= 0.4.0, < 2.0.0)
-    zeitwerk (2.2.2)
+    zeitwerk (2.4.0)
 
 PLATFORMS
   ruby
@@ -75,9 +77,10 @@ DEPENDENCIES
   nxt_http_client!
   nxt_vcr_harness
   pry
-  rake (~> 10.0)
+  rake (~> 13.0)
   redis
   rspec (~> 3.0)
+  rspec_junit_formatter
   vcr
   webmock
 

data/README.md

@@ -43,6 +43,7 @@ class MyClient < NxtHttpClient
   
   register_response_handler do |handler|
     handler.on(:error) do |response|
+      Raven.extra_context(error_details: error.to_h) # call error.to_h to inspect request and response
       raise StandardError, "I can't handle this: #{response.code}"
     end
   end
@@ -93,6 +94,42 @@ class MyClient < NxtHttpClient
 end
 ```
 
+### HTTP Methods
+
+Instead of fire you can simply use the http verbs as methods
+
+```ruby
+class MyClient < NxtHttpClient
+  
+  def initialize(url)
+    @url = url
+  end
+
+  attr_reader :url
+
+  def fetch
+    get(url) do
+      handler.on(:success) { |response| response.body }
+    end
+  end
+
+  def create(params)
+    post(url, params: params) do
+      handler.on(:success) { |response| response.body }
+    end
+  end
+
+  def update(params)
+    put(url, params: params) do
+      handler.on(:success) { |response| response.body }
+    end
+  end
+
+  # ... there are others as you know ...
+end
+```
+
+
 ### configure
 
 Register default request options on the class level. Available options are `request_options` that are passed directly to 

data/lib/nxt_http_client.rb

@@ -11,5 +11,4 @@ require 'nxt_http_client/error'
 
 module NxtHttpClient
   class Error < StandardError; end
-  # Your code goes here...
 end

data/lib/nxt_http_client/client.rb

@@ -1,80 +1,48 @@
 module NxtHttpClient
   class Client
     extend ClientDsl
-    CACHE_STRATEGIES = %w[global thread]
+    CACHE_STRATEGIES = %w[global thread].freeze
+    HTTP_METHODS = %w[get post patch put delete head].freeze
 
     def build_request(url, **opts)
-      base_url = opts.delete(:base_url) || default_config.base_url
-      url = [base_url, url].reject(&:blank?).join('/')
-
-      duplicated_slashes = url.match(/([^:]\/{2,})/)
-      duplicated_slashes && duplicated_slashes.captures.each do |capture|
-        url.gsub!(capture, "#{capture[0]}/")
-      end
-
-      opts = default_config.request_options.with_indifferent_access.deep_merge(opts.with_indifferent_access)
-      opts[:headers] ||= {}
-
-      if default_config.x_request_id_proc
-        opts[:headers]['X-Request-ID'] ||= default_config.x_request_id_proc.call
-      end
-
-      if opts[:cache] ||= false
-        strategy = opts.delete(:cache)
-
-        case strategy.to_s
-        when 'thread'
-          cache_key = Thread.current[:nxt_http_client_cache_key] ||= "#{SecureRandom.base58}::#{DateTime.current}"
-          opts[:headers].reverse_merge!(cache_key: cache_key)
-        when 'global'
-          opts[:headers].delete(:cache_key)
-        else
-          raise ArgumentError, "Cache strategy unknown: #{strategy}. Options are #{CACHE_STRATEGIES}"
-        end
-      end
+      url = build_url(opts, url)
+      opts = build_headers(opts)
 
       Typhoeus::Request.new(url, **opts.symbolize_keys)
     end
 
-    def fire(url = '', **opts, &block)
-      # calling_method = caller_locations(1,1)[0].label
-      response_handler = opts.fetch(:response_handler) do
-        dup_handler_from_class || NxtHttpClient::ResponseHandler.new
-      end
+    delegate :before_fire_callback, :after_fire_callback, to: :class
 
+    def fire(url = '', **opts, &block)
+      response_handler = opts.fetch(:response_handler) { dup_handler_from_class || NxtHttpClient::ResponseHandler.new }
       response_handler.configure(&block) if block_given?
-      request = build_request(url, opts.except(:response_handler))
-
-      before_fire_callback = self.class.before_fire_callback
+      request = build_request(url, **opts.except(:response_handler))
       before_fire_callback && instance_exec(self, request, response_handler, &before_fire_callback)
 
-      if response_handler.callbacks['headers']
+      if response_handler.callbacks.resolve('headers')
         request.on_headers do |response|
           response_handler.eval_callback(self, 'headers', response)
         end
       end
 
-      if response_handler.callbacks['body']
+      if response_handler.callbacks.resolve('body')
         request.on_body do |response|
           response_handler.eval_callback(self, 'body', response)
         end
       end
 
       result = nil
-      error = nil
+      current_error = nil
 
       request.on_complete do |response|
-        callback = response_handler.callback_for_response(response)
-        result = callback && instance_exec(response, &callback) || response
-      rescue StandardError => e
-        error = e
+        result = callback_or_response(response, response_handler)
+      rescue StandardError => error
+        current_error = error
       ensure
-        after_fire_callback = self.class.after_fire_callback
-
         if after_fire_callback
-          result = instance_exec(self, request, response, result, error, &after_fire_callback)
+          result = instance_exec(self, request, response, result, current_error, &after_fire_callback)
         else
-          result || (raise error)
+          result || (raise current_error)
         end
       end
 
@@ -83,10 +51,32 @@ module NxtHttpClient
       result
     end
 
-    %w[get post patch put delete head].each do |method|
+    HTTP_METHODS.each do |method|
       define_method method do |url = '', **opts, &block|
-        fire(url, opts.reverse_merge(method: method), &block)
+        fire(url, **opts.reverse_merge(method: method), &block)
+      end
+    end
+
+    private
+
+    def build_url(opts, url)
+      base_url = opts.delete(:base_url) || default_config.base_url
+      url = [base_url, url].reject(&:blank?).join('/')
+
+      url_without_duplicated_hashes(url)
+      url
+    end
+
+    def build_headers(opts)
+      opts = default_config.request_options.with_indifferent_access.deep_merge(opts.with_indifferent_access)
+      opts[:headers] ||= {}
+
+      if default_config.x_request_id_proc
+        opts[:headers]['X-Request-ID'] ||= default_config.x_request_id_proc.call
       end
+
+      build_cache_header(opts)
+      opts
     end
 
     def dup_handler_from_class
@@ -96,5 +86,35 @@ module NxtHttpClient
     def default_config
       self.class.default_config
     end
+
+    def build_cache_header(opts)
+      if opts[:cache] ||= false
+        strategy = opts.delete(:cache)
+
+        case strategy.to_s
+        when 'thread'
+          cache_key = Thread.current[:nxt_http_client_cache_key] ||= "#{SecureRandom.base58}::#{DateTime.current}"
+          opts[:headers].reverse_merge!(cache_key: cache_key)
+        when 'global'
+          opts[:headers].delete(:cache_key)
+        else
+          raise ArgumentError, "Cache strategy unknown: #{strategy}. Options are #{CACHE_STRATEGIES}"
+        end
+      end
+    end
+
+    def url_without_duplicated_hashes(url)
+      duplicated_slashes = url.match(/([^:]\/{2,})/)
+      duplicated_slashes && duplicated_slashes.captures.each do |capture|
+        url.gsub!(capture, "#{capture[0]}/")
+      end
+
+      url
+    end
+
+    def callback_or_response(response, response_handler)
+      callback = response_handler.callback_for_response(response)
+      callback && instance_exec(response, &callback) || response
+    end
   end
 end

data/lib/nxt_http_client/client_dsl.rb

@@ -1,9 +1,7 @@
 module NxtHttpClient
   module ClientDsl
     def configure(opts = {}, &block)
-      opts.each do |k,v|
-        default_config.send(k, v)
-      end
+      opts.each { |k, v| default_config.send(k, v) }
       default_config.tap { |d| block.call(d) }
       default_config
     end
@@ -13,7 +11,7 @@ module NxtHttpClient
     end
 
     def before_fire_callback
-      @before_fire ||= dup_instance_variable_from_ancestor_chain(:@before_fire_callback)
+      @before_fire_callback ||= dup_instance_variable_from_ancestor_chain(:@before_fire_callback)
     end
 
     def after_fire(&block)
@@ -44,21 +42,16 @@ module NxtHttpClient
     end
 
     def instance_variable_from_ancestor_chain(instance_variable_name)
-      client = client_ancestors.find do |client|
-        client.instance_variable_get(instance_variable_name)
-      end
+      client = client_ancestors.find { |c| c.instance_variable_get(instance_variable_name) }
 
       client.instance_variable_get(instance_variable_name)
     end
 
     def dup_instance_variable_from_ancestor_chain(instance_variable_name)
       result = instance_variable_from_ancestor_chain(instance_variable_name).dup
+      return result unless block_given?
 
-      if block_given?
-        result || yield
-      else
-        result
-      end
+      result || yield
     end
   end
 end

data/lib/nxt_http_client/default_config.rb

@@ -1,5 +1,5 @@
 module NxtHttpClient
-  CONFIGURABLE_OPTIONS = %i[request_options base_url x_request_id_proc]
+  CONFIGURABLE_OPTIONS = %i[request_options base_url x_request_id_proc].freeze
 
   DefaultConfig = Struct.new('DefaultConfig', *CONFIGURABLE_OPTIONS) do
     def initialize(request_options: ActiveSupport::HashWithIndifferentAccess.new, base_url: '', x_request_id_proc: nil)

data/lib/nxt_http_client/error.rb

@@ -1,17 +1,24 @@
 module NxtHttpClient
   class Error < StandardError
-    def initialize(response)
+    def initialize(response, message = nil)
       @response = response.blank? ? Typhoeus::Response.new : response
+      @id = SecureRandom.uuid
+      @message = message || default_message
+
+      super(@message)
     end
 
-    attr_reader :response
+    attr_reader :response, :id, :message
+
+    alias_method :to_s, :message
 
-    def to_s
+    def default_message
       "NxtHttpClient::Error::#{response_code}"
     end
 
     def to_h
       {
+        id: id,
         url: url,
         response_code: response_code,
         request_options: request_options,
@@ -21,7 +28,7 @@ module NxtHttpClient
     end
 
     def body
-      if response_content_type == 'application/json'
+      if response_content_type&.starts_with?('application/json')
         JSON.parse(response.body)
       else
         response.body

data/lib/nxt_http_client/response_handler.rb

@@ -4,20 +4,14 @@ module NxtHttpClient
     include NxtRegistry
 
     def initialize
-      @callbacks = registry(
-        :callbacks,
-        call: false,
-        on_key_already_registered: ->(key) { raise_callback_already_registered(key) }
-      )
-
       @result = nil
     end
 
     attr_accessor :result
-    attr_reader :callbacks
 
     def eval_callback(target, key, response)
       return unless callbacks.resolve!(key)
+
       target.instance_exec(response, &callbacks.resolve(key))
     end
 
@@ -37,23 +31,31 @@ module NxtHttpClient
       register_callback(code, overwrite: true, &block)
     end
 
-    alias_method :on, :register_callback
-    alias_method :on!, :register_callback!
+    alias on register_callback
+    alias on! register_callback!
 
     def callback_for_response(response)
       key_from_response = response.code.to_s
-      return callbacks['any'] if callbacks['any'].present?
+      return callbacks.resolve('any') if callbacks.resolve('any').present?
 
       first_matching_key = callbacks.keys.sort.reverse.find do |key|
         regex_key = key.gsub('*', '[0-9]{1}')
         key_from_response =~ /\A#{regex_key}\z/
       end
 
-      first_matching_key && callbacks[first_matching_key] ||
-      response.success? && callbacks['success'] ||
-      response.timed_out? && callbacks['timed_out'] ||
-      !response.success? && callbacks['error'] ||
-      callbacks['others']
+      first_matching_key && callbacks.resolve(first_matching_key) ||
+        response.success? && callbacks.resolve('success') ||
+        response.timed_out? && callbacks.resolve('timed_out') ||
+        !response.success? && callbacks.resolve('error') ||
+        callbacks.resolve('others')
+    end
+
+    def callbacks
+      @callbacks ||= NxtRegistry::Registry.new(
+        :callbacks,
+        call: false,
+        on_key_already_registered: ->(key) { raise_callback_already_registered(key) }
+      )
     end
 
     private

data/lib/nxt_http_client/version.rb

@@ -1,3 +1,3 @@
 module NxtHttpClient
-  VERSION = "0.2.8"
+  VERSION = "0.3.3"
 end

data/nxt_http_client.gemspec

@@ -40,11 +40,12 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'nxt_registry'
 
   spec.add_development_dependency 'bundler', '~> 1.17'
-  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'rake', '~> 13.0'
   spec.add_development_dependency 'rspec', '~> 3.0'
   spec.add_development_dependency 'pry'
   spec.add_development_dependency 'vcr'
   spec.add_development_dependency 'webmock'
   spec.add_development_dependency 'nxt_vcr_harness'
   spec.add_development_dependency 'redis'
+  spec.add_development_dependency 'rspec_junit_formatter'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: nxt_http_client
 version: !ruby/object:Gem::Version
-  version: 0.2.8
+  version: 0.3.3
 platform: ruby
 authors:
 - Andreas Robecke
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-01-11 00:00:00.000000000 Z
+date: 2020-09-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: typhoeus
@@ -75,14 +75,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -167,6 +167,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec_junit_formatter
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: NxtHttpClinet allows you to easily create and configure http clients.
 email:
 - a.robecke@getsafe.de
@@ -174,9 +188,11 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".circleci/config.yml"
 - ".gitignore"
 - ".rspec"
 - ".travis.yml"
+- CHANGELOG.md
 - Gemfile
 - Gemfile.lock
 - LICENSE.txt
@@ -214,7 +230,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: NxtHttpClinet is a simple DSL on top the typhoeus http gem