celluloid / reel

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

remove rack dep, add fwd to ping #188

Closed kenichi closed 9 years ago

kenichi commented 9 years ago

this should resolve #186 and #187 - in my testing, this limited amount of rack mimicry works, no need for a full gem dependency and Rack::MockRequest instantiation.

also, this adds a forward to the driver for WebSocket#ping.

digitalextremist commented 9 years ago

So far, extremely valiant and speedy effort @kenichi. Wondering if we have all the tests needed to verify this works in the wild.

digitalextremist commented 9 years ago

When you're ready, re-pull this to the 0.6.0-milestone branch please. We're still cutting off that until Garland lands fully.

tarcieri commented 9 years ago

@jcoglan any thoughts here?

kenichi commented 9 years ago

ah, oops, i'll rebase from that branch.

kenichi commented 9 years ago

see #189