Add info about SeqClientException to README

[tenhobi]

We should add info about SeqClientException to the README. In this library its not catched, which is probably ok, but users should be aware that debug/info/warning/error/... logs might throw.

Backstory: for some reason we are getting Failed host lookup, Connection reset by peer, Connection refused, ... issues and it probably crashes the app, because we are not catching logging?

[ricardoboss]

Hmm, yes that's indeed not a good design. Maybe the library should catch these exceptions and log them to the diagnostic logger instead of throwing them further up the stack?

Other logging libraries differentiate between normal logging (fire and forget) and audit logging (make sure logs are stored). This library effectively only implements audit logging, so one could argue this is the correct behaviour.

However, ideally this library would support both, normal and audit logging, and leave the choice to the user.

At the very least, as you suggested, we should include a warning to users in the README.

[tenhobi]

I will add suggestion (for now) tomorrow.

Thinking about it more, we have to disable exceptions by default. So I will do that tomorrow. + add info to readme how to enable throwing if somebody wants that.