Railsでログイン、ログアウト時間を取得

Question

userテーブルを持つRailsアプリのユーザーひとりひとりと紐づける形で、毎日ログイン時間とログアウト時間を記録したい（一週間ごとくらいで消滅してもいい）んですが、どのような方法が適切なのか、そもそもどういったやりかたで取得したらいいのかがあまりつかめません。。。

ちなみにDeviseは使っていなくて、Sorceryというgemを使っています。

そしてFacebookでオンラインの人の横に緑の丸が出るみたいな感じで、ログイン状態の表示もしたいです。

それらを実現する方法として、

ログインテーブルみたいなのを作り、
・ユーザーID（userテーブルにあるもの。これで紐付けを行う。）
・IPアドレス（今回の件とは関係ないので気にしないでください。）
・ログイン時間
・ログアウト時間
・ログイン状態（enumで管理？ 1:login 2:logout）
ログインが成功した時点でログインテーブルに1を入れて「1の時にオンラインであることを表示」という処理にする。

で、やろうと思っているんですが、

・これで適切なのかがわからないのと、
・ログイン、ログアウト時間の取得方法がわからない
・ログイン状態に関しては他にいいやり方はありそうか

というのが主な質問です。

その他指摘があればお願いいたします。

Answer 1

ログイン状態を履歴から取るのは良くないと思います、2つに分けるべきでしょう

1. 最新の情報を取得するために無駄な処理が必要
2. ログアウトせずブラウザを閉じた場合の対処が必要です、またそれはログ情報には不要

■1. ログイン状態に関して

gem sorceryには活動具合を記録してくれるモジュールがあるのでそれを適用するのはどうでしょう

https://github.com/Sorcery/sorcery/wiki/Activity-logging

ただし上のwikiにある current_users はgemでやることではないと既に削除されています

旧実装のソースと思わしき場所: lib/sorcery/adapters/mongoid_adapter.rb#L96

ログアウトしてないときは最終アクセス時刻からセッションタイムアウトの時間が過ぎるまでをログイン扱いにするなどの工夫をしているみたいですね
実際にどのような判定にするかは作るもの次第だと思いますが参考にどうぞ

■2. ログに関して

上記モジュールで各種の保存処理が増えています
https://github.com/Sorcery/sorcery/blob/master/lib/sorcery/controller/submodules/activity_logging.rb

そのモジュールのタイミングに追記するような形で拡張するのはどうでしょう？

class ApplicationController < ActionController::Base
  protected
  def register_login_time_to_db(user, credentials)
    super(user, credentials)
    user.login_logs.create(login_at: user.last_login_at)
  end

  # ログインせずに閉じるとここが動かないので、ログで未ログアウトとして残る
  def register_logout_time_to_db
    super
    user.login_logs
        .find_by(login_at: user.last_login_at)
        .update(logout_at: user.last_logout_at)
  end
end

また「一週間ごとくらいで消滅してもいい」というより別途バッチ処理等で意図して古いのは消すべきでしょう
ログ情報をDBに貯め続けるのはあまり良い仕組みではありません

Comments

[on_off_on]

丁寧に回答していただきありがとうございます。
一度試してみます。

ところで
credentialsの正体がわからないです。。

すみません、 credential_regexメソッドなどを読み込んで見ても、他の読み込まれてるモジュールを辿って見ても credentialが一体どこからきてるのかさっぱりわからず...

調べてみても今回の実装とは関係ないような気がして....。

[Ighrs]

関係ありませんが関数を上書きして super で本来の処理に流すためには同じ引数である必要があります、ただそれだけの物であって本筋ではないので気にしなくて良いですよ

[on_off_on]

おお、ありがとうございます！

[on_off_on]

たびたびすみません。。

user.login_logsのlogin_logsはログを残す用のテーブルだと思うんですが、その前ついてるuserとはなんのことでしょうか...？

undefined local variable or method `user'となってしまいます。。。

user_logsテーブルを作ったので少し変えてますが、

------------------------------------------------------------------------
class ApplicationController < ActionController::Base
protect_from_forgery with: :exception

protected
def register_login_time_to_db(user, credentials)
super(user, credentials)
user.user_logs.create(login_at: user.last_login_at)
end

# ログインせずに閉じるとここが動かないので、ログで未ログアウトとして残る
def register_logout_time_to_db
super
user.user_logs
.find_by(login_at: user.last_login_at)
.update(logout_at: user.last_logout_at)
end
end
--------------------------------------------------------------------

また、いただいた案とは別に、こんな感じで書いてみましたが、
undefined method `user_logs’ for nil:NilClass　となってしまいました

--------------------------------------------------------------------------------
class SessionsController < ApplicationController
def new
end

def create
user = login(params[:username], params[:password])
if user
#TODO: ここに処理を書く
@user_log = @user.user_logs.create(user_id: @user.id, login_at: @user.last_login_at)
redirect_back_or_to user_path(user)
else
render :new
end
end

def destroy
logout
#TODO: ここに処理を書く
@user_log = @user.user_logs.create(user_id: @user.id,logout_at: @user.last_logout_at)
redirect_to root_path
end
end
-----------------------------------------------------------------------------

こんな感じになりました。

[Ighrs]

userとlogin_logs を has_many で繋げていると繋がる値である user_id を省略して書けますよ

LoginLog.create(user_id: user.id, logind_at: ***)

もしくは

user.login_logs.create(login_at: ***)

のどちらかですね、userから繋げた場合に user_id を指定する必要はありません
なので has_many で繋げないなら普通にLoginLogクラスの関数を使えば大丈夫です

[on_off_on]

ありがとうございます。
この場合　

has_manyで繋げて教えていただいたようにやると

You have incorrectly defined user_class or have forgotten to define it in intitializer file
というエラーが出ます。

https://github.com/Sorcery/sorcery/blob/master/lib/sorcery/controller.rb
ここの160行目以降に関係している感じですが、まったく対処法がわからないです......

----------------------------------------------------------------------
class SessionsController < ApplicationController
def new
end

def create
user = login(params[:username], params[:password])
if user
#TODO: ここに処理を書く
user.user_logs.create(login_at: last_login_at)
redirect_back_or_to user_path(user)
else
render :new
end
end

def destroy
logout
#TODO: ここに処理を書く
user.user_logs.create(logout_at: last_logout_at)
redirect_to root_path
end
end
----------------------------------------------------------------------

もしよろしければ回答かヒントをいただけると助かります。

インデントなくてみづらいですね。。すみません。。

Answer 2

require "date"
DateTime.now

でその時点での日時を取得できます。

Comments

[on_off_on]

前の質問に引き続き回答ありがとうございます。