search

celluloid / reel

UNMAINTAINED: See celluloid/celluloid#779 - Celluloid::IO-powered web server
https://celluloid.io
MIT License
596 stars 87 forks source link

Raise SocketError on all possible Socket Errors #105

Closed Asmod4n closed 10 years ago

Asmod4n commented 10 years ago

I don't know if this is desired, but shouldn't a SocketError be raised on IOError, Errno::ECONNRESET and Errno::EPIPE?

coveralls commented 10 years ago

Coverage Status

Coverage decreased (-2.22%) when pulling db25be7c65325d927f01eb409a01fd517c568518 on Asmod4n:patch-3 into ea7124bfa7bda45caa11dff68bf0919616f3de7b on celluloid:master.

tarcieri commented 10 years ago

Yep, thanks! I also intend to eventually make an extraction from Reel that wraps all this stuff up as well. Celluloid::IO::ServerCell or something.