Add ":" as acceptable char for node ID

Author: matoval

docs/source/user_guide/configuration_options.rst

@@ -130,7 +130,7 @@ Node
       - Default value
       - Type
     * - ``id``
-      - Node ID can only contain a-z, A-Z, 0-9 or special characters . - _ @
+      - Node ID can only contain a-z, A-Z, 0-9 or special characters . - _ @ :
       - local hostname
       - string
     * - ``datadir``

pkg/types/main.go

@@ -36,10 +36,10 @@ func (cfg NodeCfg) Init() error {
 		}
 		cfg.ID = host
 	} else {
-		submitIDRegex := regexp.MustCompile(`^[.\-_@a-zA-Z0-9]*$`)
+		submitIDRegex := regexp.MustCompile(`^[.\-_@:a-zA-Z0-9]*$`)
 		match := submitIDRegex.FindSubmatch([]byte(cfg.ID))
 		if match == nil {
-			return fmt.Errorf("node id can only contain a-z, A-Z, 0-9 or special characters . - _ @ but received: %s", cfg.ID)
+			return fmt.Errorf("node id can only contain a-z, A-Z, 0-9 or special characters . - _ @ : but received: %s", cfg.ID)
 		}
 	}
 	if strings.ToLower(cfg.ID) == "localhost" {

pkg/types/main_test.go

@@ -0,0 +1,36 @@
+package types
+
+import "testing"
+
+func TestMainInitNodeID(t *testing.T) {
+	mainInitNodeIDTestCases := []struct {
+		name 				string
+		nodeID 			string
+		expectedErr string
+	}{
+		{
+			name: "successful, no error",
+			nodeID: "t.e-s_t@1:234",
+			expectedErr: "",
+		},
+		{
+			name: "failed, charactered not allowed",
+			nodeID: "test!#&123",
+			expectedErr: "node id can only contain a-z, A-Z, 0-9 or special characters . - _ @ : but received: test!#&123",
+		},
+	}
+
+	for _, testCase := range mainInitNodeIDTestCases {
+		t.Run(testCase.name, func(t *testing.T) {
+			cfg := NodeCfg{
+				ID: testCase.nodeID,
+			}
+			err := cfg.Init()
+			if err == nil && testCase.expectedErr != "" {
+				t.Errorf("exected error but got no error")
+			} else if err != nil && err.Error() != testCase.expectedErr {
+				t.Errorf("expected error to be %s, but got: %s", testCase.expectedErr, err.Error())
+			}
+		})
+	}
+}