pelim / laravel-zmq

laravel zmq broadcast driver
21 stars 8 forks source link

bug fixes #9

Open mitinsany opened 3 years ago

mitinsany commented 3 years ago

I see, that in you code for every new ZMQ message new ZMQ context created and after some work time it called error, that cant create context...

PHP Fatal error:  Uncaught ZMQSocketException: Error creating socket: Too many open files in /home/hunter/bet/Laravel-Admin-Panel-Course/vendor/pelim/laravel-zmq/src/Connector/ZmqPublish.php:28
Stack trace:
#0 /home/hunter/bet/Laravel-Admin-Panel-Course/vendor/pelim/laravel-zmq/src/Connector/ZmqPublish.php(28): ZMQContext->getsocket()
#1 /home/hunter/bet/Laravel-Admin-Panel-Course/vendor/pelim/laravel-zmq/src/Zmq.php(41): Pelim\LaravelZmq\Connector\ZmqPublish->connect()
#2 /home/hunter/bet/Laravel-Admin-Panel-Course/vendor/pelim/laravel-zmq/src/Zmq.php(87): Pelim\LaravelZmq\Zmq->connection()
#3 /home/hunter/bet/Laravel-Admin-Panel-Course/vendor/pelim/laravel-zmq/src/Broadcasting/Broadcaster/ZmqBroadcaster.php(37): Pelim\LaravelZmq\Zmq->publish()
#4 /home/hunter/bet/Laravel-Admin-Panel-Course/vendor/laravel/framework/src/Illuminate/Broadcasting/BroadcastEvent.php(65): Pelim\LaravelZmq\Broadcasting\Broadcaster\ZmqBroadcaster->broadcast()
#5 /home/hunter/bet/Laravel-Admin-Panel-Course/vendor/laravel/framework/src/Illuminate/Conta in /home/hunter/bet/Laravel-Admin-Panel-Course/vendor/pelim/laravel-zmq/src/Connector/ZmqPublish.php on line 28

In it code I try to use exists context. Maybe it Pull request need place in separate, testing branch. Now I testing it code and it work as I need.