Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix dynamic properties PHP 8.2 #1004

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

krajcikondra
Copy link

Hi,

on php 8.2 I get errors like this:

Deprecated: Creation of dynamic property Ratchet\Server\IoConnection::$httpBuffer is deprecated

@krajcikondra krajcikondra changed the title fix dynamic properties fix dynamic properties PHP 8.2 Mar 17, 2023
@SimonFrings
Copy link

@krajcikondra The test suite doesn't execute the tests with PHP 8.2 currently and we can't see if your changes are completely fixing the issue, but I am looking into that, see #1003 for more details. Once I get everything updated, we can come back to this one. Depending on how far I go with #1003, I might need to apply the same changes as you did, I'll keep you posted 👍

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