Merge pull request #1329 from slingamn/issue1327

fix #1327

irc/database.go

@@ -24,7 +24,7 @@ const (
 	// 'version' of the database schema
 	keySchemaVersion = "db.version"
 	// latest schema of the db
-	latestDbSchema = "14"
+	latestDbSchema = "15"
 
 	keyCloakSecret = "crypto.cloak_secret"
 )
@@ -792,6 +792,26 @@ func schemaChangeV13ToV14(config *Config, tx *buntdb.Tx) error {
 	return nil
 }
 
+// #1327: delete any invalid klines
+func schemaChangeV14ToV15(config *Config, tx *buntdb.Tx) error {
+	prefix := "bans.klinev2 "
+	var keys []string
+	tx.AscendGreaterOrEqual("", prefix, func(key, value string) bool {
+		if !strings.HasPrefix(key, prefix) {
+			return false
+		}
+		if key != strings.TrimSpace(key) {
+			keys = append(keys, key)
+		}
+		return true
+	})
+	// don't bother trying to fix these up
+	for _, key := range keys {
+		tx.Delete(key)
+	}
+	return nil
+}
+
 func init() {
 	allChanges := []SchemaChange{
 		{
@@ -859,6 +879,11 @@ func init() {
 			TargetVersion:  "14",
 			Changer:        schemaChangeV13ToV14,
 		},
+		{
+			InitialVersion: "14",
+			TargetVersion:  "15",
+			Changer:        schemaChangeV14ToV15,
+		},
 	}
 
 	// build the index

irc/strings.go

@@ -222,6 +222,7 @@ func realSkeleton(name string) (string, error) {
 // Shivaram@good-fortune -> *!shivaram@good-fortune
 // EDMUND -> edmund!*@*
 func CanonicalizeMaskWildcard(userhost string) (expanded string, err error) {
+	userhost = strings.TrimSpace(userhost)
 	var nick, user, host string
 	bangIndex := strings.IndexByte(userhost, '!')
 	strudelIndex := strings.IndexByte(userhost, '@')

irc/strings_test.go

@@ -215,6 +215,7 @@ func TestCanonicalizeMaskWildcard(t *testing.T) {
 	tester("shivaram*", "shivaram*!*@*", nil)
 	tester("Shivaram*", "shivaram*!*@*", nil)
 	tester("*SHIVARAM*", "*shivaram*!*@*", nil)
+	tester("*SHIVARAM*   ", "*shivaram*!*@*", nil)
 }
 
 func validFoldTester(first, second string, equal bool, folder func(string) (string, error), t *testing.T) {