zendframework / zend-expressive-swoole

Swoole support for Expressive applications
BSD 3-Clause "New" or "Revised" License
158 stars 14 forks source link

Bravo!! Is it ready for production environment ? #6

Closed huangzhhui closed 6 years ago

huangzhhui commented 6 years ago

Bravo!! Is it ready for production environment ? I would like to use it !

ezimuel commented 6 years ago

@huangzhhui Thanks! Not yet, we are working on it and we'll release asap.

huangzhhui commented 6 years ago

@ezimuel Have any todo list or bug list about this project ? I will try to complete some issue, I am the heavy user of Zend Framework, And I am the author of Swoft ( Most popular coroutine framework base on Swoole ), I have experience with this

ezimuel commented 6 years ago

@huangzhhui thanks a ton for your help! I provided a short To Do list for the Swoole integration here.

huangzhhui commented 6 years ago

@ezimuel I notice that the issues in Project rely on your un-merged pr https://github.com/zendframework/zend-expressive-swoole/pull/5 , I would like to know when you might merge this pr.

ezimuel commented 6 years ago

@huangzhhui very soon, waiting for @weierophinney review.

ezimuel commented 6 years ago

@huangzhhui maybe this can be interesting for you https://github.com/ezimuel/swoole-benchmark

huangzhhui commented 6 years ago

@ezimuel The results of your test are interesting, but usually we use Nginx for static files, cuz HTTP Protocol Support of Swoole is not well, https and e-tag etc.

weierophinney commented 6 years ago

@huangzhhui You're absolutely correct - we should likely allow users to define a suite of headers to use by default with static files as well. I've created #10 to track it.

huangzhhui commented 6 years ago

@weierophinney @ezimuel sendfile also does not support gzip, it is not really good for static file transport.

ezimuel commented 6 years ago

@huangzhhui I know that Swoole\Http\Response::sendfile() does not support gzip, but we can implement it using the Zlib functions of PHP.