search

google / quiver-log

Logging utility for Dart
Apache License 2.0
24 stars 14 forks source link

Don't silently fail if an appender fails #18

Open dballesteros7 opened 5 years ago

dballesteros7 commented 5 years ago

This line should do something:

https://github.com/google/quiver-log/blob/master/lib/src/log.dart#L39

At least in debug mode, otherwise it can lead to hard to debug issues with logging which is critical during development.

adambender commented 5 years ago

I could drop an assert() in here which would break in development mode. Would that solve the problem you ran into? Maybe you can explain a bit more about the issue you had? It might help me to ensure I come up with a reasonable fix.

No matter what I try, I do think it is important to preserve the property that the logger can't take the app down in production.