HomePhorge

Guarantee the existence of the Phabricator access log

Description

Guarantee the existence of the Phabricator access log

Summary:
We have a fair number of conditionals on the existence of the access log. Instead, always build it and just don't write it if the user doesn't want a version on disk.

Also, formalize logged-in user PHID (avoids object existence juggling) in the access log and move microseconds-since-startup to PhabricatorStartup (simplifies index.php).

Depends on D5532. Fixes T2860. Ref T2870.

Test Plan: Disabled access log, verified XHProf writes occurred correctly.

Reviewers: btrahan, chad

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T2860, T2870

Differential Revision: https://secure.phabricator.com/D5533

Details

Provenance
epriestleyAuthored on Apr 2 2013, 9:53 AM
themackabuPushed on Tue, Mar 25, 8:07 PM
Parents
rP0f9bfa3bfdc3: Fix XHProf index page
Branches
Unknown
Tags
Unknown

Event Timeline