Remove invalid characters from DC name

uchiwa/config/config.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"math/rand"
 	"os"
+	"strings"
 
 	"github.com/palourde/logger"
 	"github.com/sensu/uchiwa/uchiwa/auth"
@@ -99,6 +100,10 @@ func (c *Config) initSensu() {
 			logger.Warningf("Sensu API %s has no name property. Generating random one...", api.URL)
 			c.Sensu[i].Name = fmt.Sprintf("sensu-%v", rand.Intn(100))
 		}
+		// escape special characters in DC name
+		r := strings.NewReplacer(":", "", "/", "", ";", "", "?", "")
+		c.Sensu[i].Name = r.Replace(api.Name)
+
 		if api.Host == "" {
 			logger.Fatalf("Sensu API %q Host is missing", api.Name)
 		}

uchiwa/config/config_test.go

@@ -20,7 +20,7 @@ func TestLoad(t *testing.T) {
 	assert.NotEqual(t, "*****", conf.Uchiwa.User, "Uchiwa user in private config shouldn't be masked")
 	assert.NotEqual(t, "*****", conf.Uchiwa.Pass, "Uchiwa pass in private config shouldn't be masked")
 	for i := range conf.Sensu {
-		assert.NotEqual(t, "*****",conf.Sensu[i].User, "Sensu APIs user in private config shouldn't be masked")
+		assert.NotEqual(t, "*****", conf.Sensu[i].User, "Sensu APIs user in private config shouldn't be masked")
 		assert.NotEqual(t, "*****", conf.Sensu[i].Pass, "Sensu APIs pass in private config shouldn't be masked")
 	}
 	assert.Equal(t, 1, len(conf.Uchiwa.Users))
@@ -55,3 +55,20 @@ func TestLoadArrayOfUsersOnPublicGet(t *testing.T) {
 	public := conf.GetPublic()
 	assert.Equal(t, 0, len(public.Uchiwa.Users))
 }
+
+func TestInitSensu(t *testing.T) {
+	c := Config{
+		Sensu: []SensuConfig{
+			{Name: "foo ? bar", Host: "127.0.0.1"},
+			{Name: "bar / foo", Host: "127.0.0.1"},
+		},
+	}
+
+	c.initSensu()
+
+	expectedConfig := []SensuConfig{
+		{Name: "foo  bar", Host: "127.0.0.1", Port: 4567, Ssl: false, Insecure: false, URL: "http://127.0.0.1:4567", User: "", Path: "", Pass: "", Timeout: 10},
+		{Name: "bar  foo", Host: "127.0.0.1", Port: 4567, Ssl: false, Insecure: false, URL: "http://127.0.0.1:4567", User: "", Path: "", Pass: "", Timeout: 10},
+	}
+	assert.Equal(t, expectedConfig, c.Sensu)
+}