Page MenuHomePhabricator

Fix excessively severe CSP URI error during first-time setup
ClosedPublic

Authored by epriestley on Apr 21 2018, 4:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 19, 5:54 PM
Unknown Object (File)
Sun, Dec 8, 6:03 AM
Unknown Object (File)
Tue, Dec 3, 3:38 AM
Unknown Object (File)
Mon, Dec 2, 7:59 PM
Unknown Object (File)
Mon, Dec 2, 7:59 PM
Unknown Object (File)
Mon, Dec 2, 7:59 PM
Unknown Object (File)
Mon, Dec 2, 7:59 PM
Unknown Object (File)
Fri, Nov 29, 5:03 AM
Subscribers
None

Details

Summary

See D19394. Currently, during first-time setup before you configure "phabricator.base-uri", we may attempt to generate a setup page, try to generate a CSP header for it, and fail to access the environmental config. This causes a too-severe error page ("configure phabricator.base-uri") instead of preflight guidance (like "can't connect to MySQL").

Instead, treat this more like "security.alternate-file-domain" and just bail on CSP if we can't fetch it.

Test Plan

On a fresh (non-explodey laptop) install with critical setup errors (no MySQL installed yet), loaded Phabricator. Before: error about phabricator.base-uri. After: more helpful guidance about installing/configuring MySQL.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable