sirn-se / websocket-php

[php-lib] WebSocket client and server in PHP
Other
117 stars 18 forks source link

Allow additional content in Connection header during handshake #40

Closed sirn-se closed 8 months ago

sirn-se commented 8 months ago

According to RFC, the Connection header MAY include additional content, but MUST include the "Upgrade" keyword.

The request MUST contain a |Connection| header field whose value MUST include the "Upgrade" token.

Issue #39 got "keep-alive, Upgrade" which would be valid, but caused an error in current version.

Closing #39

coveralls commented 8 months ago

Coverage Status

coverage: 100.0%. remained the same when pulling 7cf68ca83189dc207822ab8ccffb41459f400f11 on headerfix into 6d21dfb03eb30c67a3c5fab4e2fc0d77686eebe7 on v2.1-main.