Merge pull request #2938 from immarvin/onretcode

fix issue #2636: xcat command returns 0 when cmd fails due authorization

perl-xCAT/xCAT/Client.pm

@@ -265,7 +265,11 @@ sub submit_request {
             %sslargs,
         );
     } else {
+        print "warning: the client certificates under $homedir/.xcat/ are not setup correctly, please run '/opt/xcat/share/xcat/scripts/setup-local-client.sh"." $ENV{'USER'}' as 'root' to generate the client certificates; otherwise, the SSL connection between xcat client and xcatd will be setup without certificate verification and open to Man-In-The-Middle attacks.\n";
+        #Using the default of SSL_verify_mode of SSL_VERIFY_NONE for client is deprecated!
+        #need to specify SSL_verify_mode => SSL_VERIFY_NONE explicitly 
         $client = IO::Socket::SSL->start_SSL($pclient,
+            SSL_verify_mode => SSL_VERIFY_NONE,
             Timeout => 0,
         );
     }