doorkeeper-mongodb 5.2.2 → 5.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6ca032c54f4098b2652bde530b4cc204b124d76e5d066ddd04d302302dd854b7
-  data.tar.gz: 24fe8d9ad134bcd6eba2b8f9ae8e76b956d0e5eabe527ecece33c8ed51211049
+  metadata.gz: f2798483ff1a989a1b8cefc2e94771e735c08f52879675c69b1808a912e2f730
+  data.tar.gz: ee24b6384a5378b6353a0a9006648feea656ab9d2c17d5fd596f6bf8637cadc9
 SHA512:
-  metadata.gz: 3ff745bcac86d2cc5c10d8e7548c487fce3626e9570da0f66e1fc47de25bf5f5b323dd650512b14b4e248567e278a66707a1d2af52e0e15f5d9f14af9eda68e8
-  data.tar.gz: 7cdb26333c287aa9731561a4b9e59dd9115a1205be92e6e1a4e2686bcfb5d5d67b77609ec3aeff94a00dc2f96decce40ce7f55d748cc461c709b35a4fa6cedee
+  metadata.gz: 260955b7a8b92403cf62913ee6eea0cd230b7f19cb030d8432a04916c821e392609641fc13f080e097703a1c96a8139df6e1800034e499b342c53940fdf40054
+  data.tar.gz: 3c79d4fdfbc2968716d2940c5f732a3476950cb5bcbc4120b040538fbd246299e398069a81fc5b2a001277af0fe2d4e48dcd08d0a7fc9a972de2e3e0ac82817d

data/README.md

@@ -9,26 +9,15 @@ of doorkeeper-mongodb you are using in: https://github.com/doorkeeper-gem/doorke
 
 ## Installation
 
