Added EmailSubject null/empty check to overload

serilog / serilog-sinks-email

A Serilog sink that writes events to SMTP email

Apache License 2.0

70 stars 68 forks source link

Added EmailSubject null/empty check to overload #89

Closed Gondost closed 6 months ago

Gondost commented 3 years ago

One of the EmailConnectionInfo overloads was not doing a null or empty check on mailSubject, so it was using the default subject instead of the one provided in connectionInfo.

nblumhardt commented 6 months ago

Thanks for this, sorry about the lack of response, time constraints have kept me from digging into this repository for some time. #122 should now fix this issue by virtue of having removed a number of those overloads, and changed the handling of default subjects etc. (They're now nullable and default to null.) Thanks again!