Minor readme fixes, disable sniffer loggin by default

yabeda-rb · Mar 23, 2020 · 222dca7 · 222dca7
1 parent 4b26abd
commit 222dca7
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -7,6 +7,8 @@
 
 Built-in metrics for external services HTTP calls! This gem is a Part of the [yabeda](https://github.com/yabeda-rb/yabeda) suite.
 
+Read [introduction article on dev.to](https://dev.to/dsalahutdinov/monitoring-external-services-with-prometheus-and-grafana-5eh6).
+
 ## Metrics
 
 Works as the Puma plugin and provides following metrics:
@@ -63,7 +65,7 @@ After docker image builds and all the services get up, you can browse applicatio
 
 Follow the [yabeda-external-http-requests](http://localhost:3000/d/OGd-oEXWz/yabeda-external-http-requests?orgId=1&refresh=5s) dashboard in Grafana.
 Finally, after a couple of minutes when data collected you will see the following:
-![Monitor external HTTP calls with Grafana](docs/http_requests_rate_by_host.png)
+![Monitor external HTTP calls with Grafana](docs/dashboard.png)
 
 ## Development with Docker
 

diff --git a/docs/dashboard.png b/docs/dashboard.png
diff --git a/docs/http_requests_rate_by_host.png b/docs/http_requests_rate_by_host.png
diff --git a/lib/yabeda/http_requests.rb b/lib/yabeda/http_requests.rb
@@ -8,7 +8,7 @@
 module Yabeda
   # Common module
   module HttpRequests
-    SNIFFER_STORAGE_SIZE = 1
+    SNIFFER_STORAGE_SIZE = 0
 
     Yabeda.configure do
       group :http
@@ -37,6 +37,7 @@ module HttpRequests
         c.enabled = true
         c.store = { capacity: SNIFFER_STORAGE_SIZE }
         c.middleware do |chain|
+          chain.remove(Sniffer::Middleware::Logger)
           chain.add(Yabeda::HttpRequests::Sniffer)
         end
       end