Page MenuHomePhabricator

Remove PhabricatorStartup::getGlobal/setGlobal mechanism
ClosedPublic

Authored by epriestley on Jun 4 2015, 6:59 PM.
Tags
None
Referenced Files
F15460121: D13152.id31803.diff
Mon, Mar 31, 8:15 PM
F15448061: D13152.diff
Fri, Mar 28, 3:11 AM
F15446361: D13152.id31790.diff
Thu, Mar 27, 6:41 PM
F15446088: D13152.id.diff
Thu, Mar 27, 5:07 PM
F15442422: D13152.diff
Wed, Mar 26, 11:22 PM
F15387453: D13152.id31790.diff
Sat, Mar 15, 2:25 AM
F15332726: D13152.diff
Mar 7 2025, 7:38 PM
Unknown Object (File)
Feb 21 2025, 7:46 AM
Subscribers

Details

Summary

Ref T8424. Fixes T7114. This was envisioned as a per-request cache for reusing interpreters, but isn't a good fit for that in modern Phabricator.

In particular, it isn't loaded by the daemons, but they have equal need for per-request caching.

Since I finally need such a cache for Spaces, throw the old stuff away before I built a more modern cache.

Also resolves T7114 by dropping filtering on $_SERVER. I'm pretty sure this is the simplest fix, see D12977 for a bit more discussion.

Test Plan

Called didFatal() from somewhere in normal code and verified it was able to use the access log.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Remove PhabricatorStartup::getGlobal/setGlobal mechanism.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jun 4 2015, 9:15 PM
This revision was automatically updated to reflect the committed changes.