V0.6 Support for Symfony 7

[joem236]

No description provided.

[Ovski4]

Thanks a lot for opening this PR. I was able to use your fork as a composer vcs repository and update a symfony application to version 7 successfully thanks to you

[treeindark]

Hello, i use plesk, with no problem, but i planed to upgrade symfony to 7.0. With your PR install is ok, but now, no more connection my messagehandler is no more call, and client have
Error: socket hang up

It a difference from plesk ?

[treeindark]

From plesk : (work with sf 6.4)

$wsServer = new WsServer(new WebSocketHandler());
$app = new HttpServer($wsServer);
$server = IoServer::factory($app, 8080);
$wsServer->enableKeepAlive($server->loop, 60);      
$server->run();

And i try with doc provided :

$app = new \Ratchet\App('192.168.1.200', 8080, '0.0.0.0');
$app->route('/', new WebSocketHandler(), ['*']);
$app->run();

but my handler is never call ( with breakpoint) with this 2methods

WebSocketHandler is MessageHandler implements MessageComponentInterface

[treeindark]

WsServer.php
126: $conn->send(Message::toString($response));

replace
17:
use GuzzleHttp\Psr7 as gPsr;
by
use GuzzleHttp\Psr7\Message;

[dvdknaap]

why is this one not merged yet ?

[Tofandel]

This namespace removal causes fatal errors because Query::parse and Query::build have not been replaced, in fact it's not even used

[Tofandel]

Suggested change

	use GuzzleHttp\Psr7 as gPsr;
	use GuzzleHttp\Psr7\Query;

[Tofandel]

Suggested change

	use GuzzleHttp\Psr7 as gPsr;
	use GuzzleHttp\Psr7\Message;

Same here

[Tofandel]

Suggested change

	, "react/socket": "^1.0 || ^0.8 || ^0.7 || ^0.6 || ^0.5 || ^v1.15"
	, "react/event-loop": ">=v1.5.0"
	, "react/socket": "^1.0 || ^0.8 || ^0.7 || ^0.6 || ^0.5"
	, "react/event-loop": ">=0.4"

*Those changes are unnecessary because the previous range already allows those newer versions

[Tofandel]

Suggested change

	, "symfony/http-foundation": "^2.6|^3.0|^4.0|^5.0|^6.0|^v7.0"
	, "symfony/http-foundation": "^2.6|^3.0|^4.0|^5.0|^6.0|^7.0"

[PaulRotmann]

Hey! Thanks for this pull request 🙏

As part of our cleanup initiative (#1100), we're closing old PRs that have already been resolved. We've recently merged #1098 which implements the same idea, but turned out to be a bit more involved than initially anticipated.

If you have further improvements to suggest, feel free to open a new pull request. We appreciate your contributions and welcome any future enhancements. If you want to support Ratchet, please consider sponsoring our work! ❤️

Thanks again for your contribution.