lhc 11.1.1 → 12.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 236cb60f77bb99d1d14c61dd9fc9cd3c78a8bcacd734deb34b20eb72eee2c7c2
-  data.tar.gz: 6956f64dc5409d253b7bff60f384dba7b02aff6df7286c50a8c762011eeb8575
+  metadata.gz: ad768b826ac53e7175609b97943ffbb746f514c1f99bdfb7174d48307052df4c
+  data.tar.gz: 7bcbcf0c7d08b7be73eb7ece1b688fb8be003d68d269b4ef70d5994fab9cc07a
 SHA512:
-  metadata.gz: '0559059bd1594c8cf6efd9ea3805318af91835e20e87e200176f3eb2c74b7fb83e40f682725a7b4043d443e8950debf9edf1016cc40b41518a8f608a22e10fae'
-  data.tar.gz: 93846206064d7c3547814aa6e8f4aecd857dc59b139de09c0b921937709c6088be9e1fd8b0c0e8ab5218f6fa3a6ab85bd86ba80cc3bddac36affc449c6f13456
+  metadata.gz: c181e5bc874086374c453e6056d31c948eaa2d4a869dfa002819e3236d486c6b98e7d3c058e5cca2dbc423d58a0777b3208471426dc8e47c085cc92cedef6afb
+  data.tar.gz: 3203b6a2ae1c3180271b7d7f16b5dbf1b499cad6c623145139d74a5e65bd59d7376267702417e483f6c82b0942d89e72b0b24fe7e978b349de75a9a1cffd6e55

data/README.md

