Fix some logging inefficiencies

lostisland / faraday

Simple, but flexible HTTP client library, with support for multiple backends.

https://lostisland.github.io/faraday

MIT License

5.71k stars 972 forks source link

Closed semaperepelitsa closed 1 year ago

semaperepelitsa commented 1 year ago

When reviewing Faraday logger I noticed two inefficiencies:

It validates :log_level option on every log message, when it could only do it once during initialisation.
It allocates a Proc object on every log message for no reason.

This PR fixes both issues.

semaperepelitsa commented 1 year ago

@iMacTia, done!

cc @amrrbakry