-`doorkeeper-mongodb` provides [Doorkeeper](https://github.com/doorkeeper-gem/doorkeeper) support to [Mongoid](https://github.com/mongodb/mongoid)
-(2, 3, 4 and 5 for doorkeeper-mongodb `3.0` and 4, 5, 6 and 7 for version `4.0` and higher). To start using it, add
-to your Gemfile:
+`doorkeeper-mongodb` provides [Doorkeeper](https://github.com/doorkeeper-gem/doorkeeper) support
+for [Mongoid](https://github.com/mongodb/mongoid) versions 6 and later. Earlier versions of Mongoid
+are supported on earlier versions of `doorkeeper-mongodb`.
 
-``` ruby
-# For Doorkeeper >= 5.2
-gem 'doorkeeper', '~> 5.2'
-gem 'doorkeeper-mongodb', '~> 5.2'
+To start using it, add both `doorkeeper` and `doorkeeper-mongodb` to your Gemfile:
 
-# For Doorkeeper >= 5.0
-gem 'doorkeeper', '~> 5.0'
-gem 'doorkeeper-mongodb', '~> 5.0'
-
-# For Doorkeeper >= 4.4 && < 5.0
-gem 'doorkeeper', '~> 4.4'
-gem 'doorkeeper-mongodb', '~> 4.2'
-
-# For Doorkeeper < 4.4
-gem 'doorkeeper', '~> 4.3'
-gem 'doorkeeper-mongodb', '~> 4.1.0'
+```ruby
+gem 'doorkeeper'
+gem 'doorkeeper-mongodb'
 
 # or if you want to use cutting edge version:
 # gem 'doorkeeper-mongodb', github: 'doorkeeper-gem/doorkeeper-mongodb'
@@ -36,7 +25,9 @@ gem 'doorkeeper-mongodb', '~> 4.1.0'
 
 Run [doorkeeper’s installation generator]:
 
-    rails generate doorkeeper:install
+```bash
+$ rails generate doorkeeper:install
+```
 
 [doorkeeper’s installation generator]: https://github.com/doorkeeper-gem/doorkeeper#installation
 
@@ -45,20 +36,14 @@ This will install the doorkeeper initializer into
 
 Set the ORM configuration:
 
-``` ruby
+```ruby
 Doorkeeper.configure do
   orm :mongoid7 # or any other version of mongoid
 end
 ```
 
-## MongoMapper
-
-**NOTE**: Support of MongoMapper gem was dropped.
-
 ## Indexes
 
-### Mongoid
-
 Make sure you create indexes for doorkeeper models. You can do this either by
 running `rake db:mongoid:create_indexes` or (if you're using Mongoid 2) by
 adding `autocreate_indexes: true` to your `config/mongoid.yml`
@@ -71,7 +56,7 @@ variables defined in `.travis.yml` file.
 
 To run locally, you need to choose a gemfile, with a command similar to:
 
-```
+```bash
 $ export RAILS=5.1
 $ export BUNDLE_GEMFILE=$PWD/gemfiles/Gemfile.mongoid6.rb
 ```

data/lib/doorkeeper/orm/concerns/mongoid/ownership.rb

@@ -13,7 +13,7 @@ module Doorkeeper
             belongs_to_options[:optional] = true if ::Mongoid::VERSION[0].to_i >= 6
 
             belongs_to :owner, belongs_to_options
-            validates :owner, presence: true, if: :validate_owner?
+            validates_presence_of :owner, if: :validate_owner?
           end
 
           def validate_owner?

data/lib/doorkeeper/orm/concerns/mongoid/resource_ownerable.rb

@@ -19,7 +19,7 @@ module Doorkeeper
             #
             def by_resource_owner(resource_owner)
               if DoorkeeperMongodb.doorkeeper_version?(5, 3) &&
-                Doorkeeper.configuration.try(:polymorphic_resource_owner?)
+                 Doorkeeper.configuration.try(:polymorphic_resource_owner?)
                 where(resource_owner: resource_owner)
               else
                 where(resource_owner_id: resource_owner_id_for(resource_owner))

data/lib/doorkeeper/orm/mongoid4.rb

@@ -5,7 +5,22 @@ require "active_support/lazy_load_hooks"
 module Doorkeeper
   module Orm
     module Mongoid4
+      def self.run_hooks
+        lazy_load do
+          require "doorkeeper/orm/mongoid4/access_grant"
+          require "doorkeeper/orm/mongoid4/access_token"
+          require "doorkeeper/orm/mongoid4/application"
+          require "doorkeeper/orm/mongoid4/stale_records_cleaner"
+          require "doorkeeper/orm/concerns/mongoid/ownership"
+          Doorkeeper::Application.include Doorkeeper::Orm::Concerns::Mongoid::Ownership
+        end
+        @initialized_hooks = true
+      end
+
+      # @deprecated
       def self.initialize_models!
+        return if @initialized_hooks
+
         lazy_load do
           require "doorkeeper/orm/mongoid4/access_grant"
           require "doorkeeper/orm/mongoid4/access_token"
@@ -14,7 +29,10 @@ module Doorkeeper
         end
       end
 
+      # @deprecated
       def self.initialize_application_owner!
+        return if @initialized_hooks
+
         lazy_load do
           require "doorkeeper/orm/concerns/mongoid/ownership"
 

data/lib/doorkeeper/orm/mongoid5.rb

@@ -5,7 +5,22 @@ require "active_support/lazy_load_hooks"
 module Doorkeeper
   module Orm
     module Mongoid5
+      def self.run_hooks
+        lazy_load do
+          require "doorkeeper/orm/mongoid5/access_grant"
+          require "doorkeeper/orm/mongoid5/access_token"
+          require "doorkeeper/orm/mongoid5/application"
+          require "doorkeeper/orm/mongoid5/stale_records_cleaner"
+          require "doorkeeper/orm/concerns/mongoid/ownership"
+          Doorkeeper::Application.include Doorkeeper::Orm::Concerns::Mongoid::Ownership
+        end
+        @initialized_hooks = true
+      end
+
+      # @deprecated
       def self.initialize_models!
+        return if @initialized_hooks
+
         lazy_load do
           require "doorkeeper/orm/mongoid5/access_grant"
           require "doorkeeper/orm/mongoid5/access_token"
@@ -14,7 +29,10 @@ module Doorkeeper
         end
       end
 
+      # @deprecated
       def self.initialize_application_owner!
+        return if @initialized_hooks
+
         lazy_load do
           require "doorkeeper/orm/concerns/mongoid/ownership"
 

data/lib/doorkeeper/orm/mongoid6.rb

@@ -5,7 +5,22 @@ require "active_support/lazy_load_hooks"
 module Doorkeeper
   module Orm
     module Mongoid6
+      def self.run_hooks
+        lazy_load do
+          require "doorkeeper/orm/mongoid6/access_grant"
+          require "doorkeeper/orm/mongoid6/access_token"
+          require "doorkeeper/orm/mongoid6/application"
+          require "doorkeeper/orm/mongoid6/stale_records_cleaner"
+          require "doorkeeper/orm/concerns/mongoid/ownership"
+          Doorkeeper::Application.include Doorkeeper::Orm::Concerns::Mongoid::Ownership
+        end
+        @initialized_hooks = true
+      end
+
+      # @deprecated
       def self.initialize_models!
+        return if @initialized_hooks
+
         lazy_load do
           require "doorkeeper/orm/mongoid6/access_grant"
           require "doorkeeper/orm/mongoid6/access_token"
@@ -14,7 +29,10 @@ module Doorkeeper
         end
       end
 
+      # @deprecated
       def self.initialize_application_owner!
+        return if @initialized_hooks
+
         lazy_load do
           require "doorkeeper/orm/concerns/mongoid/ownership"
 

data/lib/doorkeeper/orm/mongoid7.rb

@@ -5,7 +5,22 @@ require "active_support/lazy_load_hooks"
 module Doorkeeper
   module Orm
     module Mongoid7
+      def self.run_hooks
+        lazy_load do
+          require "doorkeeper/orm/mongoid7/access_grant"
+          require "doorkeeper/orm/mongoid7/access_token"
+          require "doorkeeper/orm/mongoid7/application"
+          require "doorkeeper/orm/mongoid7/stale_records_cleaner"
+          require "doorkeeper/orm/concerns/mongoid/ownership"
+          Doorkeeper::Application.include Doorkeeper::Orm::Concerns::Mongoid::Ownership
+        end
+        @initialized_hooks = true
+      end
+
+      # @deprecated
       def self.initialize_models!
+        return if @initialized_hooks
+
         lazy_load do
           require "doorkeeper/orm/mongoid7/access_grant"
           require "doorkeeper/orm/mongoid7/access_token"
@@ -14,7 +29,10 @@ module Doorkeeper
         end
       end
 
+      # @deprecated
       def self.initialize_application_owner!
+        return if @initialized_hooks
+
         lazy_load do
           require "doorkeeper/orm/concerns/mongoid/ownership"
 

data/lib/doorkeeper-mongodb/mixins/mongoid/access_grant_mixin.rb

@@ -36,8 +36,9 @@ module DoorkeeperMongodb
             belongs_to :resource_owner, polymorphic: true
           end
 
-          validates :resource_owner_id, :application_id, :token, :expires_in, :redirect_uri, presence: true
-          validates :token, uniqueness: true
+          validates_presence_of :resource_owner_id, :application_id, :token,
+                                :expires_in, :redirect_uri
+          validates_uniqueness_of :token
 
           before_validation :generate_token, on: :create
         end
@@ -123,15 +124,14 @@ module DoorkeeperMongodb
 
           # @param code_verifier [#to_s] a one time use value (any object that responds to `#to_s`)
           #
-          # @return [#to_s] An encoded code challenge based on the provided verifier suitable for PKCE validation
+          # @return [#to_s] An encoded code challenge based on the provided verifier suitable
+          #   for PKCE validation
           def generate_code_challenge(code_verifier)
             padded_result = Base64.urlsafe_encode64(Digest::SHA256.digest(code_verifier))
             padded_result.split("=")[0] # Remove any trailing '='
           end
 
-          def pkce_supported?
-            new.pkce_supported?
-          end
+          delegate :pkce_supported?, to: :new
 
           ##
           # Determines the secret storing transformer

data/lib/doorkeeper-mongodb/mixins/mongoid/access_token_mixin.rb

@@ -41,8 +41,9 @@ module DoorkeeperMongodb
             belongs_to :resource_owner, opts
           end
 
-          validates :token, presence: true, uniqueness: true
-          validates :refresh_token, uniqueness: true, if: :use_refresh_token?
+          validates_presence_of :token
+          validates_uniqueness_of :token
+          validates_uniqueness_of :refresh_token, if: :use_refresh_token?
 
           # @attr_writer [Boolean, nil] use_refresh_token
           #   indicates the possibility of using refresh token
@@ -368,10 +369,10 @@ module DoorkeeperMongodb
         # and clears `:previous_refresh_token` attribute.
         #
         def revoke_previous_refresh_token!
-          return unless self.class.refresh_token_revoked_on_use?
+          return if !self.class.refresh_token_revoked_on_use? || previous_refresh_token.blank?
 
           old_refresh_token&.revoke
-          update(previous_refresh_token: "")
+          update_attribute(:previous_refresh_token, "")
         end
 
         private
@@ -419,7 +420,8 @@ module DoorkeeperMongodb
 
           return generator if generator.respond_to?(:generate)
 
-          raise Doorkeeper::Errors::UnableToGenerateToken, "#{generator} does not respond to `.generate`."
+          raise Doorkeeper::Errors::UnableToGenerateToken,
+                "#{generator} does not respond to `.generate`."
         rescue NameError
           raise Doorkeeper::Errors::TokenGeneratorNotFound, "#{generator_name} not found"
         end

data/lib/doorkeeper-mongodb/mixins/mongoid/application_mixin.rb

@@ -36,8 +36,8 @@ module DoorkeeperMongodb
           has_many :access_grants, has_many_options.merge(class_name: access_grants_class_name)
           has_many :access_tokens, has_many_options.merge(class_name: access_tokens_class_name)
 
-          validates :name, :secret, :uid, presence: true
-          validates :uid, uniqueness: true
+          validates_presence_of :name, :secret, :uid
+          validates_uniqueness_of :uid
 
           # Before Doorkeeper 5.2.3
           if defined?(::RedirectUriValidator)
@@ -46,7 +46,7 @@ module DoorkeeperMongodb
             validates :redirect_uri, "doorkeeper/redirect_uri": true
           end
 
-          validates :confidential, inclusion: { in: [true, false] }
+          validates_inclusion_of :confidential, in: [true, false]
 
           validate :scopes_match_configured, if: :enforce_scopes?
 
@@ -242,7 +242,7 @@ module DoorkeeperMongodb
         end
 
         def generate_secret
-          return unless secret.blank?
+          return if secret.present?
 
           @raw_secret = UniqueToken.generate
           secret_strategy.store_secret(self, :secret, @raw_secret)
@@ -250,7 +250,8 @@ module DoorkeeperMongodb
 
         def scopes_match_configured
           if scopes.present? &&
-             !ScopeChecker.valid?(scope_str: scopes.to_s, server_scopes: Doorkeeper.configuration.scopes)
+             !ScopeChecker.valid?(scope_str: scopes.to_s,
+                                  server_scopes: Doorkeeper.configuration.scopes,)
             errors.add(:scopes, :not_match_configured)
           end
         end

data/lib/doorkeeper-mongodb/version.rb

@@ -9,7 +9,7 @@ module DoorkeeperMongodb
     # Semver
     MAJOR = 5
     MINOR = 2
-    TINY = 2
+    TINY = 3
 
     # Full version number
     STRING = [MAJOR, MINOR, TINY].compact.join(".")

data/spec/dummy/app/models/user.rb

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-# [NOTE] MongoMapper support was dropped, so now only Mongoid
 class User
   include Mongoid::Document
   include Mongoid::Timestamps

data/spec/dummy/config/mongo.yml

@@ -4,8 +4,8 @@ defaults: &defaults
 
 development:
   <<: *defaults
-  database: doorkeeper-mongomapper-development
+  database: doorkeeper-mongoid-development
 
 test:
   <<: *defaults
-  database: doorkeeper-mongomapper-test-suite
+  database: doorkeeper-mongoid-test