Support disabling logs, at least for requests forwarded upstream?

[nogweii]

Wondering if you would consider a PR that added a new environment variable to disable logs for HTTP requests sent upstream? Or all logs, if you wanted it more coarse.

I'm running thruster & puma with a Rails application in kubernetes, with probes turned on. That's pinging my healthcheck endpoint a lot. I can disable the logs within Rails, but I don't see a way to stop the logs from thruster itself.

I already have logs from the Rails code that has the information I care about, so the log message from thruster are repetitious for me.

[kevinmcconnell]

@nogweii yes, that's a great idea, thanks for bringing it up. I agree that the request logs can be superfluous in some cases.

I'd lean towards keeping any other logs, in case we need to emit something important, but being able to silence the request logs seems useful. So maybe we can support something like LOG_REQUESTS=false which prevents the logging middleware from being loaded. Default would be to log the requests, as we do now.

Happy to look at a PR for this!