rodauth-rails 0.18.1 → 1.0.0

data/lib/generators/rodauth/templates/app/views/rodauth/webauthn_remove.html.erb

@@ -1,6 +1,6 @@
 <% content_for :title, rodauth.webauthn_remove_page_title %>
 
-<%= form_with url: rodauth.webauthn_remove_path, method: :post, id: "webauthn-remove-form" do |form| %>
+<%= form_with url: rodauth.webauthn_remove_path, method: :post, id: "webauthn-remove-form", data: { turbo: false } do |form| %>
   <% if rodauth.two_factor_modifications_require_password? %>
     <div class="form-group mb-3">
       <%= form.label "password", rodauth.password_label, class: "form-label" %>

data/lib/generators/rodauth/templates/app/views/rodauth/webauthn_setup.html.erb

@@ -2,7 +2,7 @@
 
 <% cred = rodauth.new_webauthn_credential %>
 
-<%= form_with url: rodauth.webauthn_setup_path, method: :post, id: "webauthn-setup-form", data: { credential_options: cred.as_json.to_json } do |form| %>
+<%= form_with url: rodauth.webauthn_setup_path, method: :post, id: "webauthn-setup-form", data: { credential_options: cred.as_json.to_json, turbo: false } do |form| %>
   <%= form.hidden_field rodauth.webauthn_setup_challenge_param, value: cred.challenge %>
   <%= form.hidden_field rodauth.webauthn_setup_challenge_hmac_param, value: rodauth.compute_hmac(cred.challenge) %>
   <%= form.text_field rodauth.webauthn_setup_param, value: "", id: "webauthn-setup", aria: { hidden: "true" } %>

data/lib/rodauth/rails/app.rb

@@ -10,7 +10,6 @@ module Rodauth
 
       plugin :hooks
       plugin :render, layout: false
-      plugin :pass
 
       unless Rodauth::Rails.api_only?
         require "rodauth/rails/app/flash"
@@ -25,9 +24,7 @@ module Rodauth
 
         auth_class ||= Class.new(Rodauth::Rails::Auth)
 
-        plugin :rodauth, auth_class: auth_class, name: name, csrf: false, flash: false, json: true, **options do
-          instance_exec(&block) if block
-        end
+        plugin :rodauth, auth_class: auth_class, name: name, csrf: false, flash: false, json: true, **options, &block
       end
 
       before do

data/lib/rodauth/rails/auth.rb

@@ -3,23 +3,8 @@ require "rodauth/rails/feature"
 
 module Rodauth
   module Rails
-    # Base auth class that applies some default configuration and supports
-    # multi-level inheritance.
+    # Base auth class that applies some changes to the default configuration.
     class Auth < Rodauth::Auth
-      def self.inherited(subclass)
-        super
-        superclass = self
-        subclass.class_eval do
-          @roda_class = Rodauth::Rails.app
-          @features = superclass.features.clone
-          @routes = superclass.routes.clone
-          @route_hash = superclass.route_hash.clone
-          @configuration = superclass.instance_variable_get(:@configuration).clone
-          @configuration.instance_variable_set(:@auth, self)
-        end
-      end
-
-      # apply default configuration
       configure do
         enable :rails
 

data/lib/rodauth/rails/controller_methods.rb

@@ -32,7 +32,7 @@ module Rodauth
           rescue ActiveRecord::RecordNotFound
             not_found.call
           end
-        elsif model < Sequel::Model
+        elsif defined?(Sequel::Model) && model < Sequel::Model
           begin
             model.with_pk!(id)
           rescue Sequel::NoMatchingRow

data/lib/rodauth/rails/feature/internal_request.rb

@@ -5,16 +5,17 @@ module Rodauth
         def domain
           return super unless missing_host?
 
-          Rodauth::Rails.url_options[:host]
+          rails_url_options.fetch(:host)
         end
 
         def base_url
           return super unless missing_host? && domain
 
-          url_options = Rodauth::Rails.url_options
+          scheme = rails_url_options[:protocol] || "http"
+          port = rails_url_options[:port]
 
-          url = "#{url_options[:protocol]}://#{domain}"
-          url << ":#{url_options[:port]}" if url_options[:port]
+          url = "#{scheme}://#{domain}"
+          url << ":#{port}" if port
           url
         end
 
@@ -40,6 +41,11 @@ module Rodauth
         def missing_host?
           internal_request? && request.host == INVALID_DOMAIN || scope.nil?
         end
+
+        def rails_url_options
+          ::Rails.application.config.action_mailer.default_url_options or
+            fail Error, "There is no information to set the URL host from. Please set config.action_mailer.default_url_options in your Rails application, or configure #domain and #base_url in your Rodauth configuration."
+        end
       end
     end
   end

data/lib/rodauth/rails/feature/render.rb

@@ -42,6 +42,14 @@ module Rodauth
           controller.formats = rails_request.formats.map(&:ref).compact
           controller
         end
