Devise更新邮箱确认后未同步至email字段问题排查

问题根因 #

你重写的Devise Confirmable 模块逻辑存在设计错误，是邮箱更新确认失败的核心原因：

具体问题点 #

• 在布尔判断方法中修改实例属性
  原生Devise的pending_reconfirmation?仅做状态判断，不会修改模型属性。你的实现直接在该方法内给self.email赋值、清空unconfirmed_email，而该方法会在多个场景被调用，一旦调用后，不管后续保存是否成功，内存中实例的unconfirmed_email已经被清空，无法还原。
• 邮箱校验失败导致保存不生效
  你在saved方法中处理重确认逻辑时开启了validate: true，如果新邮箱存在格式错误、已被其他账号占用等违反模型校验规则的情况，save!会抛出异常，此时数据库不会更新，但内存中的属性已经被修改，最终表现为unconfirmed_email没有同步到正式email字段。
• 手机号更新正常的原因
  手机号的校验规则通常更简单，没有触发校验失败的场景，刚好能保存成功，所以没有暴露出逻辑缺陷。

修复方案 #

1. 首先修正pending_reconfirmation?方法，仅保留判断逻辑，不要修改属性：

def pending_reconfirmation?
  unconfirmed_email.present? || unconfirmed_mobile.present?
end

2. 将属性迁移逻辑移到保存前的环节，修改saved方法：

def saved(args)
  @saved ||= if pending_reconfirmation?
               skip_reconfirmation!
               # 仅在确认要保存时才修改属性
               if unconfirmed_email.present?
                 self.email = unconfirmed_email
                 self.unconfirmed_email = nil
               elsif unconfirmed_mobile.present?
                 self.mobile_number = unconfirmed_mobile
                 self.unconfirmed_mobile = nil
               end
               save!(validate: true)
             else
               save!(validate: args[:ensure_valid] == true)
             end
end

3. 排查调用confirm方法的上层代码，增加异常捕获逻辑，或者将save!改为save并判断返回值，校验失败时返回错误信息给前端。你可以在控制台调用confirm后执行puts user.errors.full_messages查看具体的校验失败原因。

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