aderyabin/sniffer

Broken support of EventMachine

sclinede opened this issue · 0 comments

When I'm trying to use Sniffer with Thin - I'm failing on Rails server start with Runtime Error.

Problem happens with the string EventMachine::HttpClient.send(:include, Sniffer::Adapters::EventMachineAdapter::Client) if defined?(::EventMachine).
When you use Thin - EventMachine is defined, but EventMachine::HttpClient is implemented in separate gem called em-http-request.

Do you want me to change the EvenMachineAdapter (to verify that EventMachine::HttpClient is defined) , or you could fix in other "right" way?