dariocravero / padrino-websockets

Agnostic websockets support for Padrino
MIT License
14 stars 9 forks source link

send_message on error #4

Open grillbiff opened 9 years ago

grillbiff commented 9 years ago

Hi,

I had trouble getting the send_message when rescuing working on the Faye implementation. Not sure if I registered padrino-websockets correctly into padrino or if it is in fact a bug.

padrino-websockets/lib/padrino-websockets/base-event-manager.rb:101:in `rescue in on_message': undefined method `send_message' for #<Padrino::WebSockets::Faye::EventManager:0x00000106813c40> (NoMethodError)

Anyway, I uploaded a fix, if you want me to create a pull request let me know https://github.com/grillbiff/padrino-websockets/compare/dariocravero:master...master

dariocravero commented 9 years ago

Hey @grillbiff, sorry about this ridiculous delay. Please do. Darío