Page MenuHomePhabricator

Read POST data sightly earlier in request startup

Authored by epriestley on Jan 31 2019, 3:33 PM.



On instances, the "SiteSource" (for site config) pretty much copy-pastes the "read POST data" block because it needs to make some decisions based on POST data when handling inbound mail webhooks.

Move the upstream read a little earlier so we can get rid of this. Now that this step is separated and must happen before the profiler, there's no reason not to do it earlier.

Test Plan

POSTed some data across pages without issue, will remove duplicate code in upcoming change.

Diff Detail

rP Phabricator
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Jan 31 2019, 3:33 PM
epriestley requested review of this revision.Jan 31 2019, 3:35 PM
epriestley edited the summary of this revision. (Show Details)Jan 31 2019, 3:39 PM
amckinley accepted this revision.Feb 1 2019, 4:01 AM
This revision is now accepted and ready to land.Feb 1 2019, 4:01 AM
This revision was automatically updated to reflect the committed changes.