nypl_sierra_api_client 1.0.3 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 227304609f16f96cf643ec758501e7ba23952da2c77cc513d6b08781622da9a8
-  data.tar.gz: a723fc2eb6e3fc3e33937c513eaddaa05d7a194b9f11fc8eb12f9d2cfd7da594
+  metadata.gz: 82e6e4a869d4151703b816abea4f6774b015be7d0a6703648de2306ed42a20da
+  data.tar.gz: 1aa79afd819567f025bc049bbd8f2c5e16613a807161deb045b3475fca4558d8
 SHA512:
-  metadata.gz: 213eed76eb255b12e38b00d7460dfb91bc8fac2afa2c92e025fb62019bee8a35d3c664b4713988865ed6b57327b736f3301858a40ae3a8019fad24e50342eb0e
-  data.tar.gz: 97b00de6b5114deb32da9e72b3490e03e26077c9e23db72f015adc584c5a56a6b912798156fd59021c3d9a43840755b96c4e2b92a113c0a6216cde9edbb53cd5
+  metadata.gz: 69e7d9f2889259802eeb7cb48a97251cb713e8cdddb30189300c2ba49c2847caa040b69641ada2a8e6a2269dec240fda6f3bd5ab03a8029fb5072d8e851b05ca
+  data.tar.gz: dc5e94f8650598f978dc068eacb3b76dd9eca87fb6f41ee5b489f62fe8279a51898bb24d79c4dd7b1be1c6933a6071dccc2b73ce37270f7bbe0a3b9c692fed1a

data/lib/nypl_sierra_api_client.rb

@@ -27,6 +27,8 @@ class SierraApiClient
     config_defaults[:env].each do |key, value|
       raise SierraApiClientError.new "Missing config: neither config.#{key} nor ENV.#{value} are set" unless @config[key]
     end
+
+    @retries = 0
   end
 
   def get (path, options = {})
@@ -55,17 +57,14 @@ class SierraApiClient
 
     authenticate! if options[:authenticated]
 
-    uri = URI.parse("#{@config[:base_url]}#{path}")
-
-    http = Net::HTTP.new(uri.host, uri.port)
-    http.use_ssl = uri.scheme === 'https'
+    @uri = URI.parse("#{@config[:base_url]}#{path}")
 
     # Build request headers:
     request_headers = {}
     request_headers['Content-Type'] = options[:headers]['Content-Type'] unless options.dig(:headers, 'Content-Type').nil?
 
     # Create HTTP::Get or HTTP::Post
-    request =  Net::HTTP.const_get(method.capitalize).new(uri, request_headers)
+    request =  Net::HTTP.const_get(method.capitalize).new(@uri, request_headers)
 
     # Add bearer token header
     request['Authorization'] = "Bearer #{@access_token}" if options[:authenticated]
@@ -73,31 +72,57 @@ class SierraApiClient
     # Allow caller to modify the request before we send it off:
     yield request if block_given?
 
-    logger.debug "SierraApiClient: #{method} to Sierra api", { uri: uri, body: request.body }
+    logger.debug "SierraApiClient: #{method} to Sierra api", { uri: @uri, body: request.body }
+
+    execute request, options
+  end
+
+  def execute (request, options)
+    http = Net::HTTP.new(@uri.host, @uri.port)
+    http.use_ssl = @uri.scheme === 'https'
 
     begin
-      # Execute request:
       response = http.request(request)
+      logger.debug "SierraApiClient: Got Sierra api response", { code: response.code, body: response.body }
     rescue => e
-      raise SierraApiClientError.new(e), "Failed to #{method} to #{path}: #{e.message}"
+      raise SierraApiClientError.new "Failed to #{request.method} to #{request.path}: #{e.message}"
     end
 
-    logger.debug "SierraApiClient: Got Sierra api response", { code: response.code, body: response.body }
-
-    parse_response response
+    handle_response response, request, options
   end
 
-  def parse_response (response)
+  def handle_response (response, request, options)
     if response.code == "401"
       # Likely an expired access-token; Wipe it for next run
-      # TODO: Implement token refresh
       @access_token = nil
-      raise SierraApiClientTokenError.new("Got a 401: #{response.body}")
+      if @retries < 3
+        if options[:authenticated]
+          logger.debug "SierraApiClient: Refreshing oauth token for 401", { code: 401, body: response.body, retry: @retries }
+
+          return reauthenticate_and_reattempt request, options
+        end
+      else
+        retries_exceeded = true
+      end
+
+      reset_retries
+      message = "Got a 401: #{retries_exceeded ? "Maximum retries exceeded, " : ''}#{response.body}"
+      raise SierraApiClientTokenError.new(message)
     end
 
+    reset_retries if @retries > 0
     SierraApiResponse.new(response)
   end
 
+  def reauthenticate_and_reattempt request, options
+    @retries += 1
+    authenticate!
+    # Reset bearer token header
+    request['Authorization'] = "Bearer #{@access_token}"
+
+    execute request, options
+  end
+
   def parse_http_options (_options)
     options = {
       authenticated: true
@@ -143,4 +168,8 @@ class SierraApiClient
   def logger
     @logger ||= NyplLogFormatter.new(STDOUT, level: @config[:log_level])
   end
+
+  def reset_retries
+    @retries = 0
+  end
 end

metadata

@@ -1,11 +1,11 @@
 --- !ruby/object:Gem::Specification
 name: nypl_sierra_api_client
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.1.0
 platform: ruby
 authors:
 - nonword
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
 date: 2020-04-21 00:00:00.000000000 Z
@@ -23,7 +23,7 @@ homepage: https://github.com/NYPL/ruby-sierra-api-client
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -39,7 +39,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.2
-signing_key: 
+signing_key:
 specification_version: 4
 summary: NYPL Sierra API client
 test_files: []