When using User#send_reset_password_instructions, redirect_url should…

… be set to default url if not present in opts
lynndylanhurley · Jan 24, 2019 · 552a753 · 552a753
1 parent 1254483
commit 552a753
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/models/devise_token_auth/concerns/user.rb b/app/models/devise_token_auth/concerns/user.rb
@@ -69,6 +69,7 @@ def send_reset_password_instructions(opts = {})
 
       # fall back to "default" config name
       opts[:client_config] ||= 'default'
+      opts[:redirect_url] ||= DeviseTokenAuth.default_password_reset_url
 
       send_devise_notification(:reset_password_instructions, token, opts)
       token

diff --git a/test/controllers/devise_token_auth/passwords_controller_test.rb b/test/controllers/devise_token_auth/passwords_controller_test.rb
@@ -330,8 +330,7 @@ class DeviseTokenAuth::PasswordsControllerTest < ActionController::TestCase
           DeviseTokenAuth.default_password_reset_url = @redirect_url
 
           post :create,
-               params: { email: @resource.email,
-                         redirect_url: @redirect_url }
+               params: { email: @resource.email }
 
           @mail = ActionMailer::Base.deliveries.last
           @resource.reload