Merge pull request #1547 from MartinTherriault/master

LogReceiverWebServiceTarget is leaking communication channels
NLog · Jul 19, 2016 · 698ca65 · 698ca65
2 parents 72bfc70 + 0fb4250
commit 698ca65
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/src/NLog/Targets/LogReceiverWebServiceTarget.cs b/src/NLog/Targets/LogReceiverWebServiceTarget.cs
@@ -401,10 +401,17 @@ protected virtual IWcfLogReceiverClient CreateLogReceiver()
 
         private void ClientOnProcessLogMessagesCompleted(object sender, AsyncCompletedEventArgs asyncCompletedEventArgs)
         {
-            var client = sender as WcfLogReceiverClient;
+            var client = sender as IWcfLogReceiverClient;
             if (client != null && client.State == CommunicationState.Opened)
             {
-                client.CloseCommunicationObject();
+                try
+                {
+                    client.Close();
+                }
+                catch
+                {
+                    client.Abort();
+                }
             }
         }
 #endif