optimize the certificate notice (#301)

Author: cyjseagull

sdk-core/src/main/java/org/fisco/bcos/sdk/network/ConnectionManager.java

@@ -359,7 +359,7 @@ private boolean checkConnectionResult(
             /** connect success, check ssl handshake result. */
             SslHandler sslhandler = connectFuture.channel().pipeline().get(SslHandler.class);
             String checkerMessage =
-                    "! Please check the certificate and ensure that the SDK and the node are in the same agency!";
+                    "! Please make sure the certificate is correctly configured and copied, ensure that the SDK and the node are in the same agency!";
             if (Objects.isNull(sslhandler)) {
                 String sslHandshakeFailedMessage =
                         " ssl handshake failed:/"

sdk-core/src/main/java/org/fisco/bcos/sdk/network/NetworkException.java

@@ -29,12 +29,12 @@ public class NetworkException extends Exception {
     private int errorCode = 0;
 
     public NetworkException(String message, int errorCode) {
-        super(message + "\n" + SystemInformation.getSystemInformation());
+        super(message);
         this.errorCode = errorCode;
     }
 
     public NetworkException(String message) {
-        super(message + "\n" + SystemInformation.getSystemInformation());
+        super(message);
     }
 
     public NetworkException(Throwable cause) {

sdk-core/src/main/java/org/fisco/bcos/sdk/network/NetworkImp.java

@@ -25,6 +25,7 @@
 import org.fisco.bcos.sdk.config.exceptions.ConfigException;
 import org.fisco.bcos.sdk.model.CryptoType;
 import org.fisco.bcos.sdk.model.Message;
+import org.fisco.bcos.sdk.utils.SystemInformation;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -152,30 +153,39 @@ private CheckCertExistenceResult checkCertExistence(boolean isSM) {
     public void start() throws NetworkException {
         boolean tryEcdsaConnect = false;
         CheckCertExistenceResult result = null;
+        String ecdsaCryptoInfo = configOption.getCryptoMaterialConfig().toString();
         try {
             try {
                 result = checkCertExistence(false);
                 if (result.isCheckPassed()) {
                     logger.debug("start connManager with ECDSA sslContext");
                     connManager.startConnect(configOption);
                     connManager.startReconnectSchedule();
-                    tryEcdsaConnect = true;
                     return;
                 } else {
                     logger.warn(
-                            "Try to connect node with ECDSA sslContext failed, expected certPath: "
-                                    + configOption.getCryptoMaterialConfig().toString()
+                            "Try to connect node with ECDSA sslContext failed, the tried NON-SM certPath: "
+                                    + ecdsaCryptoInfo
                                     + ", currentPath: "
                                     + new File("").getAbsolutePath());
                 }
             } catch (NetworkException e) {
-                tryEcdsaConnect = true;
                 configOption.reloadConfig(CryptoType.SM_TYPE);
-                result = checkCertExistence(true);
-                if (e.getErrorCode() == NetworkException.CONNECT_FAILED
-                        || !result.isCheckPassed()) {
-                    throw e;
+                if (e.getErrorCode() == NetworkException.CONNECT_FAILED) {
+                    String errorMessage = e.getMessage();
+                    errorMessage +=
+                            "\n* If your blockchain is NON-SM, please provide the NON-SM certificates: "
+                                    + ecdsaCryptoInfo
+                                    + ".\n";
+                    errorMessage +=
+                            "\n* If your blockchain is SM, please provide the SM certificates: "
+                                    + configOption.getCryptoMaterialConfig().toString()
+                                    + "\n";
+                    throw new NetworkException(
+                            errorMessage + "\n" + SystemInformation.getSystemInformation());
                 }
+                // means that all the ECDSA certificates exist
+                tryEcdsaConnect = true;
                 connManager.stopNetty();
                 logger.debug(
                         "start connManager with the ECDSA sslContext failed, try to use SM sslContext, error info: {}",
@@ -186,24 +196,49 @@ public void start() throws NetworkException {
             result = checkCertExistence(true);
             if (!result.isCheckPassed()) {
                 if (tryEcdsaConnect) {
-                    throw new NetworkException("Certificate not exist:" + result.getErrorMessage());
+                    String errorMessage =
+                            "\n* Try init the sslContext failed.\n\n* If your blockchain channel config is NON-SM, please provide the NON-SM certificates: "
+                                    + ecdsaCryptoInfo
+                                    + ".\n";
+                    errorMessage +=
+                            "\n* If your blockchain channel config is SM, please provide the missing certificates: "
+                                    + result.getErrorMessage()
+                                    + "\n";
+                    throw new NetworkException(errorMessage);
                 } else {
-                    throw new NetworkException(
-                            "Not providing all the certificates to connect to the node! Please provide the certificates to connect with the block-chain, expected certPath: ["
+                    String errorMessage =
+                            "\n# Not providing all the certificates to connect to the node! Please provide the certificates to connect with the block-chain.\n";
+                    errorMessage +=
+                            "\n* If your blockchain is NON-SM, please provide the NON-SM certificates: "
+                                    + ecdsaCryptoInfo
+                                    + ". \n";
+                    errorMessage +=
+                            "\n* If your blockchain is SM, please provide the SM certificates: "
                                     + configOption.getCryptoMaterialConfig().toString()
-                                    + "]");
+                                    + "\n";
+                    throw new NetworkException(errorMessage);
                 }
             }
-            if (tryEcdsaConnect) {
+            try {
                 // create a new connectionManager to connect the node with the SM sslContext
                 connManager = new ConnectionManager(configOption, handler);
+                connManager.startConnect(configOption);
+                connManager.startReconnectSchedule();
+            } catch (NetworkException e) {
+                String errorMessage = e.getMessage();
+                errorMessage +=
+                        "\n* If your blockchain channel config is NON-SM, please provide the NON-SM certificates: "
+                                + ecdsaCryptoInfo
+                                + ".\n";
+                errorMessage +=
+                        "\n* If your blockchain channel config is SM, please provide the SM certificates: "
+                                + configOption.getCryptoMaterialConfig().toString()
+                                + "\n";
+                throw new NetworkException(
+                        errorMessage + "\n" + SystemInformation.getSystemInformation());
             }
-            connManager.startConnect(configOption);
-            connManager.startReconnectSchedule();
         } catch (ConfigException e) {
-            throw new NetworkException(
-                    "start connManager with the SM algorithm failed, error info: " + e.getMessage(),
-                    e);
+            throw new NetworkException(e);
         }
     }