@@ -1,6 +1,3 @@
-LHC
-===
-
 LHC is an extended/advanced HTTP client. Implementing basic http-communication enhancements like interceptors, exception handling, format handling, accessing response data, configuring endpoints and placeholders and fully compatible, RFC-compliant URL-template support.
 
 LHC uses [typhoeus](https://github.com/typhoeus/typhoeus) for low level http communication.
@@ -30,69 +27,70 @@ use it like:
 ```
 
 ## Table of contents
-   * [LHC](#lhc)
-      * [Quick start guide](#quick-start-guide)
-      * [Table of contents](#table-of-contents)
-      * [Basic methods](#basic-methods)
-      * [Request](#request)
-         * [Formats](#formats)
-            * [Default format](#default-format)
-            * [Unformatted requests](#unformatted-requests)
-               * [Upload with LHC](#upload-with-lhc)
-         * [Parallel requests](#parallel-requests)
-         * [Follow redirects](#follow-redirects)
-         * [Transfer data through the request body](#transfer-data-through-the-request-body)
-         * [Request parameters](#request-parameters)
-            * [Array Parameter Encoding](#array-parameter-encoding)
-         * [Request URL encoding](#request-url-encoding)
-         * [Request URL-Templates](#request-url-templates)
-         * [Request timeout](#request-timeout)
-         * [Request Agent](#request-agent)
-      * [Response](#response)
-         * [Accessing response data](#accessing-response-data)
-      * [Exceptions](#exceptions)
-         * [Custom error handling](#custom-error-handling)
-         * [Ignore certain errors](#ignore-certain-errors)
-      * [Configuration](#configuration)
-         * [Configuring endpoints](#configuring-endpoints)
-         * [Configuring placeholders](#configuring-placeholders)
-      * [Interceptors](#interceptors)
-         * [Quick start: Configure/Enable Interceptors](#quick-start-configureenable-interceptors)
-         * [Interceptors on local request level](#interceptors-on-local-request-level)
-         * [Core Interceptors](#core-interceptors)
-            * [Authentication Interceptor](#authentication-interceptor)
-               * [Bearer Authentication](#bearer-authentication)
-               * [Basic Authentication](#basic-authentication)
-               * [Reauthenticate](#reauthenticate)
-               * [Bearer Authentication with client access token](#bearer-authentication-with-client-access-token)
-            * [Caching Interceptor](#caching-interceptor)
-               * [Options](#options)
-            * [Default Timeout Interceptor](#default-timeout-interceptor)
-               * [Overwrite defaults](#overwrite-defaults)
-            * [Logging Interceptor](#logging-interceptor)
-               * [Installation](#installation)
-               * [What and how it logs](#what-and-how-it-logs)
-               * [Configure](#configure)
-            * [Monitoring Interceptor](#monitoring-interceptor)
-               * [Installation](#installation-1)
-               * [Environment](#environment)
-               * [What it tracks](#what-it-tracks)
-               * [Configure](#configure-1)
-            * [Prometheus Interceptor](#prometheus-interceptor)
-            * [Retry Interceptor](#retry-interceptor)
-               * [Limit the amount of retries while making the request](#limit-the-amount-of-retries-while-making-the-request)
-               * [Change the default maximum of retries of the retry interceptor](#change-the-default-maximum-of-retries-of-the-retry-interceptor)
-               * [Retry all requests](#retry-all-requests)
-            * [Rollbar Interceptor](#rollbar-interceptor)
-               * [Forward additional parameters](#forward-additional-parameters)
-            * [Throttle](#throttle)
-            * [Zipkin](#zipkin)
-         * [Create an interceptor from scratch](#create-an-interceptor-from-scratch)
-            * [Interceptor callbacks](#interceptor-callbacks)
-            * [Interceptor request/response](#interceptor-requestresponse)
-            * [Provide a response replacement through an interceptor](#provide-a-response-replacement-through-an-interceptor)
-      * [Testing](#testing)
-      * [License](#license)
+  * [Quick start guide](#quick-start-guide)
+  * [Basic methods](#basic-methods)
+  * [Request](#request)
+     * [Formats](#formats)
+        * [Default format](#default-format)
+        * [Unformatted requests](#unformatted-requests)
+           * [Upload with LHC](#upload-with-lhc)
+     * [Parallel requests](#parallel-requests)
+     * [Follow redirects](#follow-redirects)
+     * [Transfer data through the request body](#transfer-data-through-the-request-body)
+     * [Request parameters](#request-parameters)
+        * [Array Parameter Encoding](#array-parameter-encoding)
+     * [Request URL encoding](#request-url-encoding)
+     * [Request URL-Templates](#request-url-templates)
+     * [Request timeout](#request-timeout)
+     * [Request Agent](#request-agent)
+  * [Response](#response)
+     * [Accessing response data](#accessing-response-data)
+  * [Exceptions](#exceptions)
+     * [Custom error handling (rescue)](#custom-error-handling-rescue)
+     * [Ignore certain errors](#ignore-certain-errors)
+  * [Configuration](#configuration)
+     * [Configuring endpoints](#configuring-endpoints)
+     * [Configuring placeholders](#configuring-placeholders)
+  * [Interceptors](#interceptors)
+     * [Quick start: Configure/Enable Interceptors](#quick-start-configureenable-interceptors)
+     * [Interceptors on local request level](#interceptors-on-local-request-level)
+     * [Core Interceptors](#core-interceptors)
+        * [Authentication Interceptor](#authentication-interceptor)
+           * [Bearer Authentication](#bearer-authentication)
+           * [Basic Authentication](#basic-authentication)
+           * [Reauthenticate](#reauthenticate)
+           * [Bearer Authentication with client access token](#bearer-authentication-with-client-access-token)
+        * [Caching Interceptor](#caching-interceptor)
+           * [Options](#options)
+        * [Default Timeout Interceptor](#default-timeout-interceptor)
+           * [Overwrite defaults](#overwrite-defaults)
+        * [Logging Interceptor](#logging-interceptor)
+           * [Installation](#installation)
+           * [What and how it logs](#what-and-how-it-logs)
+           * [Configure](#configure)
+        * [Monitoring Interceptor](#monitoring-interceptor)
+           * [Installation](#installation-1)
+           * [Environment](#environment)
+           * [What it tracks](#what-it-tracks)
+           * [Configure](#configure-1)
+        * [Prometheus Interceptor](#prometheus-interceptor)
+        * [Retry Interceptor](#retry-interceptor)
+           * [Limit the amount of retries while making the request](#limit-the-amount-of-retries-while-making-the-request)
+           * [Change the default maximum of retries of the retry interceptor](#change-the-default-maximum-of-retries-of-the-retry-interceptor)
+           * [Retry all requests](#retry-all-requests)
+           * [Do not retry certain response codes](#do-not-retry-certain-response-codes)
+        * [Rollbar Interceptor](#rollbar-interceptor)
+           * [Forward additional parameters](#forward-additional-parameters)
+        * [Throttle](#throttle)
+        * [Zipkin](#zipkin)
+     * [Create an interceptor from scratch](#create-an-interceptor-from-scratch)
+        * [Interceptor callbacks](#interceptor-callbacks)
+        * [Interceptor request/response](#interceptor-requestresponse)
+        * [Provide a response replacement through an interceptor](#provide-a-response-replacement-through-an-interceptor)
+  * [Testing](#testing)
+  * [License](#license)
+
+
 
 ## Basic methods
 
@@ -403,7 +401,7 @@ timeout? => LHC::Timeout
 anything_else => LHC::UnknownError
 ```
 
-### Custom error handling
+### Custom error handling (rescue)
 
 You can provide custom error handlers to handle errors happening during the request.
 
@@ -413,19 +411,21 @@ If your error handler returns anything else but `nil` it replaces the response b
 
 ```ruby
 handler = ->(response){ do_something_with_response; return {name: 'unknown'} }
-response = LHC.get('http://something', error_handler: handler)
+response = LHC.get('http://something', rescue: handler)
 response.data.name # 'unknown'
 ```
 
 ### Ignore certain errors
 
 As it's discouraged to rescue errors and then don't handle them (ruby styleguide)[https://github.com/bbatsov/ruby-style-guide#dont-hide-exceptions],
-but you often want to continue working with `nil`, LHC provides the `ignored_errors` option.
+but you often want to continue working with `nil`, LHC provides the `ignore` option.
 
 Errors listed in this option will not be raised and will leave the `response.body` and `response.data` to stay `nil`.
 
+You can either pass the LHC error class you want to be ignored or an array of LHC error classes.
+
 ```ruby
-response = LHC.get('http://something', ignored_errors: [LHC::NotFound])
+response = LHC.get('http://something', ignore: LHC::NotFound)
 
 response.body # nil
 response.data # nil
@@ -817,16 +817,16 @@ If you want to retry all requests made from your application, you just need to c
 
 ##### Do not retry certain response codes
 
-If you do not want to retry based on certain response codes, use retry in combination with explicit `ignore_errors`:
+If you do not want to retry based on certain response codes, use retry in combination with explicit `ignore`:
 
 ```ruby
-  LHC.get('http://local.ch', ignore_errors: [LHC::NotFound], retry: { max: 1 })
+  LHC.get('http://local.ch', ignore: LHC::NotFound, retry: { max: 1 })
 ```
 
 Or if you use `LHC::Retry.all`:
 
 ```ruby
-LHC.get('http://local.ch', ignore_errors: [LHC::NotFound])
+LHC.get('http://local.ch', ignore: LHC::NotFound)
 ```
 
 #### Rollbar Interceptor

data/lhc.gemspec

@@ -35,5 +35,5 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'timecop'
   s.add_development_dependency 'webmock'
 
-  s.license = 'GPL-3'
+  s.license = 'GPL-3.0'
 end

data/lib/lhc/error.rb

@@ -59,6 +59,10 @@ class LHC::Error < StandardError
     self.response = response
   end
 
+  def self.to_a
+    [self]
+  end
+
   def to_s
     return response if response.is_a?(String)
     request = response.request

data/lib/lhc/interceptors/auth.rb

@@ -27,7 +27,7 @@ class LHC::Auth < LHC::Interceptor
   def basic_authentication!
     auth = auth_options[:basic]
     credentials = "#{auth[:username]}:#{auth[:password]}"
-    set_authorization_header("Basic #{Base64.encode64(credentials).chomp}")
+    set_authorization_header("Basic #{Base64.strict_encode64(credentials).chomp}")
   end
 
   def bearer_authentication!

data/lib/lhc/request.rb

@@ -15,10 +15,10 @@ class LHC::Request
   attr_accessor :response, :options, :raw, :format, :error_handler, :errors_ignored, :source
 
   def initialize(options, self_executing = true)
-    self.errors_ignored = (options.fetch(:ignored_errors, []) || []).compact
+    self.errors_ignored = (options.fetch(:ignore, []) || []).to_a.compact
     self.source = options&.dig(:source)
     self.options = format!(options.deep_dup || {})
-    self.error_handler = options.delete :error_handler
+    self.error_handler = options.delete :rescue
     use_configured_endpoint!
     generate_url_from_template!
     self.interceptors = LHC::Interceptors.new(self)

data/lib/lhc/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module LHC
-  VERSION ||= '11.1.1'
+  VERSION ||= '12.0.3'
 end

data/spec/interceptors/auth/long_basic_auth_credentials_spec.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+describe LHC::Auth do
+  before(:each) do
+    LHC.config.interceptors = [LHC::Auth]
+  end
+
+  it 'adds basic auth in a correct way even if username and password are especially long' do
+    options = { basic: { username: '123456789101234', password: '12345678901234567890123456789012' } }
+    LHC.config.endpoint(:local, 'http://local.ch', auth: options)
+    stub_request(:get, 'http://local.ch')
+      .with(headers: { 'Authorization' => 'Basic MTIzNDU2Nzg5MTAxMjM0OjEyMzQ1Njc4OTAxMjM0NTY3ODkwMTIzNDU2Nzg5MDEy' })
+    LHC.get(:local)
+  end
+end

data/spec/interceptors/retry/main_spec.rb

@@ -66,7 +66,7 @@ describe LHC::Rollbar do
 
     it 'does not retry if the error is explicitly ignored' do
       request_stub
-      LHC.get('http://local.ch', retry: { max: 1 }, ignored_errors: [LHC::NotFound])
+      LHC.get('http://local.ch', retry: { max: 1 }, ignore: [LHC::NotFound])
       expect(request_stub).to have_been_requested.times(1)
     end
   end

data/spec/request/error_handling_spec.rb

@@ -67,21 +67,21 @@ describe LHC::Request do
     it 'handles errors with the provided handler and does not raise them' do
       stub_request(:get, "http://something").to_return(status: 400)
       handler = double('handler', call: -> {})
-      LHC::Request.new(url: "http://something", error_handler: handler)
+      LHC::Request.new(url: "http://something", rescue: handler)
       expect(handler).to have_received(:call)
     end
 
     it 'exchanges body with handlers return if the handler returns something' do
       stub_request(:get, "http://something").to_return(status: 400)
       handler = ->(_response) { { name: 'unknown' }.to_json }
-      request = LHC::Request.new(url: "http://something", error_handler: handler)
+      request = LHC::Request.new(url: "http://something", rescue: handler)
       expect(request.response.data.name).to eq 'unknown'
     end
 
     it 'does not exchange body with handlers return if the handler returns nil' do
       stub_request(:get, "http://something").to_return(status: 400, body: { message: 'an error occurred' }.to_json)
       handler = ->(_response) { nil }
-      request = LHC::Request.new(url: "http://something", error_handler: handler)
+      request = LHC::Request.new(url: "http://something", rescue: handler)
       expect(request.response.data.message).to eq 'an error occurred'
     end
   end

data/spec/request/ignore_errors_spec.rb

@@ -4,7 +4,7 @@ require 'rails_helper'
 
 describe LHC::Request do
   context 'ignoring LHC::NotFound' do
-    let(:response) { LHC.get('http://local.ch', ignored_errors: [LHC::NotFound]) }
+    let(:response) { LHC.get('http://local.ch', ignore: [LHC::NotFound]) }
 
     before { stub_request(:get, 'http://local.ch').to_return(status: 404) }
 
@@ -36,13 +36,13 @@ describe LHC::Request do
 
     it "does not raise an error when it's a subclass of the ignored error" do
       expect {
-        LHC.get('http://local.ch', ignored_errors: [LHC::Error])
+        LHC.get('http://local.ch', ignore: [LHC::Error])
       }.not_to raise_error
     end
 
     it "does raise an error if it's not a subclass of the ignored error" do
       expect {
-        LHC.get('http://local.ch', ignored_errors: [ArgumentError])
+        LHC.get('http://local.ch', ignore: [ArgumentError])
       }.to raise_error(LHC::NotFound)
     end
   end
@@ -52,14 +52,22 @@ describe LHC::Request do
 
     it "does not raise an error when ignored errors is set to array with nil" do
       expect {
-        LHC.get('http://local.ch', ignored_errors: [nil])
+        LHC.get('http://local.ch', ignore: [nil])
       }.to raise_error(LHC::NotFound)
     end
 
     it "does not raise an error when ignored errors is set to nil" do
       expect {
-        LHC.get('http://local.ch', ignored_errors: nil)
+        LHC.get('http://local.ch', ignore: nil)
       }.to raise_error(LHC::NotFound)
     end
   end
+
+  context 'passing keys instead of arrays' do
+    before { stub_request(:get, 'http://local.ch').to_return(status: 404) }
+
+    it "does not raise an error when ignored errors is a key instead of an array" do
+      LHC.get('http://local.ch', ignore: LHC::NotFound)
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lhc
 version: !ruby/object:Gem::Version
-  version: 11.1.1
+  version: 12.0.3
 platform: ruby
 authors:
 - https://github.com/local-ch/lhc/contributors
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-04-20 00:00:00.000000000 Z
+date: 2020-07-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -325,6 +325,7 @@ files:
 - spec/interceptors/after_response_spec.rb
 - spec/interceptors/auth/basic_auth_spec.rb
 - spec/interceptors/auth/bearer_spec.rb
+- spec/interceptors/auth/long_basic_auth_credentials_spec.rb
 - spec/interceptors/auth/reauthentication_configuration_spec.rb
 - spec/interceptors/auth/reauthentication_spec.rb
 - spec/interceptors/before_request_spec.rb
@@ -384,7 +385,7 @@ files:
 - spec/timeouts/timings_spec.rb
 homepage: https://github.com/local-ch/lhc
 licenses:
-- GPL-3
+- GPL-3.0
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -475,6 +476,7 @@ test_files:
 - spec/interceptors/after_response_spec.rb
 - spec/interceptors/auth/basic_auth_spec.rb
 - spec/interceptors/auth/bearer_spec.rb
+- spec/interceptors/auth/long_basic_auth_credentials_spec.rb
 - spec/interceptors/auth/reauthentication_configuration_spec.rb
 - spec/interceptors/auth/reauthentication_spec.rb
 - spec/interceptors/before_request_spec.rb