we shouldn't log to the console from a library

rdavisau · Jan 10, 2016 · 463bdaf · 463bdaf
1 parent 99bc1f1
commit 463bdaf
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/.gitignore b/.gitignore
@@ -191,4 +191,9 @@ ModelManifest.xml
 # Unwanted items
 nuget.exe
 *.nupkg
-*.DotSettings
+*.DotSettings
+/Sockets/Sockets.Plugin.Abstractions/Properties/Resources.resx
+/Sockets/Sockets.Plugin.Abstractions/Properties/Resources.Designer.cs
+/Sockets/Sockets.Plugin.Abstractions/Properties
+/Sockets/Samples/SocketsSample.XF/SocketsSample.XF.Droid/SocketsSample.XF.Droid.csproj.bak
+/Sockets/Samples/SocketsSample.XF/SocketsSample.XF.Droid
diff --git a/Sockets/Sockets.Implementation.NET/TcpSocketClient.cs b/Sockets/Sockets.Implementation.NET/TcpSocketClient.cs
@@ -78,18 +78,12 @@ private bool ServerValidationCallback (object sender, X509Certificate certificat
             switch (sslPolicyErrors)
             {
                 case SslPolicyErrors.RemoteCertificateNameMismatch:
-                    Console.WriteLine("Server name mismatch. End communication ...\n");
                     return false;
                 case SslPolicyErrors.RemoteCertificateNotAvailable:
-                    Console.WriteLine("Server's certificate not available. End communication ...\n");
                     return false;
                 case SslPolicyErrors.RemoteCertificateChainErrors:
-                    Console.WriteLine("Server's certificate validation failed. End communication ...\n");
                     return false;
             }
-            //TODO: Perform others checks using the "certificate" and "chain" objects ...
-
-            Console.WriteLine("Server's authentication succeeded ...\n");
             return true;
         }