search

celluloid / celluloid-io

UNMAINTAINED: See celluloid/celluloid#779 - Evented sockets for Celluloid actors
https://celluloid.io
MIT License
879 stars 93 forks source link

Review of net-http-celluloid-io gem modifications #183

Closed kamk closed 7 years ago

kamk commented 7 years ago

I have forked net-http-celluloid-io gem. I would be appreciate if anyone could review the code of current version. Tests don't seem to work as original gem is based on older version of Celluloid::IO

tarcieri commented 7 years ago

I'm afraid I don't know anything about this library and don't have time to review it.

This is an issue tracker for bugs in Celluloid. If you would like to find people who are interested in reviewing this, I would suggest posting to the Celluloid Google Group:

https://groups.google.com/forum/#!forum/celluloid-ruby

ioquatix commented 6 years ago

Interesting fork. I don't have time to review it, but I can point you towards a more recent attempt for concurrent Ruby IO: https://github.com/socketry/async-io

There is some existing integration with Net::HTTP in here https://github.com/socketry/async-io/blob/master/spec/async/io/wrap/tcp_spec.rb