[1.x] Support PSR-7 v2 along side v1#549
Conversation
d199fd9 to
39bb017
Compare
|
Great work, would be nice of have this be merged? I ran into an issue while updating some vendors recently who also bumped their version to v2 already. |
|
My colleague tested most of the flows as much as possible. This were his findings:
|
9bd94a3 to
0f6f30a
Compare
This changeset introduces our PSR-7 v2 implementation alongside our PSR-7 v1 implementation and goes with v1 or v2 depending on which `psr/http-message` version is installed. This is admittedly not my best code, nor am I proud of it, but it gets the job done. This relies on clue/reactphp-http-proxy#65 being merged first. Implements reactphp#513 for `react/http` v1.
0f6f30a to
362d4ed
Compare
Great! Had a call yesterday with @clue about this PR, among things. Been working on it to reduce the changed line count from +4.6K/-2.1K to +2.3K/-9 so it's easier to review. Also updated |
This changeset introduces our PSR-7 v2 implementation alongside our PSR-7 v1 implementation and goes with v1 or v2 depending on which
psr/http-messageversion is installed. This is admittedly not my best code, nor am I proud of it, but it gets the job done.This relies on clue/reactphp-http-proxy#65 being merged first.
Implements #513 for
react/httpv1.