mais_orcid_client 0.3.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5f67823f01c96aa641315a8d64c463bc469c18d746ad5fd05a449a66f3c63925
-  data.tar.gz: 2f1435dabf75dd8c0956771f4519f978f0b745511310d3bfdc978bb0bf4bafb0
+  metadata.gz: b93e75dc3042635205b947c21d9697808b9bdefa9d9dae8598e03db1eac3e128
+  data.tar.gz: d714d5c0985596191dba26e801a43455dd926b307e5a88753c66e5a8550d918d
 SHA512:
-  metadata.gz: '083990f9b00f45ee537c8f07ea34450160a6b161167345a9e058cacb08fcb68b3c4c710877ae03aefe6f02729d48345536ecba981e9784697646ad6a89857636'
-  data.tar.gz: 93b51cbc03f7f5e2f8ea8e664cda30c07f817dbe592f0610ff69bfef1c0cbc50218122bb81d1e5a5bd631f5e99c12d4c82bca15add09c8d2a68fcf4f83238c78
+  metadata.gz: 227b68aa40fee6c5a55dbea773ef788bc5235c094a8ede1b717881ad9cb4e431de368a4d8e20b7564e60dc9c02f4f0657d47835a2bc490a4f2db174f6ba0f8e0
+  data.tar.gz: 8860974d178d49ddca9153d9a71a31ca25d4e2050541c02fc7d185c7ad04d45854dc4087022c6194b9c52883a2d7e385a4a6beb7be0ab5858c41608627e4cb37

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    mais_orcid_client (0.3.0)
+    mais_orcid_client (0.3.1)
       activesupport (>= 4.2, < 8)
       faraday
       faraday-retry
@@ -11,12 +11,12 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (7.0.6)
+    activesupport (7.0.7)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
       tzinfo (~> 2.0)
-    addressable (2.8.4)
+    addressable (2.8.5)
       public_suffix (>= 2.0.2, < 6.0)
     ast (2.4.2)
     byebug (11.1.3)
@@ -39,7 +39,7 @@ GEM
     jwt (2.7.1)
     language_server-protocol (3.17.0.3)
     lint_roller (1.1.0)
-    minitest (5.18.1)
+    minitest (5.19.0)
     multi_xml (0.6.0)
     oauth2 (2.0.9)
       faraday (>= 0.17.3, < 3.0)
@@ -58,7 +58,7 @@ GEM
     rainbow (3.1.1)
     rake (13.0.6)
     regexp_parser (2.8.1)
-    rexml (3.2.5)
+    rexml (3.2.6)
     rspec (3.12.0)
       rspec-core (~> 3.12.0)
       rspec-expectations (~> 3.12.0)
@@ -91,7 +91,7 @@ GEM
     rubocop-performance (1.18.0)
       rubocop (>= 1.7.0, < 2.0)
       rubocop-ast (>= 0.4.0)
-    rubocop-rspec (2.22.0)
+    rubocop-rspec (2.23.2)
       rubocop (~> 1.33)
       rubocop-capybara (~> 2.17)
       rubocop-factory_bot (~> 2.22)
@@ -127,7 +127,7 @@ GEM
       addressable (>= 2.8.0)
       crack (>= 0.3.2)
       hashdiff (>= 0.4.0, < 2.0.0)
-    zeitwerk (2.6.8)
+    zeitwerk (2.6.11)
 
 PLATFORMS
   x86_64-darwin-19

data/lib/mais_orcid_client/authenticator.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+class MaisOrcidClient
+  # The namespace for the "login" command
+  class Authenticator
+    attr_reader :client_id, :client_secret, :base_url
+
+    def self.token(client_id, client_secret, base_url)
+      new(client_id, client_secret, base_url).token
+    end
+
+    def initialize(client_id, client_secret, base_url)
+      @client_id = client_id
+      @client_secret = client_secret
+      @base_url = base_url
+    end
+
+    # @return [String]
+    def token
+      client = OAuth2::Client.new(client_id, client_secret, site: base_url,
+        token_url: "/api/oauth/token", authorize_url: "/api/oauth/authorize",
+        auth_scheme: :request_body)
+      client.client_credentials.get_token.token
+    end
+  end
+end

data/lib/mais_orcid_client/token_wrapper.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+class MaisOrcidClient
+  # Wraps API operations to request new access token if expired
+  class TokenWrapper
+    def self.refresh(config)
+      yield
+    rescue UnexpectedResponse::UnauthorizedError
+      config.token = Authenticator.token(config.client_id, config.client_secret, config.base_url)
+      yield
+    end
+  end
+end

