-
-
Notifications
You must be signed in to change notification settings - Fork 21
Open
Labels
Description
I have a test install running at work and noticed something: even with ignoreSslErrors=true in my config a message is being printed to NLog internal logging.
I figure that if I'm not interested in SSL errors, nothing should be printed to NLog's internal log.
It seems to be coming from this line.
NLog.Targets.Splunk/src/NLog.Targets.Splunk/Splunk.Logging.Common/HttpEventCollectorSender.cs
Line 599 in fb14598
| OnError(new HttpEventCollectorException(HttpStatusCode.NotAcceptable, reply: warning)); |
Perhaps you could refactor this into a warnOnSslErrors boolean as a separate option, or maybe turn the ignore boolean into an enum ("yes", "no", "warn").