AvroTurf::Messaging uses a logger inside that defaults to Logger.new($stderr). This makes these logs behave differently from the other deimos logs. A custom log can be provided upon initialisation, so it makes sense to pass the configurable Deimos logger there.
Type of change
Not really sure if this qualifies as a breaking change, considering the avro_turf logs will now be sent elsewhere compared to before. But to be honest this difference in behaviour was unexpected and in my mind, this looks more like a bugfix.
How Has This Been Tested?
[x] Tested locally pointing bundler to the modified gem
dorner
commented
2 years ago
Description
AvroTurf::Messaging uses a logger inside that defaults to
Logger.new($stderr). This makes these logs behave differently from the other deimos logs. A custom log can be provided upon initialisation, so it makes sense to pass the configurable Deimos logger there.Type of change
Not really sure if this qualifies as a breaking change, considering the
avro_turflogs will now be sent elsewhere compared to before. But to be honest this difference in behaviour was unexpected and in my mind, this looks more like a bugfix.How Has This Been Tested?