fix: check for number in the semver prefix (#291)

* check for number in the semver prefix

* use isNumber

Author: thomaszurkan-optimizely

pkg/decision/evaluator/matchers/semver.go

@@ -128,6 +128,12 @@ func (sv SemanticVersion) splitSemanticVersion(targetedVersion string) ([]string
 		return []string{}, errors.New(string(reasons.AttributeFormatInvalid))
 	}
 
+	for i := 0; i < len(targetedVersionParts);i++ {
+		if  !sv.isNumber(targetedVersionParts[i]) {
+			return []string{}, errors.New(string(reasons.AttributeFormatInvalid))
+		}
+	}
+
 	targetedVersionParts = append(targetedVersionParts, targetSuffix...)
 	return targetedVersionParts, nil
 }

pkg/decision/evaluator/matchers/semver_test.go

@@ -199,6 +199,19 @@ func TestInvalidAttributes(t *testing.T) {
 		37,
 		nil,
 		"",
+		"-",
+		".",
+		"..",
+		"+",
+		"+test",
+		" ",
+		"2 .3. 0",
+		"2.",
+		".2.2",
+		"3.7.2.2",
+		"3.x",
+		",",
+		"+build-prerelese",
 	}
 
 	for _, matchType := range matchTypes {