Rails自定义Devise重置密码控制器：表单取值与邮件功能异常排查

我最近在开发Ruby on Rails项目，需要自定义Devise的Recoverable控制器，实现满足特定条件的用户接收不同重置密码邮件的功能，但当前代码无法正常运行，求帮忙调试。

Devise视图代码 #

<%= form_for(resource, as: resource_name, url: password_path(resource_name), html: { method: :post }) do |f| %>
  <%= render "devise/shared/error_messages", resource: resource %>

  <div class="field">
    <p><%= f.label :email %></p>
    <p><%= f.email_field :email, autofocus: true, autocomplete: "email" %></p>
  </div>

  <div class="actions">
    <%= f.submit "Send me password reset instructions" %>
  </div>
<% end %>

自定义控制器代码 #

# POST /resource/password
def create
   @user = User.find_by_email(params[:email])
   if @user.some_condition?
     PasswordMailer.with(user: @user).reset.deliver_now
     redirect_to '/', notice: 'If an account with that email was found, we have sent a link to reset password'
   else
     super
   end
end

问题排查与修正方向 #

• 参数获取错误：Devise表单提交的参数嵌套在resource_name（通常是:user）下，原代码params[:email]无法拿到值，应改为params[:user][:email]。
• 控制器继承与路由配置：自定义控制器必须继承Devise::PasswordsController，同时在routes.rb里指定Devise使用该控制器：

  devise_for :users, controllers: { passwords: 'passwords' }
  

• 用户不存在的边界处理：原代码未处理@user为nil的情况，调用some_condition?会抛异常，建议用@user&.some_condition?做安全调用。
• 密码重置令牌缺失：手动发邮件时未生成重置令牌，用户点击链接会无效，需先为用户生成并保存令牌：

  @user.reset_password_token = User.generate_reset_password_token
  @user.reset_password_sent_at = Time.now.utc
  @user.save(validate: false)
  

• 邮件发送验证：检查PasswordMailer的reset方法是否正确实现、邮件模板是否存在，开发环境可使用letter_opener预览邮件确认逻辑。

修正后的控制器示例 #

class PasswordsController < Devise::PasswordsController
  # POST /resource/password
  def create
    @user = User.find_by(email: params[:user][:email])
    if @user&.some_condition?
      # 生成并保存密码重置令牌
      @user.reset_password_token = User.generate_reset_password_token
      @user.reset_password_sent_at = Time.now.utc
      @user.save(validate: false)
      
      PasswordMailer.with(user: @user).reset.deliver_now
      redirect_to root_path, notice: '如果该邮箱对应的账户存在，我们已发送密码重置链接'
    else
      super
    end
  end
end

内容的提问来源于stack exchange，提问作者Yuan Ma