zaikio-oauth_client 0.18.1 → 0.19.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 25dbc767b924d59fa2706f4d7da22fd940ff80da966856447b8027e1ebca986c
-  data.tar.gz: 0a7dbef1ae4c475583c5958ce4320285ca32532b7f55cc7a2b33386109657e2d
+  metadata.gz: 34a7bf1aa494d16086807798f3f15a3634571b485afae332ec57e7871b61dbd6
+  data.tar.gz: 3ee46611f92593cdf2cfd60403fe08f0d5401aca3217b0d063e040d6583a95c0
 SHA512:
-  metadata.gz: f7c542585a0f6d9b47736f587f3b4f75b4544c439f23da8013c67d2f5d27d081c649e7c62985d787956bc04363c0d27315a6cdeb6085c8d2d51b30e022f5e55c
-  data.tar.gz: 2f418622e4ff38e2f2e3f0b9550ffc170a0278584cc2261012a2632c5e7d7de6ca472f73e0a2720e11dcc8129663377772e692085ffc28cd13cac0573e3f9d4b
+  metadata.gz: b81867f191fd382ab76e06c2e3f4d7c7e76ad351ef10bd477f4eb352a7bf9545e677fdaefb7b83c6cdf3706caa86b095da8fbcd610ef9fae851c78c393312fbf
+  data.tar.gz: 66de7443b83d3976aa42216969809ccb88c7e0d1e4c22c1c7a43bb59aad4c30b87e44fc41f3a72bdab4ecdb91ba1c4789676e97b48d40a5f03cc72a7555d72e2

data/README.md

@@ -145,12 +145,19 @@ redirect_to zaikio_oauth_client.new_subscription_path(plan: "free")
 
 #### Session handling
 
