8
7

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

More than 5 years have passed since last update.

[Rails] rescue_fromは後に書いた指定が優先される

Posted at

application_controller.rbrescue_from を書いて Exception を扱うことがよくあると思います。今日はこの優先順位でハマりました。最初はステータスコードの番号順だよねという単純な理由で以下のようにしていました。

app/controllers/application_controller.rb
  rescue_from CanCan::Unauthorized, with: :render_403
  rescue_from Exception, with: :render_500 if Rails.env.production?

render_500 の中ではエラーをメールで飛ばす処理とかが実装されているので、本番だけ動かすようにしていました。この指定で、開発環境では CanCan::Unauthorized は正しく render_403 で拾うことができました。

ただ、このコードを production に載せると問題が発生しました。 CanCan::Unauthorizedrender_500 に行ってしまうのです。「テストも通っていたのになぜ……」と悩みましたが、どうやらこの指定は後に指定したものが優先されるようです。上記の指定では、 if Rails.env.production? がついているせいで後に指定したものが無視されていました。

結論

CanCan::Unauthorized を優先するため後に書き、 unless Rails.env.development? に変えました。 こうしておくことで test と production を同じ動きにすることができ、正しくテストを落とせます。

app/controllers/application_controller.rb
  rescue_from Exception, with: :render_500 unless Rails.env.development?
  rescue_from CanCan::Unauthorized, with: :render_403
8
7
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
8
7

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?