check endpoint of OpenSergoClient

Signed-off-by: Jiangnan Jia <[email protected]>

Author: jnan806

src/main/java/io/opensergo/OpenSergoClient.java

@@ -52,8 +52,8 @@ public class OpenSergoClient implements AutoCloseable {
 
     public static class Builder {
 
-        private String host;
-        private int port;
+        private String host = "localhost";
+        private int port = 10246;
         private OpenSergoClientConfig openSergoConfig;
 
         public OpenSergoClient.Builder endpoint(String host, int port) {
@@ -83,6 +83,7 @@ public OpenSergoClient(String host, int port) {
     }
 
     public OpenSergoClient(String host, int port, OpenSergoClientConfig clientConfig) {
+        checkEndpoint(host, port);
         checkClientConfig(clientConfig);
         // TODO: support TLS
         this.clientConfig = clientConfig;
@@ -95,6 +96,11 @@ public OpenSergoClient(String host, int port, OpenSergoClientConfig clientConfig
         this.reqId = new AtomicInteger(0);
     }
 
+    private void checkEndpoint(String host, int port) {
+        AssertUtils.notEmpty(host, "host cannot be empty, need to give a valid host");
+        AssertUtils.isTrue(port > 0, "port < 1 is invalid, need to give a valid port");
+    }
+
     private void checkClientConfig(OpenSergoClientConfig clientConfig) {
         AssertUtils.assertNotNull(clientConfig, "clientConfig cannot be null");
     }