before_actionのonlyがうまく行かない

Question

###実現したいこと
forbid_login_userのbefore_actionで指定したアクションのみで適用させたい。

class UsersController < ApplicationController
  before_action :authenticate_user, {only: [:index, :show, :edit, :update]}
  before_action :forbid_login_user, {only: [:new, :create, :login_form, :login]}
  before_action :ensure_correct_user, {only: [:edit, :update]}

現在、top.htmlやabout.htmlなど、onlyで指定していないページにも関わらず、session[:user_id]があれば勝手に/posts/indexにログインしてしまいます。

###コード

get "/" => "home#top"
get "about" => "home#about"
root "home#top"
get "/login" => "users#login_form"
post "login" => "users#login"
post "users/create" => "users#create"
get "signup" => "users#new"

class ApplicationController < ActionController::Base
  before_action :set_current_user

  def set_current_user
    @current_user = User.find_by(id: session[:user_id])
  end

  def authenticate_user
    if @current_user == nil
      flash[:notice] = "ログインが必要です"
      redirect_to("/login")
    end
  end
  def forbid_login_user
    if @current_user
      flash[:notice] = "すでにログインしています"
      redirect_to("/posts/index")
    end
  end
end

下記のURLでも記載の通り、やはりbefore_actionでonlyと限定すれば、forbid_login_userは起きないはずなのですが...。
https://qiita.com/jonnyjonnyj1397/items/4691d723860a8d927d15

もしかしたら、ApplicationController内でforbid_login_userが着火されてしまって、onlyで限定できていない状態なのかとも考えています。
お分かりの方、ぜひお力添えを宜しくお願いします。

Answer 1

class HomeController < ApplicationController
  before_action :forbid_login_user, {only: [:top, :about]}

  def top
  end
  
  def about
  end
end

HomeControllerにこのコードがないからです
top.htmlやabout.htmlは両方ともHomeController下にあるためUserControllerでのbefore_actionとは無関係です

PS　routes.rbの「get "/" => "home#top」"と「root "home#top"」はダブっているのでどちらか一つでよいかと思います

Comments

[tomtom1]

HomeControllerでしたか！把握しました！有難うございます。
今回は、top/aboutでは自動ログインをしたくないという事でしたので、
記載のforbid_login_user, {only: [:top, :about]}を削除する事で解決しました。
ありがとうございます。

Answer 2

「勝手に/posts/indexにログインしてしまいます」
のログインする　という意味ですが

1. /posts/index はloginしなければ見せないページである
2. session[:user_id]があれば /posts/indexにアクセスするとログイン画面経由せずに見えてしまう

ということでしょうか？

application_controllerのbefore actionでset_current_userが呼ばれています。このため session[:user_id]があれば　ログイン済として扱われるようになります。

/posts/indexにアクセスした時に、どのようになりたいのでしょうか？

Comments

[tomtom1]

ご回答有難うございます！
topやaboutのページに関しては、自動でログインをしたくなかったです。わかりづらくて申し訳ありません！
原因は、HomeControllerに、before_action :forbid_login_user, {only: [:top, :about]}に記載があったためでした。

Answer 3

すべて{}が不要だと思います。

before_action :authenticate_user, only: [:index, :show, :edit, :update]

にしてみたらどうでしょうか？

Comments

[winterboum]

only: [] は「最後のhashの引数」ですから、{}を省略出来ますが、省略しないのが元々ですから、つけてもつけなくても同じです。

[tomtom1]

ご回答ありがとうございます！winterboumさんのおっしゃる通り、変化ありませんでした！