Skip to content

Add support for guzzlehttp/psr7 3.0 - #169

Open
mbabker wants to merge 1 commit into
ratchetphp:masterfrom
mbabker:upgrade-guzzle
Open

mbabker wants to merge 1 commit into
ratchetphp:masterfrom
mbabker:upgrade-guzzle

Conversation

@mbabker

@mbabker mbabker commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

To get a green CI, this will require a release of ratchet/rfc6455 with ratchetphp/RFC6455#79 included.

@ArtemGoutsoul

Copy link
Copy Markdown

+1, thanks for this. We use Pawl as the WebSocket client, and it's the last dependency blocking our move to Guzzle 8.

I ran tests/unit on this branch (PHP 8.1):

guzzlehttp/psr7 ratchet/rfc6455 Result
2.13.1 v0.4.1 12/12 pass
3.1.0 v0.4.0 or v0.4.1 12/12 fail: InvalidArgumentException: Header value must be a string or array of strings but int provided. (in ClientNegotiator, called from Connector::__construct)
3.1.0 master (with ratchetphp/RFC6455#79) 12/12 pass

Once RFC6455 0.4.2 is tagged, could this also change the rfc6455 constraint to ^0.3.1 || ^0.4.2? 0.3.x requires psr7 ^1.7 || ^2, so it can't be installed next to psr7 3. 0.4.0 and 0.4.1 can, and then the client fails at runtime instead of at install time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants