rescueされない例外？! #megurorb @suusan2go 2018/9/27 Meguro.rb #19 1

● M3, Inc. Software Engineer ● 半年くらいKotlin書いてましたが最近はPythonと Railsで機械学習系のアプリの機械学習以外を全部 やる仕事をしています ● 今日はRailsの話をします！ 自己紹介: @suusan2go 2

突然ですが開発環境でこのコントローラにアクセスすると 何のエラーが表示されるでしょう 3 class HogeController < ApplicationController def hoge raise 'hogehoge' rescue raise 'piyopiyo' end end

答え piyopiyo 4

じゃあこれだったら？ 5 def fuga raise ActiveRecord::RecordInvalid rescue raise 'piyopiyo' end

6

7

8

れ、rescueされてないだと・・・ 9

10

11 というやりとりをTwitterでみかけて、 気になったので調べてみた話です

rails consoleで試してみる 12 irb(main):001:0> def fuga irb(main):002:1> raise ActiveRecord::RecordInvalid irb(main):003:1> rescue irb(main):004:1> raise 'piyopiyo' irb(main):005:1> end => :fuga irb(main):006:0> fuga Traceback (most recent call last): 3: from (irb):6 2: from (irb):1:in `fuga' 1: from (irb):4:in `rescue in fuga' RuntimeError (piyopiyo) irb(main):007:0>

binding.pryを仕込んでみる 13 def fuga raise ActiveRecord::RecordInvalid rescue binding.pry raise 'piyopiyo' end

rescueされてる箇所にとまる！つまりrescueはされてるっぽい 14 8: def fuga 9: raise ActiveRecord::RecordInvalid 10: rescue 11: binding.pry => 12: raise 'piyopiyo' 13: end

15 rescueはされてるっぽい つまり表示の問題？ エラー表示箇所のコードを 見てみよう

actionpack/lib/action_dispatch/middleware/debug_exceptions.rb 16 def call(env) request = ActionDispatch::Request.new env _, headers, body = response = @app.call(env) if headers["X-Cascade"] == "pass" body.close if body.respond_to?(:close) raise ActionController::RoutingError, "No route matches [#{env['REQUEST_METHOD']}] #{env['PATH_INFO'].inspect}" end response rescue Exception => exception invoke_interceptors(request, exception) raise exception unless request.show_exceptions? render_exception(request, exception) end エラー画面を表示して いる箇所はここ！ exceptionはそのまま 渡している

actionpack/lib/action_dispatch/middleware/debug_exceptions.rb 17 def render_exception(request, exception) backtrace_cleaner = request.get_header("action_dispatch.backtrace_cleaner") wrapper = ExceptionWrapper.new(backtrace_cleaner, exception) log_error(request, wrapper) if request.get_header("action_dispatch.show_detailed_exceptions") content_type = request.formats.first if api_request?(content_type) render_for_api_request(content_type, wrapper) else render_for_browser_request(request, wrapper) end else raise exception end end ExceptionWrapper 怪しい

actionpack/lib/action_dispatch/middleware/exception_wrapper.rb 18 module ActionDispatch class ExceptionWrapper # 〜〜〜省略〜〜〜〜 def initialize(backtrace_cleaner, exception) @backtrace_cleaner = backtrace_cleaner @exception = original_exception(exception) expand_backtrace if exception.is_a?(SyntaxError) || exception.cause.is_a?(SyntaxError) end # 〜〜〜省略〜〜〜〜 end end original_exception！

actionpack/lib/action_dispatch/middleware/exception_wrapper.rb 19 def original_exception(exception) if @@rescue_responses.has_key?(exception.cause.class.name) exception.cause else exception end end exception.causeとは

https://docs.ruby-lang.org/ja/latest/method/Exception/i/cause.html 20

@@rescue_responsesはなにかというと 21 module ActionDispatch class ExceptionWrapper # 〜〜〜省略〜〜〜〜 @@rescue_responses.merge!( "ActionController::RoutingError" => :not_found, "AbstractController::ActionNotFound" => :not_found, "ActionController::MethodNotAllowed" => :method_not_allowed, "ActionController::UnknownHttpMethod" => :method_not_allowed, "ActionController::NotImplemented" => :not_implemented, "ActionController::UnknownFormat" => :not_acceptable, "ActionController::InvalidAuthenticityToken" => :unprocessable_entity, "ActionController::InvalidCrossOriginRequest" => :unprocessable_entity, "ActionDispatch::Http::Parameters::ParseError" => :bad_request, "ActionController::BadRequest" => :bad_request, "ActionController::ParameterMissing" => :bad_request, "Rack::QueryParser::ParameterTypeError" => :bad_request, "Rack::QueryParser::InvalidParameterError" => :bad_request )

activerecord/lib/active_record/railtie.rb 22 力尽きてちゃんとコード読んでないんですけど、多分ここで追加されたやつが追加 されてるっぽい config.action_dispatch.rescue_responses.merge!( "ActiveRecord::RecordNotFound" => :not_found, "ActiveRecord::StaleObjectError" => :conflict, "ActiveRecord::RecordInvalid" => :unprocessable_entity, "ActiveRecord::RecordNotSaved" => :unprocessable_entity )

結論 23 ● 例外が発生した場合、その例外の根本の例外がcauseでチェックされる ● 根本の例外がrescue_responsesに含まれる場合には画面には最終的 にraiseされた例外ではなく、根本の例外が表示される

再考 24 def fuga raise ActiveRecord::RecordInvalid rescue raise 'piyopiyo' end

再考 25 def fuga raise ActiveRecord::RecordInvalid rescue raise 'piyopiyo' end 最終的にraiseされているのはこれ しかしエラー表示の際には、この例外の cause である ActiveRecord::RecordInvalidが表示さ れる

26 なんでこんな挙動になっているのだ ろう？

27 https://github.com/rails/rails/commit/dde54f00c6c08d1704d011d8108106076e8ea033

なんと2010年のコミット #5784 はGitHubではなくlighthouseapp 28 https://rails.lighthouseapp.com/projects/8994-ruby-on-rails/tickets/5784-show exceptions-renders-original-exceptions

まとめ 29 ● この挙動はちょっとぎょっとしますが仕様です ● 試してみたら、templateでのエラーは ActionView::Template::Errorで Wrapされる ● このcauseを見に行く仕様は、templateでおもむろに `User.find(:id)` と かやってActiveRecord::RecordNotFoundがraiseされても、ちゃんと 404を返したりするためのものっぽい・・・・・・？ ● もともとの用途がtemplateでのエラーのためのものなので、causeを見 る挙動はActionView::Template::Errorに絞ってもよいのではという気が してきた。 ● なんか間違ってるところあれば教えてください :pray: