aderyabin / sniffer

Log and Analyze Outgoing HTTP Requests

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add Excon adapter

aderyabin opened this issue · comments

Add Excon adapter

Привет. Есть идея сделать это через middleware как тут https://github.com/vcr/vcr/blob/master/lib/vcr/middleware/excon.rb

единственное, что оно не очень корректно будет считать время, затраченное за запрос (с учетом других подключенных middleware), но зато код будет красивый 😄 Если норм идея - поробую наваять пул-реквест

А почему не сделать также как сделано везде?

Дак можно конечно, только прийдется весь вот этот метод перекрыть чтобы зафиксировать факт начала отравки данных, и аналогичные длинные методы на респонз
И это очень нежное место получается - что мы сильно завязываемся на внутренню реализацию библиотеки и при малейшем рефакторинге в ней - все сломается 😞

Тут да, надо будет посмотреть исходники как следует