Destroy button hangs if the process ignores SIGTERM

[sodabrew]

If the queue process ignores SIGTERM, it will cause a web process to hang until it receives the signal. There should be a timeout followed by a SIGKILL.

[drudru]

BTW - great to see all these recent updates! 👍

[tjk]

Somewhat related: a lot too many "naked" rescue lying around... many of them could be AT LEAST StandardError so as to avoid ignoring of certain exceptions (ie. SignalException).

[drudru]

Is everything on Ruby 2.x yet?

[sodabrew]

@tjeezy naked rescue is synonymous with rescue StandardError. The problematic code is rescue Exception because that will also catch signals. Documented here:
http://ruby-doc.org/core-1.9.3/StandardError.html
http://ruby-doc.org/core-2.0.0/StandardError.html

@drudru RQ runs on 1.8.7, 1.9.3, 2.0, 2.1, and 2.2 per Travis CI: https://travis-ci.org/brightroll/rq In practice we're mostly using 1.8.7 and 1.9.3.