searls-auth 1.0.0 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '09b9515ef89e82e747cc35c415a4de145cf8e61634caccb137ceb8f429a54bdc'
-  data.tar.gz: e5b456ac99cc9ad784ed8727266dc3e073e6ee850ad720be0a4a42e894e2bbe8
+  metadata.gz: 4314fcd2c4eee5b0f32ace811ceeefaa44728895592fdebd5aa6a696303133e7
+  data.tar.gz: 8d4963fb8be2ecbf4ad13762e8697c2d9d992f9d382dcc0c73b2a38d6daf6041
 SHA512:
-  metadata.gz: 6c5163daa1551674b1c128ecf8a6d155cde36123780bf2332ef5d9f79166d37a964616f6fda17c97eb2608df9d13ad54f6f25e5c384ef2308b8740f93e326302
-  data.tar.gz: 4bfdd8c989591a52aeb52cf91bbf55e189442e57031d877f0e81f88c45d51d8cd22bd8ca21f31be10370c68b3981e410b2a034ed6c69738b28ee10cfcca078d0
+  metadata.gz: 79c05ebe302bc1846f72ac29ae21a16ee629050a52b31cb4bd9ff4d5ec70bd1737e4cfe7451546cf74dd0aa82781d0952855218f4f47286419049f59265e6982
+  data.tar.gz: a309ca56baf772d1c3eb46adf1897d33af4bb0077d14768e9646b03b402311cb9793082f9f16fdc9ac27286a001561843e338858eda1a99a960e7b331268d85c

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 ## [Unreleased]
 
+## [1.0.2] - 2025-10-13
+
+* Ensure password resets will be delivered if user does not have a password
+
+## [1.0.1] - 2025-10-11
+
+* Skip database-aware configuration validations when Active Record migrations are pending
+
 ## [1.0.0] - 2025-10-04
 
 * **BREAKING:** Rename `default_redirect_path_after_register` to `redirect_path_after_register`

data/app/controllers/searls/auth/requests_password_resets_controller.rb

@@ -12,7 +12,7 @@ module Searls
         email = params[:email].to_s.strip
         user = Searls::Auth.config.user_finder_by_email.call(email)
 
-        if proceed_with_password_reset_request?(user) && deliverable_user?(user)
+        if proceed_with_password_reset_request?(user)
           Searls::Auth::DeliversPasswordReset.new.deliver(
             user:,
             redirect_path: params[:redirect_path],
@@ -41,12 +41,9 @@ module Searls
         nil
       end
 
-      def deliverable_user?(user)
+      def proceed_with_password_reset_request?(user)
         return false if user.blank?
-        Searls::Auth.config.password_present?(user)
-      end
 
-      def proceed_with_password_reset_request?(user)
         result = Searls::Auth.config.before_password_reset.call(user, params, self)
         !(result == false || result == :halt)
       end

data/lib/searls/auth/config.rb

@@ -165,9 +165,12 @@ module Searls
         if using_default_hooks && defined?(::User)
           missing = []
           missing << "User#authenticate" unless ::User.method_defined?(:authenticate)
-          has_password_digest_method = ::User.method_defined?(:password_digest)
-          has_password_digest_column = ::User.respond_to?(:column_names) && ::User.column_names.include?("password_digest")
-          missing << "users.password_digest" unless has_password_digest_method || has_password_digest_column
+          unless schema_checks_blocked?
+            has_password_digest_method = ::User.method_defined?(:password_digest)
+            has_password_digest_column = ::User.respond_to?(:column_names) && ::User.column_names.include?("password_digest")
+            missing << "users.password_digest" unless has_password_digest_method || has_password_digest_column
+          end
+
           if missing.any?
             raise Searls::Auth::Error, "Password login requires #{missing.join(" and ")}. Add bcrypt/has_secure_password or override password hooks."
           end
@@ -255,7 +258,7 @@ module Searls
           end
           has_email_method = ::User.method_defined?(:email)
           has_email_column = ::User.respond_to?(:column_names) && ::User.column_names.include?("email")
-          unless has_email_method || has_email_column
+          unless schema_checks_blocked? || has_email_method || has_email_column
             raise Searls::Auth::Error, "Default :user_finder_by_email expects a `users.email` attribute."
           end
         end
@@ -274,7 +277,7 @@ module Searls
             probe = ::User.new
             has_email_setter = probe.respond_to?(:email=)
             has_email_column = ::User.respond_to?(:column_names) && ::User.column_names.include?("email")
-            unless has_email_setter || has_email_column
+            unless schema_checks_blocked? || has_email_setter || has_email_column
               raise Searls::Auth::Error, "Default :user_initializer expects a writable email attribute on User."
             end
           rescue ArgumentError
@@ -289,6 +292,19 @@ module Searls
           end
         end
       end
+
+      def schema_checks_blocked?
+        return true unless defined?(::ActiveRecord)
+        begin
+          if !::ActiveRecord::Base.connected? ||
+              !::ActiveRecord::Base.connection&.migration_context.present? ||
+              ::ActiveRecord::Base.connection.migration_context.needs_migration?
+            true
+          end
+        rescue
+          true
+        end
+      end
     end
   end
 end

data/lib/searls/auth/parses_time_safely.rb

@@ -14,8 +14,6 @@ module Searls
         else
           if input.respond_to?(:in_time_zone)
             input.in_time_zone
-          else
-            parse_string(input.to_s)
           end
         end
       rescue ArgumentError, TypeError, NoMethodError

data/lib/searls/auth/version.rb

@@ -1,5 +1,5 @@
 module Searls
   module Auth
-    VERSION = "1.0.0"
+    VERSION = "1.0.2"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: searls-auth
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.2
 platform: ruby
 authors:
 - Justin Searls