-
Notifications
You must be signed in to change notification settings - Fork 49
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
Updating ReactPHP #61
Comments
Hey @fuad-tareq, thanks for bringing this up 👍 The problem here lies in the |
@SimonFrings Thanks for the update. That is indeed unfortunate. I am not aware of what the reported "PHP 7.3 compatibility issues" are and have not ran into any of them yet. I have been using this library (as part of PHP Ratchet) under Apache 2.4.54 (PHP-FPM) in a WordPress environment (ZMQ seems perfect for this) for over a year now. It is currently running with PHP 7.4.30, but I do believe it ran without hiccups in prior PHP versions (7.0 and 7.3) as well. Going through the comments on the reported compatibility issues, it seems most of them were when ZMQ is ran within a Windows environment, so perhaps that is it. I'm running it with no issues so far under CentOS 7. In any case, I did manually update (overwrite) the ReactPHP folder used by this library to the latest version and was then able to use ReactPHP's promises just as I had hoped without any issues so far! I should probably ask since the ZMQ library support is currently questionable, should it start failing (perhaps after a PHP version update, etc) at some point, what would you recommend as a replacement tool that will allow me to easily send server messages to ReactPHP's event loop (and then onto clients websockets) ? |
@fuad-tareq This library supports PHP versions up to 7.2, this is tested and should be working as expected. I don't want to say to go away from ZMQ, if it works for you, you should be fine. It seems like you already put much time into development, so I can understand that switching out an underlying extension/library is linked to much work. It could also be just fine the way it operates now, depends on which features you're using and your underlying use case. I can't give an better answer at this point, I wish I could. I have little intel on your application code/structure or the issues you're currently facing (if there are any), so it's hard for me to suggest a replacement. You also opened up a ticket inside reactphp/reactphp#461 where you talked about issues you had with your project and the newest version of ReactPHP, maybe my answer in there plus this one here will help you a bit to get going. |
I see why people dislike extensions now; they seem to die faster than flies! And in this case, they didn't provide an alternative for a graceful migration from ZMQ sadly. In any case, since I managed to update ReactPHP to the latest version and was able to use the new async feature with this library still enabled and no issues so far 🤞, I'm going to leave it at that until I run into a problem (like when I update to PHP 8 or something and then all hell breaks lose). Once that happens I'll look for a replacement. Thanks for your attempts to help with all of it. |
This library seems to be using ReactPHP's event loop with ZMQ. However, it's using an old version of ReactPHP. I was hoping I could use some of ReactPHP's newer Async/Promise tasks but that would require updating ReactPHP. Would this library allow updating ReactPHP to the latest version?
The text was updated successfully, but these errors were encountered: