search

discourse / prometheus_exporter

A framework for collecting and aggregating prometheus metrics
MIT License
525 stars 153 forks source link

Fix logger for web_server #185

Closed vin01 closed 2 years ago

vin01 commented 2 years ago

Hi,

it looks like this has been broken since https://github.com/discourse/prometheus_exporter/commit/c3e26a827b59c225a2ffb34eec57d8541e6ee97f

I just noticed this when there was a silent 500 because of a timeout and got:

 ERROR NameError: undefined local variable or method `logger' for #<PrometheusExporter::Server::WebServer:0x000055f23ed4db50>\nDid you mean?  @logger
SamSaffron commented 2 years ago

nice catch!