Page MenuHomePhabricator

Remove the (hopefully) obsolete "post_max_size" check during startup
ClosedPublic

Authored by epriestley on Apr 9 2020, 8:27 PM.
Tags
None
Referenced Files
F18100962: D21077.id.diff
Sat, Aug 9, 12:12 PM
F18097714: D21077.id50208.diff
Fri, Aug 8, 3:09 PM
F18093267: D21077.id50207.diff
Thu, Aug 7, 5:01 PM
F18090943: D21077.id.diff
Wed, Aug 6, 6:13 PM
F18088069: D21077.id50208.diff
Wed, Aug 6, 6:43 AM
F18070085: D21077.diff
Mon, Aug 4, 4:05 PM
Unknown Object (File)
Jun 29 2025, 6:22 PM
Unknown Object (File)
Jun 17 2025, 3:14 PM
Subscribers
None

Details

Summary

Ref T13507. See that task for discussion. This check appears to be obsolete in all common cases and misfires if the client submits compressed requests.

Since the cases where it could still trigger correctly are extremely rare and should still have plausible behavior, just remove it.

Test Plan

Grepped for calls.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Apr 9 2020, 8:33 PM
This revision was automatically updated to reflect the committed changes.