data/lib/mais_orcid_client/unexpected_response.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+class MaisOrcidClient
+  # Handles unexpected responses when communicating with Mais
+  class UnexpectedResponse
+    # Error raised when the Mais API returns a 401 Unauthorized
+    class UnauthorizedError < StandardError; end
+
+    # Error raised when the Mais API returns a 500 error
+    class ServerError < StandardError; end
+
+    # Error raised when the Mais API returns a response with an error message in it
+    class ResponseError < StandardError; end
+
+    def self.call(response)
+      case response.status
+      when 401
+        raise UnauthorizedError, "There was a problem with the access token: #{response.body}"
+      when 500
+        raise ServerError, "Mais server error: #{response.body}"
+      else
+        raise StandardError, "Unexpected response: #{response.status} #{response.body}"
+      end
+    end
+  end
+end

data/lib/mais_orcid_client/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class MaisOrcidClient
-  VERSION = "0.3.0"
+  VERSION = "0.3.1"
 end

data/lib/mais_orcid_client.rb

@@ -28,16 +28,20 @@ class MaisOrcidClient
     # @param client_secret [String] the client secret to authenticate with MAIS
     # @param base_url [String] the base URL for the API
     def configure(client_id:, client_secret:, base_url:)
-      instance.base_url = base_url
-      instance.client_id = client_id
-      instance.client_secret = client_secret
+      instance.config = OpenStruct.new(
+        token: Authenticator.token(client_id, client_secret, base_url),
+        client_id:,
+        client_secret:,
+        base_url:
+      )
+
       self
     end
 
-    delegate :fetch_orcid_users, :fetch_orcid_user, to: :instance
+    delegate :config, :fetch_orcid_users, :fetch_orcid_user, to: :instance
   end
 
-  attr_accessor :base_url, :client_id, :client_secret
+  attr_accessor :config
 
   # @param [int] limit number of users requested
   # @param [int] page_size number of users per page
@@ -104,41 +108,32 @@ class MaisOrcidClient
   end
 
   def get_response(path, allow404: false)
-    response = conn.get("/mais/orcid/v1#{path}")
+    TokenWrapper.refresh(config) do
+      response = conn.get("/mais/orcid/v1#{path}")
 
-    return if allow404 && response.status == 404
+      return if allow404 && response.status == 404
 
-    raise "UIT MAIS ORCID User API returned #{response.status}" if response.status != 200
+      return UnexpectedResponse.call(response) unless response.success?
 
-    body = JSON.parse(response.body).with_indifferent_access
-    raise "UIT MAIS ORCID User API returned an error: #{response.body}" if body.key?(:error)
+      body = JSON.parse(response.body).with_indifferent_access
+      raise UnexpectedResponse::ResponseError, "UIT MAIS ORCID User API returned an error: #{response.body}" if body.key?(:error)
 
-    body
+      body
+    end
   end
 
-  # @return [Faraday::Connection]
   def conn
-    @conn ||= begin
-      conn = Faraday.new(url: base_url) do |faraday|
-        faraday.request :retry, max: 3,
-          interval: 0.5,
-          interval_randomness: 0.5,
-          backoff_factor: 2
-      end
-      conn.options.timeout = 500
-      conn.options.open_timeout = 10
-      conn.headers[:user_agent] = "stanford-library-sul-pub"
-      conn.headers[:authorization] = token
-      conn
+    conn = Faraday.new(url: config.base_url) do |faraday|
+      faraday.request :retry, max: 3,
+        interval: 0.5,
+        interval_randomness: 0.5,
+        backoff_factor: 2
     end
-  end
-
-  def token
-    client = OAuth2::Client.new(client_id, client_secret, site: base_url,
-      token_url: "/api/oauth/token", authorize_url: "/api/oauth/authorize",
-      auth_scheme: :request_body)
-    token = client.client_credentials.get_token
-    "Bearer #{token.token}"
+    conn.options.timeout = 500
+    conn.options.open_timeout = 10
+    conn.headers[:user_agent] = "stanford-library-sul-pub"
+    conn.headers[:authorization] = "Bearer #{config.token}"
+    conn
   end
 
   # @param [string] orcidid which can include a full URI, e.g. "https://sandbox.orcid.org/0000-0002-7262-6251"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mais_orcid_client
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors:
 - Peter Mangiafico
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-07-31 00:00:00.000000000 Z
+date: 2023-08-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -217,6 +217,9 @@ files:
 - README.md
 - Rakefile
 - lib/mais_orcid_client.rb
+- lib/mais_orcid_client/authenticator.rb
+- lib/mais_orcid_client/token_wrapper.rb
+- lib/mais_orcid_client/unexpected_response.rb
 - lib/mais_orcid_client/version.rb
 - mais_orcid_client.gemspec
 homepage: https://github.com/sul-dlss/mais_orcid_client
@@ -241,7 +244,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.10
+rubygems_version: 3.4.13
 signing_key:
 specification_version: 4
 summary: Interface for interacting with the MAIS's ORCID API.