Roda rescues both StandardError and ScriptError in its error handling plugin, it might be a good idea for Puma to do something similar.

Fixed! Thanks for pointing it out, I agree (and thusly followed suit with the code change) that it should catch more exceptions here.

Thanks! Any reasons for not simply rescue Exception?

I've done that before though I'm generally against it because it also catches NoMemoryError, for which there is really no recovery and the process needs to just die.

Though, ThreadPool does rescue Exception so, actually, I should probably change it to Exception.

I see, it makes sense. Even when no memory is available I believe it's a valid attempt trying to better handle the response. Thanks.