-The Zaikio gem engine will set a cookie for the user after a successful OAuth flow: `session[:zaikio_person_id]`.
+The Zaikio gem engine will set a cookie for the access token after a successful OAuth flow: `session[:zaikio_access_token_id]`.
 
 If you are using for example `Zaikio::Hub::Models`, you can use this snippet to set the current user:
 
 ```ruby
-Current.user ||= Zaikio::Hub::Models::Person.find_by(id: session[:zaikio_person_id])
+access_token = Zaikio::OAuthClient.find_active_access_token(session[:zaikio_access_token_id])
+session[:zaikio_access_token_id] = access_token&.id
+Current.user = Zaikio::Hub::Models::Person.find_by(id: access_token&.bearer_id)
+
+unless Current.user
+  session[:origin] = request.fullpath
+  redirect_to zaikio_oauth_client.new_session_path
+end
 ````
 
 You can then use `Current.user` anywhere.

data/app/models/zaikio/access_token.rb

@@ -9,7 +9,7 @@ module Zaikio
     encrypts :token
     encrypts :refresh_token
 
-    def self.build_from_access_token(access_token, requested_scopes: nil)
+    def self.build_from_access_token(access_token, requested_scopes: nil, include_refresh_token: true)
       payload = JWT.decode(access_token.token, nil, false).first rescue {} # rubocop:disable Style/RescueModifier
       scopes = access_token.params["scope"].split(",")
       new(
@@ -18,7 +18,7 @@ module Zaikio
         bearer_id: access_token.params["bearer"]["id"],
         audience: access_token.params["audiences"].first,
         token: access_token.token,
-        refresh_token: access_token.refresh_token,
+        refresh_token: (access_token.refresh_token if include_refresh_token),
         expires_at: Time.strptime(access_token.expires_at.to_s, "%s"),
         scopes: scopes,
         requested_scopes: requested_scopes || scopes
@@ -67,7 +67,7 @@ module Zaikio
     end
 
     def bearer_klass
-      return unless Zaikio.const_defined?("Hub::Models", false) # rubocop:disable Performance/StringIdentifierArgument
+      return unless Zaikio.const_defined?("Hub::Models", false)
 
       if Zaikio::Hub::Models.configuration.respond_to?(:"#{bearer_type.underscore}_class_name")
         Zaikio::Hub::Models.configuration.public_send(:"#{bearer_type.underscore}_class_name").constantize
@@ -95,5 +95,15 @@ module Zaikio
       destroy
       nil
     end
+
+    def revoke!
+      return unless Zaikio.const_defined?("Hub::RevokedAccessToken", false)
+
+      Zaikio::Hub.with_token(token) do
+        Zaikio::Hub::RevokedAccessToken.create
+      end
+    rescue Zaikio::ConnectionError => e
+      Zaikio::OAuthClient.configuration.logger.warn "Access Token #{id} could not be revoked: #{e.message}"
+    end
   end
 end

data/lib/zaikio/oauth_client/authenticatable.rb

@@ -55,13 +55,16 @@ module Zaikio
       end
 
       def destroy
-        access_token_id = session[:zaikio_access_token_id]
+        if (access_token = Zaikio::AccessToken.valid.or(Zaikio::AccessToken.valid_refresh)
+                                             .find_by(id: session[:zaikio_access_token_id]))
+          access_token.revoke!
+        end
         session.delete(:zaikio_access_token_id)
         session.delete(:origin)
 
         redirect_to send(
           respond_to?(:after_destroy_path_for) ? :after_destroy_path_for : :default_after_destroy_path_for,
-          access_token_id
+          access_token.id
         )
       end
 

data/lib/zaikio/oauth_client/test_helper.rb

@@ -3,6 +3,24 @@ module Zaikio
     module TestHelper
       extend ActiveSupport::Concern
 
+      VirtualAccessToken = Struct.new(:id, :bearer_id, :bearer_type, :audience, :expired?, keyword_init: true)
+
+      class << self
+        def find_active_access_token(id)
+          return unless id.to_s.starts_with?("AT:")
+
+          _, audience, person_id = id.split(":")
+
+          VirtualAccessToken.new(
+            id: id,
+            bearer_id: person_id,
+            audience: audience,
+            bearer_type: "Person",
+            expired?: false
+          )
+        end
+      end
+
       class TestSessionController < ActionController::Base # rubocop:disable Rails/ApplicationController
         def show
           if session[params[:key]].nil?
@@ -43,7 +61,15 @@ module Zaikio
         get "/zaikio/oauth_client/test_helper/session", params: { id: value, key: key }
       end
 
-      def logged_in_as(person)
+      def logged_in_as(person, access_token: nil, client_name: nil)
+        client_name ||= Zaikio::OAuthClient.client_name ||
+                        Zaikio::OAuthClient.configuration.all_client_names.first
+        set_session(
+          :zaikio_access_token_id,
+          access_token&.id || "AT:#{client_name}:#{person.id}"
+        )
+
+        # Deprecated please use zaikio_access_token_id
         set_session(:zaikio_person_id, person.id)
       end
     end

data/lib/zaikio/oauth_client/version.rb

@@ -1,5 +1,5 @@
 module Zaikio
   module OAuthClient
-    VERSION = "0.18.1".freeze
+    VERSION = "0.19.2".freeze
   end
 end

data/lib/zaikio/oauth_client.rb

@@ -6,7 +6,7 @@ require "zaikio/oauth_client/configuration"
 require "zaikio/oauth_client/authenticatable"
 
 module Zaikio
-  module OAuthClient
+  module OAuthClient # rubocop:disable Metrics/ModuleLength
     class << self
       attr_reader :client_name
 
@@ -58,9 +58,8 @@ module Zaikio
         end
       end
 
-      # Finds the best possible access token, using the DB or an API call
-      #   * If the token has expired, it will be refreshed using the refresh_token flow
-      #     (if this fails, we fallback to getting a new token using client_credentials)
+      # Finds active access token, using the DB or Client Credentials flow
+      #   * It searches in the DB for an active access token
       #   * If the token does not exist, we'll get a new one using the client_credentials flow
       def get_access_token(bearer_id:, client_name: nil, bearer_type: "Person", scopes: nil, valid_for: 30.seconds)
         client_config = client_config_for(client_name || self.client_name)
@@ -72,8 +71,6 @@ module Zaikio
                                          requested_scopes: scopes,
                                          valid_for: valid_for)
 
-        token = token.refresh! if token&.expired?
-
         token ||= fetch_new_token(client_config: client_config,
                                   bearer_type: bearer_type,
                                   bearer_id: bearer_id,
@@ -81,21 +78,36 @@ module Zaikio
         token
       end
 
-      # Finds the best usable access token. Note that this token may have expired and
-      # would require refreshing.
-      def find_usable_access_token(client_name:, bearer_type:, bearer_id:, requested_scopes:, valid_for: 30.seconds)  # rubocop:disable Metrics/MethodLength
-        configuration.logger.debug "Try to fetch token for client_name: #{client_name}, "\
+      # This method can be used to find an active access token by id.
+      # It might refresh the access token to get an active one.
+      def find_active_access_token(id)
+        return unless id
+
+        if Rails.env.test?
+          access_token = TestHelper.find_active_access_token(id)
+          return access_token if access_token
+        end
+
+        access_token = Zaikio::AccessToken.find_by(id: id)
+        access_token = access_token.refresh! if access_token&.expired?
+
+        access_token
+      end
+
+      # Finds active access token with matching criteria for bearer and scopes.
+      def find_usable_access_token(client_name:, bearer_type:, bearer_id:, requested_scopes:, valid_for: 30.seconds) # rubocop:disable Metrics/MethodLength
+        configuration.logger.debug "Try to fetch token for client_name: #{client_name}, " \
                                    "bearer #{bearer_type}/#{bearer_id}, requested_scopes: #{requested_scopes}"
 
         fetch_access_token = lambda {
           Zaikio::AccessToken
             .where(audience: client_name)
-            .usable(
+            .by_bearer(
               bearer_type: bearer_type,
               bearer_id: bearer_id,
-              requested_scopes: requested_scopes,
-              valid_until: valid_for.from_now
+              requested_scopes: requested_scopes
             )
+            .valid(valid_for.from_now)
             .first
         }
 
@@ -113,7 +125,10 @@ module Zaikio
             bearer_id: bearer_id,
             scopes: scopes
           ),
-          requested_scopes: scopes
+          requested_scopes: scopes,
+          include_refresh_token: false
+          # Do not store refresh token on client credentials flow
+          # https://docs.zaikio.com/changelog/2022-08-09_client-credentials-drop-refresh-token.html
         ).tap(&:save!)
       end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: zaikio-oauth_client
 version: !ruby/object:Gem::Version
-  version: 0.18.1
+  version: 0.19.2
 platform: ruby
 authors:
 - Zaikio GmbH
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-04-29 00:00:00.000000000 Z
+date: 2022-08-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack