Page MenuHomePhabricator

Remove PhabricatorStartup::getGlobal/setGlobal mechanism
ClosedPublic

Authored by epriestley on Jun 4 2015, 6:59 PM.
Tags
None
Referenced Files
F16305325: D13152.diff
Sun, Jun 15, 12:01 PM
Unknown Object (File)
Fri, May 30, 12:58 AM
Unknown Object (File)
Wed, May 28, 3:09 PM
Unknown Object (File)
Tue, May 27, 8:52 PM
Unknown Object (File)
Apr 29 2025, 6:32 AM
Unknown Object (File)
Apr 28 2025, 1:50 AM
Unknown Object (File)
Apr 27 2025, 3:27 PM
Unknown Object (File)
Apr 26 2025, 8:40 PM
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.