Page MenuHomePhabricator

Read POST data sightly earlier in request startup
ClosedPublic

Authored by epriestley on Jan 31 2019, 3:33 PM.
Tags
None
Referenced Files
F15492103: D20073.diff
Sat, Apr 12, 10:57 AM
F15478020: D20073.id.diff
Mon, Apr 7, 7:44 PM
F15476530: D20073.id47933.diff
Mon, Apr 7, 7:06 AM
F15472186: D20073.id47946.diff
Sat, Apr 5, 11:17 AM
F15461621: D20073.id47933.diff
Tue, Apr 1, 9:35 AM
F15336113: D20073.id47946.diff
Mar 8 2025, 6:15 PM
F15336112: D20073.id47933.diff
Mar 8 2025, 6:15 PM
F15336109: D20073.id.diff
Mar 8 2025, 6:15 PM
Subscribers
None

Details

Summary

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

Repository
rP Phabricator
Branch
instance1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21783
Build 29725: Run Core Tests
Build 29724: arc lint + arc unit