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)
Mar 14 2024, 6:09 PM
Unknown Object (File)
Feb 15 2024, 10:36 PM
Unknown Object (File)
Feb 3 2024, 11:57 PM
Unknown Object (File)
Jan 26 2024, 4:46 PM
Unknown Object (File)
Jan 22 2024, 5:40 PM
Unknown Object (File)
Jan 16 2024, 11:44 AM
Unknown Object (File)
Jan 10 2024, 10:15 AM
Unknown Object (File)
Jan 9 2024, 8:17 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.