Rails中before_action使用疑问：PIN验证后无法跳转至galleries/:id

你猜的完全没错！问题就出在before_action :authenticate的逻辑上——你没有任何机制记录用户已经通过了PIN验证，所以即使auth方法验证成功跳转到galleries/:id，before_action又会立刻触发重定向回登录页，形成死循环。

要修复这个问题，我们需要通过会话（session）来跟踪用户已授权访问的画廊ID，让authenticate方法能判断用户是否已经通过验证。

1. 修改认证拦截逻辑 #

更新galleries_controller里的authenticate方法，检查会话中是否存在当前画廊的授权记录：

private
def authenticate
  # 如果会话中已有该画廊的授权标记，直接跳过拦截
  return if session[:authorized_gallery_ids]&.include?(params[:id].to_i)

  # 未授权则重定向到登录页
  redirect_to gallery_login_path(id: params[:id])
end

2. 完善auth方法的授权逻辑 #

修改auth方法，在PIN验证通过时将画廊ID存入会话，同时优化原有的错误逻辑顺序（避免错误信息被覆盖）：

def auth
  @error_login = nil
  @gallery = Gallery.find_by(id: params[:gallery_id])

  # 处理无需PIN的画廊（直接授权）
  if @gallery&.pin.nil?
    authorize_gallery(@gallery.id)
    redirect_to gallery_path(@gallery)
    return
  end

  # 检查PIN是否为空
  unless params[:pin].present?
    @error_login = 'Required PIN'
    render_error_response
    return
  end

  # 验证PIN正确性与有效期
  if @gallery.pin == params[:pin]
    if @gallery.expired_pin.past?
      @error_login = 'PIN expired'
      render_error_response
    else
      # 验证通过，添加授权标记到会话
      authorize_gallery(@gallery.id)
      redirect_to gallery_path(@gallery)
    end
  else
    @error_login = 'PIN not match'
    render_error_response
  end
end

private
# 提取公共方法管理会话中的授权画廊
def authorize_gallery(gallery_id)
  session[:authorized_gallery_ids] ||= []
  session[:authorized_gallery_ids] << gallery_id.to_i
  session[:authorized_gallery_ids].uniq! # 避免重复添加同一画廊ID
end

# 统一处理错误响应
def render_error_response
  respond_to do |format|
    format.js { render 'galleries/error_login', layout: false }
  end
end

3. 优化路由辅助方法使用 #

原代码中用字符串拼接跳转路径（redirect_to "/galleries/#{@gallery.id}"），建议改用Rails内置的路由辅助方法gallery_path(@gallery)，更符合Rails规范且不易出错。

• 如果需要支持用户退出画廊授权，可以添加一个方法清除会话中对应的画廊ID
• 若画廊的PIN或有效期更新后，需要让已授权的用户重新验证，可以考虑在authenticate方法中额外检查当前画廊的PIN状态是否变化

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