It's painful to debug, we should display the stack trace to track down the root of the problem.
Created attachment 145929 [details] [review] Bug 32777: Display stack trace for REST API unhandled exception
Do we need that in production as well? Do we actually have stack traces in production for other 500? If not, why? How do people debug crashes if they are not displayed?
I am all for those stack traces. Note that the stack trace in plack-error.log often contains only a few useful lines. All the internal Plack stuff is just waste of space.. Devel::Stacktrace allows you to filter?
Can we proceed here?
(In reply to Marcel de Rooy from comment #4) > Can we proceed here? Not sure how. We are going to display a stack trace in production, and I think we don't for other exceptions (whereas we should IMO). So maybe we should add a 'if $dev_install' condition to display the stack trace?
(In reply to Jonathan Druart from comment #5) > So maybe we should add a 'if $dev_install' condition to display the stack > trace? Configurable number of frames from the top? Zero means no trace? Not sure if this is a parameter from Devel module though.