Ratchet icon indicating copy to clipboard operation
Ratchet copied to clipboard
verified publisher icon ratchetphp

Dynamic properties

Open mmaennche opened this issue 1 year ago • 1 comments

PHP Compatability for Dynamic Properties React\Socket\Connection::$decor Ratchet\Server\IoConnection::$resourceId Ratchet\Server\IoConnection::$remoteAddress Ratchet\Server\IoConnection::$httpHeadersReceived Ratchet\Server\IoConnection::$httpBuffer Ratchet\Server\IoConnection::$httpRequest Ratchet\Server\IoConnection::$WebSocket

mmaennche avatar Jun 27 '24 21:06 mmaennche

Maybe a proper fix should be both #1060 and this, because it was previously encouraged to add properties to the connection in onOpen which would trigger the warning as well

Tofandel avatar Jun 28 '24 17:06 Tofandel

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 #1096 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.

PaulRotmann avatar May 30 '25 11:05 PaulRotmann