search

jimsynz / faye-rails

Simple Rails glue for the Faye messaging protocol.
MIT License
435 stars 79 forks source link

strip null bytes from channel name #66

Closed godsent closed 9 years ago

godsent commented 9 years ago

The problem described in https://github.com/jamesotron/faye-rails/issues/65

jimsynz commented 9 years ago

Why so closed? Just haven't had time to review yet.

godsent commented 9 years ago

It didn't fix all issues. My collegue will create new merge request from https://github.com/SoftSwiss/faye-rails/commit/d698e733c9383a4bc4ab7dd68848f422ef33728b commit with better solution.

Bishop commented 9 years ago

It here: #67