The main PhpInputStream existed was because prior to PHP 5.6, php://input was read-once and non-seekable.
As of 5.6, it's reusable AND seekable, which means there's no reason to keep the special implementation, particularly as it causes bugs (see #150)
This patch:
Removes the class
Updates the ServerRequest to, when a body of php://input is detected, create a read-only Stream instance with that value.
Xerkus
commented
1 year ago
Description
The main
PhpInputStreamexisted was because prior to PHP 5.6,php://inputwas read-once and non-seekable. As of 5.6, it's reusable AND seekable, which means there's no reason to keep the special implementation, particularly as it causes bugs (see #150)This patch:
ServerRequestto, when a body ofphp://inputis detected, create a read-onlyStreaminstance with that value.Fixes #150