+
+        # Not all Rodauth actions are Turbo-compatible (some form submissions
+        # render 200 HTML responses), so we disable Turbo on all Rodauth forms.
+        def _view(meth, *)
+          html = super
+          html = html.gsub(/<form(.+)>/, '<form\1 data-turbo="false">') if meth == :view
+          html
+        end
       end
     end
   end

data/lib/rodauth/rails/tasks.rake

@@ -5,13 +5,13 @@ namespace :rodauth do
     puts "Routes handled by #{app}:"
 
     app.opts[:rodauths].each do |configuration_name, auth_class|
-      auth_class.configure { enable :path_class_methods }
+      rodauth = auth_class.allocate
 
       routes = auth_class.routes.map do |handle_method|
         path_method = "#{handle_method.to_s.sub(/\Ahandle_/, "")}_path"
 
         [
-          auth_class.public_send(path_method),
+          rodauth.public_send(path_method),
           "rodauth#{configuration_name && "(:#{configuration_name})"}.#{path_method}",
         ]
       end

data/lib/rodauth/rails/version.rb

@@ -1,5 +1,5 @@
 module Rodauth
   module Rails
-    VERSION = "0.18.1"
+    VERSION = "1.0.0"
   end
 end

data/lib/rodauth/rails.rb

@@ -14,37 +14,32 @@ module Rodauth
     @app = nil
     @middleware = true
 
-    LOCK = Mutex.new
-
     class << self
       def rodauth(name = nil, query: nil, form: nil, account: nil, **options)
         auth_class = app.rodauth!(name)
 
-        LOCK.synchronize do
-          unless auth_class.features.include?(:internal_request)
-            auth_class.configure { enable :internal_request }
-            warn "Rodauth::Rails.rodauth requires the internal_request feature to be enabled. For now it was enabled automatically, but this behaviour will be removed in version 1.0."
-          end
-        end
-
-        if query || form
-          warn "The :query and :form keyword arguments for Rodauth::Rails.rodauth have been deprecated. Please use the :params argument supported by internal_request feature instead."
-          options[:params] = query || form
+        unless auth_class.features.include?(:internal_request)
+          fail Rodauth::Rails::Error, "Rodauth::Rails.rodauth requires internal_request feature to be enabled"
         end
 
         if account
           options[:account_id] = account.id
         end
 
-        auth_class.internal_request_eval(options) do
+        instance = auth_class.internal_request_eval(options) do
           if defined?(ActiveRecord::Base) && account.is_a?(ActiveRecord::Base)
             @account = account.attributes.symbolize_keys
           elsif defined?(Sequel::Model) && account.is_a?(Sequel::Model)
             @account = account.values
           end
-
           self
         end
+
+        # clean up inspect output
+        instance.remove_instance_variable(:@internal_request_block)
+        instance.remove_instance_variable(:@internal_request_return_value)
+
+        instance
       end
 
       def model(name = nil, **options)
@@ -80,12 +75,6 @@ module Rodauth
         end
       end
 
-      def url_options
-        options = ::Rails.application.config.action_mailer.default_url_options || {}
-        options[:protocol] ||= "http"
-        options
-      end
-
       def configure
         yield self
       end

data/rodauth-rails.gemspec

@@ -17,7 +17,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_dependency "railties", ">= 4.2", "< 8"
-  spec.add_dependency "rodauth", "~> 2.15"
+  spec.add_dependency "rodauth", "~> 2.19"
   spec.add_dependency "sequel-activerecord_connection", "~> 1.1"
   spec.add_dependency "tilt"
   spec.add_dependency "bcrypt"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rodauth-rails
 version: !ruby/object:Gem::Version
-  version: 0.18.1
+  version: 1.0.0
 platform: ruby
 authors:
 - Janko Marohnić
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-12-16 00:00:00.000000000 Z
+date: 2021-12-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties
@@ -36,14 +36,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.15'
+        version: '2.19'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.15'
+        version: '2.19'
 - !ruby/object:Gem::Dependency
   name: sequel-activerecord_connection
   requirement: !ruby/object:Gem::Requirement
@@ -173,9 +173,11 @@ files:
 - lib/generators/rodauth/migration/webauthn.erb
 - lib/generators/rodauth/migration_generator.rb
 - lib/generators/rodauth/migration_helpers.rb
+- lib/generators/rodauth/templates/INSTRUCTIONS
 - lib/generators/rodauth/templates/app/controllers/rodauth_controller.rb
-- lib/generators/rodauth/templates/app/lib/rodauth_app.rb
 - lib/generators/rodauth/templates/app/mailers/rodauth_mailer.rb
+- lib/generators/rodauth/templates/app/misc/rodauth_app.rb
+- lib/generators/rodauth/templates/app/misc/rodauth_main.rb
 - lib/generators/rodauth/templates/app/models/account.rb
 - lib/generators/rodauth/templates/app/views/rodauth/_email_auth_request_form.html.erb
 - lib/generators/rodauth/templates/app/views/rodauth/_login_form.html.erb