3

我想使用 pg_audit_log 登录 Rails 应用程序。审核日志不仅必须显示已更改的列，还必须显示进行这些更改的用户。文档没有显示如何执行此操作，但在查看 pg_audit_log 源（postgresql_adapter.rb）后，我看到它从线程局部变量 ala 中读取用户信息：

current_user = Thread.current[:current_user]

我考虑过在过滤器之前和之后设置/取消设置，如下所示：

Thread.current[:current_user] = current_user

（使用控制器中的 current_user 辅助方法来获取当前登录的用户），但这似乎很危险。我现在花时间尝试了解 Rails 请求周期和线程如何交互，以便更好地了解危险程度。同时，我很好奇当前使用 pg_audit_log 的任何 SO 用户是否已经解决了每次用户对记录进行更改时将 user_id 和 user_unique_name 记录到日志表的需要。

4

2 回答 2

4

按照您描述的方式设置当前用户是一种常见的方法。例如，参见http://rails-bestpractices.com/posts/47-fetch-current-user-in-models

一些示例代码可能如下所示：

# in your model
class User < ActiveRecord::Base
  def self.current
    Thread.current[:current_user]
  end
  def self.current=(user)
    Thread.current[:current_user] = user if user.nil? || user.is_a?(User)
  end
end

# in your controller
class ApplicationController < ActionController::Base
    before_filter :set_current_user
    def set_current_user
      User.current = user_signed_in? ? current_user : nil
    end
end
于 2013-04-02T12:19:43.377 回答
1

依靠Thread.current哈希来提供对控制器管理的对象的模型级访问确实是有争议的。例如，请参阅以下内容：

Rails 中 Thread.current[] 使用的安全性

令人担忧的是，pg_audit_loggem 中没有记录此特定功能。

假设您没有积极探索 gem 的源代码，并且假设您已经独立决定Thread.current[:current_user] = something在自己的应用程序中进行定义，以实现自己的目的。在这种情况下，pg_audit_log会在您不知情的情况下审核该对象。

诚然，这个名称current_user被普遍接受，表示由身份验证例程定义的当前登录用户，很难将这个潜在的错误想象为一个具体问题，但从设计的角度来看呢？哎哟。

另一方面，由于您知道自己在做什么，因此确保Thread.current[:current_user]在每个响应周期的开始/结束时设置/取消设置应该使过程安全。至少这是我从阅读有关该主题的大量帖子中收集到的。

干杯，朱塞佩

于 2013-04-05T06:48:34.273 回答