[ruby-core:56858] [ruby-trunk - Bug #8730] "rescue Exception" rescues Timeout::ExitException

Issue #8730 has been updated by nobu (Nobuyoshi Nakada).


kosaki (Motohiro KOSAKI) wrote:
> > 具体的にはどのように解説されてどう利用されているんでしょうか。
> 
> 残念ながら紙の書籍は手元にありませんが、
> 
> 一番多いのは http://kwfsws.g.hatena.ne.jp/kiwofusi/20111231/1325314356 みたいに
> rescue Exception => e とすると、意図せずタイムアウトも拾ってしまうので注意しましょうという記述。
> これは "ruby timeout 例外" で検索すると似たような注意は何個も出てきます。

なるほど。

> 次に出てくるのは逆に、デーモンなどで全部捕まえたいケースで、rescue Timeout::Error, StandardError =>e
> と書きましょう（書いている）という記述 http://d.hatena.ne.jp/dreammind/20090217/1234813224
> 
> ２番めは動かなくなりますよね。

Timeout.timeoutから抜けるときには(指定がなければ)Timeout::Errorを投げるので、影響はないですね。

> それでどこで議論して、Pros/Consをどう判断した結果なのかとお聞きしました。

意図せずにrescueされてしまう問題点は以前から知られていた問題なので、今回のチケットをきっかけにちょっと試してみたわけです。
非互換性として思いついたのは、timeoutされる処理の中でExceptionをrescueしてtimeoutを起こさないようにしていた場合ですが、あるとしたらむしろ意図しないバグだろうと考えました。
前者はまさにその一例ですね。



----------------------------------------
Bug #8730: "rescue Exception" rescues Timeout::ExitException
https://bugs.ruby-lang.org/issues/8730#change-41400

Author: takiuchi (Genki Takiuchi)
Status: Rejected
Priority: Normal
Assignee: 
Category: 
Target version: current: 2.1.0
ruby -v: 2.0.0
Backport: 1.9.3: UNKNOWN, 2.0.0: UNKNOWN


=begin
Timeout.timeout ブロック内で rescue Exception によって例外を捕捉している箇所があると、
タイムアウト処理が内部的に利用している Timeout::ExitException クラスの無名派生クラスを補足してしまい、
正しく Timeout::Error が発生しない。

例)

  timeout 1 do
    begin
      sleep 3
    rescue Exception => e
      puts e.class.superclass #=> "Timeout::ExitException"
    end
  end
=end


-- 
http://bugs.ruby-lang.org/