Slack webhook channel name cannot be empty or just contain an hashtag (#4786)

2 files changed, 36 insertions, 0 deletions

diff --git a/routers/utils/utils.go b/routers/utils/utils.go
index 6ead7d60e2..7c90fd7048 100644
--- a/routers/utils/utils.go
+++ b/routers/utils/utils.go
@@ -15,3 +15,22 @@ func RemoveUsernameParameterSuffix(name string) string {
 	}
 	return name
 }
+
+// IsValidSlackChannel validates a channel name conforms to what slack expects.
+// It makes sure a channel name cannot be empty and invalid ( only an # )
+func IsValidSlackChannel(channelName string) bool {
+	switch len(strings.TrimSpace(channelName)) {
+	case 0:
+		return false
+	case 1:
+		// Keep default behaviour where a channel name is still
+		// valid without an #
+		// But if it contains only an #, it should be regarded as
+		// invalid
+		if channelName[0] == '#' {
+			return false
+		}
+	}
+
+	return true
+}
diff --git a/routers/utils/utils_test.go b/routers/utils/utils_test.go
index fb56ac85c2..d96e1d7d26 100644
--- a/routers/utils/utils_test.go
+++ b/routers/utils/utils_test.go
@@ -15,3 +15,20 @@ func TestRemoveUsernameParameterSuffix(t *testing.T) {
 	assert.Equal(t, "foobar", RemoveUsernameParameterSuffix("foobar"))
 	assert.Equal(t, "", RemoveUsernameParameterSuffix(""))
 }
+
+func TestIsValidSlackChannel(t *testing.T) {
+	tt := []struct {
+		channelName string
+		expected    bool
+	}{
+		{"gitea", true},
+		{"  ", false},
+		{"#", false},
+		{"gitea   ", true},
+		{"  gitea", true},
+	}
+
+	for _, v := range tt {
+		assert.Equal(t, v.expected, IsValidSlackChannel(v.channelName))
+	}
+}