Хорошая идея получить доступ к сессии в наблюдателе или нет?

Hrm, это - липкая ситуация. В значительной степени необходимо нарушить MVC для получения его работающий приятно.

Я сделал бы что-то вроде этого:

class MyObserverClass < ActiveRecord::Observer
  cattr_accessor :current_user # GLOBAL VARIABLE. RELIES ON RAILS BEING SINGLE THREADED

  # other logging code goes here
end

class ApplicationController
  before_filter :set_current_user_for_observer

  def set_current_user_for_observer
    MyObserverClass.current_user = session[:user]
  end
end

Это - немного hacky, но это больше не hacky, чем много других базовых вещей направляющих, которые я видел.

Все, что необходимо было бы сделать для создания этого ориентированным на многопотоковое исполнение (это только имеет значение, работаете ли Вы на jruby так или иначе) должно изменить cattr_accessor, чтобы быть правильным методом и иметь его хранилище, это - данные в локальной памяти потока

Вы правы относительно этого повреждающий MVC. Я предложил бы использовать обратные вызовы в Ваших контроллерах, главным образом потому что существуют ситуации (как модель, какое сохранение называют, но приводит проверку к сбою), где Вы не хотели бы наблюдателя, регистрирующего ничто.

Я нашел, что очевидный способ сделал то, что предлагается ответом, который я выбрал.

http://pjkh.com/articles/2009/02/02/creating-an-audit-log-in-rails

Это решение использует модель AuditLog, а также модуль TrackChanges для добавления функциональности отслеживания к любой модели. Это все еще требует, чтобы Вы добавили строку к контроллеру, когда Вы обновляете или создаете все же.

В прошлом при выполнении чего-то вроде этого, я склонялся к расширению класса модели User для включения идеи 'текущего пользователя'

Смотря на предыдущие ответы, я вижу предложения для хранения фактического активного рекордного пользователя на сессии. Это имеет несколько недостатков.

• Это хранит возможно большой объект в базе данных сессии
• Это означает, что копия пользователя 'кэшируется' навсегда (или пока выход из системы не вызывается). Это означает, что любые изменения в состоянии этого пользователя не будут распознаны, пока пользователь не выйдет из системы и входит в. Это означает, например, та попытка отключить пользователя будет ждать его выходить из системы и назад на. Это - вероятно, не поведение, которое Вы хотите.

Так, чтобы в начале запроса (в фильтре) Вы взяли user_id от сессии и считали пользователя, установку User.current_user.

Что-то вроде этого...

class User
  cattr_accessor :current_user
end

class Application
  before_filter :retrieve_user

  def retrieve_user
    if session[:user_id].nil?
      User.current_user = nil
    else
      User.current_user = User.find(session[:user_id])
    end
  end
end

С тех пор это должно быть тривиально.