Page MenuHomePhabricator

Add a friendlier "in flight" error page
ClosedPublic

Authored by epriestley on Sep 6 2016, 8:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 10:13 AM
Unknown Object (File)
Fri, Dec 13, 10:55 PM
Unknown Object (File)
Nov 18 2024, 8:52 PM
Unknown Object (File)
Oct 23 2024, 7:50 AM
Unknown Object (File)
Oct 17 2024, 12:41 PM
Unknown Object (File)
Oct 9 2024, 9:30 PM
Unknown Object (File)
Sep 25 2024, 3:01 AM
Unknown Object (File)
Sep 12 2024, 8:43 AM
Subscribers
None

Details

Summary

Ref T11589. When we hit a fatal setup issue (essentially always a connection failure) after we've already survived them on at least one request, we can be pretty sure a server went down and that the problem is not a setup/configuration issue.

In this case, show a friendlier error page instead of the fairly detailed technical one.

Test Plan
  • Broke MySQL config.
  • Restarted Apache.
  • Got the "admin/setup" error page:

Screen Shot 2016-09-06 at 1.50.21 PM.png (1×1 px, 137 KB)

  • Fixed the MySQL config.
  • Loaded any page, to put us "in flight".
  • Broke MySQL config.
  • Loaded any page.
  • Got the friendly "in flight" error page:

Screen Shot 2016-09-06 at 1.48.05 PM.png (1×1 px, 68 KB)

If you want to design this better, easiest way to get to it is:

  • Set mysql.port to 9999 in conf/local/local.json.
  • Reload any page while already running (don't restart).

Diff Detail

Repository
rP Phabricator
Branch
init5
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13592
Build 17510: Run Core Tests
Build 17509: arc lint + arc unit