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
Unknown Object (File)
Fri, Nov 22, 5:04 PM
Unknown Object (File)
Tue, Nov 19, 11:47 PM
Unknown Object (File)
Tue, Nov 19, 11:38 PM
Unknown Object (File)
Tue, Nov 19, 10:48 PM
Unknown Object (File)
Tue, Nov 19, 7:47 PM
Unknown Object (File)
Sat, Oct 26, 4:34 PM
Unknown Object (File)
Oct 24 2024, 5:05 AM
Unknown Object (File)
Oct 23 2024, 5:21 AM
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
Branch
post1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 24025
Build 33069: Run Core Tests
Build 33068: arc lint + arc unit

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.