returnしているのにログイン認証処理が通ってしまう。

Question

前提・実現したいこと

ログイン認証に失敗しても、コントローラのindexアクションを通ってしまいます。
ログイン認証に失敗した場合、どのような処理にさせればよいのでしょうか？
ご教示お願いいたします。

試したこと

・スーパークラス

class CommonController < ApplicationController

    before_action :do_init

    def do_init
        if !login then
            p "認証に失敗"
            return
        end
    end

    # ログイン処理。
    def login
      # テスト用に認証に失敗させる。
      false
    end
end

・継承クラス

class SampleController < CommonController
    def index
        render plain: "ログイン成功"
    end
end

returnで通らないようにしているつもりですが、indexアクションが動いて
render plain: "ログイン成功"　が表示されてしまいます。
returnでは中止できないのでしょうか？

Answer 1

returnでは中止できないのでしょうか？

はい。throw(:abort)あるいはエラーページにredirect_toするなどしてください。

---

（以下、間違いの訂正と追記）

before_actionで処理を止めたい場合、「例外を投げて脱出してしまう（どこかでrescue_fromしないとアプリケーションエラーとなる）」あるいは「中で何かしらのレンダリングを行ってからreturnする」という2つの選択肢しかありません。

認証が通っていない場合、head :unauthorizedとして401のステータスコードを返す、ぐらいでもいいかもしれません。

Comments

[tkshp]

ご回答ありがとうございます。
throw(:abort)を書いた場合、
uncaught throw :abort
というエラーが出てしまうのですが、どう対処すればよいですか？
このキーワードで検索したのですが、わかりませんでした。

[maisumakun]

すみません、throw(:abort)は別の場所で使うべきものでした。本文も訂正します。

[tkshp]

ご回答ありがとうございます。
理解できました。
ありがとうございました。