search

celluloid / reel

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

Refactor: +SocketMixin #114

Closed digitalextremist closed 10 years ago

digitalextremist commented 10 years ago

Right now, socket optimizations happen per-connection vs. per-server, and SSLServer is not covered... which this refactor resolves by using SocketMixin across both servers, and going full per-server vs. per-connection.

digitalextremist commented 10 years ago

This might conflict some with #113. Not sure what order wouldn't produce a conflict.

digitalextremist commented 10 years ago

This seems like a good time to also add a Keep-Alive timeout option, rather than having a hard-coded default, which also I cannot seem to find. Seems like there might not even be a default, and I see threads staying open purely for Keep-Alive now that pipelining is supported. Do you know where I can add this @tarcieri or @halorgium?

tarcieri commented 10 years ago

@digitalextremist the timeout situation is a bit dire. I'm in the process of refactoring all the requests and response state machines into Celluloid::FSMs