PickupDirectoryLocation - SmtpServer is no longer RequiredParameter (#…

…179)
NLog · Aug 30, 2023 · 78fee64 · 78fee64
1 parent 8a25f6b
commit 78fee64
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/NLog.MailKit/MailTarget.cs b/src/NLog.MailKit/MailTarget.cs
@@ -177,7 +177,6 @@ public Layout Body
         /// Gets or sets SMTP Server to be used for sending.
         /// </summary>
         /// <docgen category='SMTP Options' order='10' />
-        [RequiredParameter]
         public Layout SmtpServer { get; set; }
 
         /// <summary>
@@ -467,7 +466,12 @@ private void CheckRequiredParameters()
             var smtpAuthentication = RenderLogEvent(SmtpAuthentication, LogEventInfo.CreateNullEvent());
             if (smtpAuthentication == SmtpAuthenticationMode.Ntlm)
             {
-                throw new NLogConfigurationException("NTLM not yet supported");
+                throw new NLogConfigurationException("SmtpAuthentication NTLM not yet supported");
+            }
+
+            if (PickupDirectoryLocation is null && SmtpServer is null)
+            {
+                throw new NLogConfigurationException("SmtpServer is required");
             }
         }