Page MenuHomePhabricator

Move request decoding/streaming to AphrontRequestStream
ClosedPublic

Authored by epriestley on Mar 16 2016, 5:51 PM.
Tags
None
Referenced Files
F13985011: D15483.id.diff
Sun, Oct 20, 5:31 PM
F13970557: D15483.id.diff
Thu, Oct 17, 8:22 AM
F13968273: D15483.id37329.diff
Wed, Oct 16, 7:47 PM
Unknown Object (File)
Oct 1 2024, 10:56 PM
Unknown Object (File)
Sep 24 2024, 12:23 AM
Unknown Object (File)
Sep 21 2024, 12:56 AM
Unknown Object (File)
Sep 8 2024, 6:37 PM
Unknown Object (File)
Sep 2 2024, 12:59 PM
Subscribers
None

Details

Summary

Ref T10604. This moves the php://input / gzip handling code to a separate class. More input handling responsibility can likely live here in the future.

Ref T10264 for gzip window stuff.

Test Plan

See next change. Cloned a repository, broswed around, submitted POST data.

The window parameter is still a little magic, but this interpretation appears to work (?) and is supported by the zlib documentation:

windowBits can also be greater than 15 for optional gzip decoding. Add 32 to windowBits to enable zlib and gzip decoding with automatic header detection, or add 16 to decode only the gzip format (the zlib format will return a Z_DATA_ERROR). If a gzip stream is being decoded, strm->adler is a crc32 instead of an adler32.
http://www.zlib.net/manual.html

Diff Detail

Repository
rPHU libphutil
Branch
lfs9
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 11176
Build 13862: Run Core Tests
Build 13861: arc lint + arc unit