Rails+Devise+Google OAuth2下Promo Code验证失效问题排查

问题原因分析 #

1. Devise的confirm方法强制绕过验证
   在from_omniauth方法中，你先调用user.save触发验证，但如果验证失败（比如promo_code为nil），user.save会返回false，用户不会被持久化。但紧接着执行的user.confirm方法，内部会调用save(validate: false)——这会直接跳过所有验证规则，强制将用户写入数据库，导致你的促销码验证逻辑完全被绕过，最终用户被创建。

2. Promo Code赋值逻辑错误
   代码中if promo_code != nil || promo_code != ""这个条件永远为true：

   • 当promo_code为nil时，promo_code != ""成立；
   • 当promo_code为空字符串时，promo_code != nil成立；
   • 任何其他值时两个条件都成立。
     这导致无论promo_code是否有效，都会被赋值给用户（包括nil的情况），不符合你仅在有效时赋值的预期。

3. 验证上下文定义冗余
   验证规则中on: [:create, :save]的:save不是Rails标准验证上下文，Rails仅默认识别:create和:update。虽然:create会在用户创建时生效，但:save属于无效定义，容易造成逻辑误解。

修复方案 #

1. 调整from_omniauth逻辑，先验证再确认
   仅在user.save验证通过后，再执行user.confirm，避免强制跳过验证：

   def self.from_omniauth(auth, promo_code)
     user = find_by(email: auth.info.email)
     if user
       user.update(provider: auth.provider, uid: auth.uid)
     else
       user = where(provider: auth.provider, uid: auth.uid).first_or_initialize do |new_user|
         new_user.email = auth.info.email
         new_user.username = "#{auth.info.first_name[0]}#{auth.info.last_name.strip}"
         new_user.first_name = auth.info.first_name
         new_user.last_name = auth.info.last_name
         new_user.password = Devise.friendly_token[0,20]
         # 修正赋值逻辑：仅当promo_code有效时赋值
         new_user.promo_code = promo_code if promo_code.present?
       end
     end
     # 验证通过后再执行确认操作
     if user.save
       user.confirm unless user.confirmed?
     end
     user
   end
   

2. 修正验证上下文
   将验证规则改为标准的:create（仅创建用户时验证）或:create, :update（创建和更新时都验证）：

   validate :check_promo_code_limit, on: :create
   

3. 优化Promo Code赋值逻辑
   使用Rails内置的present?方法简化有效判断：

   new_user.promo_code = promo_code if promo_code.present?
   

修改后，当promo_code为nil或空字符串时，验证会触发错误，user.save返回false，用户不会被持久化，回调控制器会进入错误分支，将验证错误展示给用户。

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