Homec4science

Improve Phabricator behavior for fatal errors

Authored by epriestley <git@epriestley.com> on Jul 16 2011, 16:49.

Description

Improve Phabricator behavior for fatal errors

Summary:

  • Exceptions on the rendering pathway currently go uncaught and result in a

blank page. Commonly, this is a bad require_celerity_resource() call. Although
we can't safely render a page if the rendering pathway is broken, we can show a
useful message.

  • When PHP exits because of a fatal error, there is an opportunity to run code

in the shutdown handler. This allows us to show messages at least some of the
time, e.g. "call to unknown function derp() in somefile.php at line 99"

  • flip dem tables

Test Plan: Added fatals ("derp();") and rendering exceptions
("require_celerity_resource('does-not-exist')") to a controller and verified
that the error handling behavior is now more useful.
Reviewed By: aran
Reviewers: jungejason, tuomaspelkonen, aran
CC: aran, epriestley
Differential Revision: 680

Details

Committed
epriestley <git@epriestley.com>Jul 18 2011, 16:23
Pushed
aubortJan 31 2017, 17:16
Parents
rPH3ed0e84a635e: Add an ID column to slowvote's list view.
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH6f388f97d959: Improve Phabricator behavior for fatal errors (authored by epriestley <git@epriestley.com>).Jul 18 2011, 16:23