GH-129 - Allow . (dot) in key name

pkg/artifactory/resource_artifactory_remote_repository_test.go

@@ -9,7 +9,36 @@ import (
 
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/resource"
 )
+func TestAccLocalAllowDotsAndDashesInKeyGH129(t *testing.T) {
+	_, fqrn, name := mkNames("terraform-local-test-repo-basic", "artifactory_remote_repository")
 
+	key := fmt.Sprintf("debian-remote.teleport%d",randomInt())
+	localRepositoryBasic := fmt.Sprintf(`
+		resource "artifactory_remote_repository" "%s" {
+			key              = "%s"
+			package_type     = "debian"
+			repo_layout_ref  = "simple-default"
+			url              = "https://deb.releases.teleport.dev/"
+			notes            = "managed by terraform"
+			property_sets    = ["artifactory"]
+			includes_pattern = "**/*"
+			content_synchronisation {
+				enabled = false
+			}
+		}
+	`,name, key )
+	resource.Test(t, resource.TestCase{
+		PreCheck:     func() { testAccPreCheck(t) },
+		CheckDestroy: testAccCheckRepositoryDestroy(fqrn),
+		Providers:    testAccProviders,
+		Steps: []resource.TestStep{
+			{
+				Config: localRepositoryBasic,
+				Check: resource.TestCheckResourceAttr(fqrn, "key", key),
+			},
+		},
+	})
+}
 func TestKeyHasSpecialCharsFails(t *testing.T) {
 	const failKey = `
 		resource "artifactory_remote_repository" "terraform-remote-test-repo-basic" {

pkg/artifactory/util.go

@@ -160,27 +160,7 @@ func mkLens(d *schema.ResourceData) func(key string, value interface{}) []error
 	}
 }
 
-// HashStrings hashcode was moved to internal in terraform-plugin-sdk, and schema does not expose a wrapper of hashcode.Strings
-func HashStrings(strings []string) string {
-	var buf bytes.Buffer
 
-	for _, s := range strings {
-		buf.WriteString(fmt.Sprintf("%s-", s))
-	}
-
-	return fmt.Sprintf("%d", schema.HashString(buf.String()))
-}
-func cascadingErr(hasErr *bool) func(error) {
-	if hasErr == nil {
-		panic("hasError cannot be nil")
-	}
-	return func(err error) {
-		if err != nil {
-			fmt.Println(err)
-			*hasErr = true
-		}
-	}
-}
 
 func sendConfigurationPatch(content []byte, m interface{}) error {
 

pkg/artifactory/validators.go

@@ -530,7 +530,7 @@ func validateIsEmail(address interface{}, _ string) ([]string, []error) {
 
 var repoKeyValidator = validation.All(
 	validation.StringDoesNotMatch(regexp.MustCompile("^[0-9].*"), "repo key cannot start with a number"),
-	validation.StringDoesNotContainAny(" !@#$%^&*()_+={}[]:;<>,./?~`|\\"),
+	validation.StringDoesNotContainAny(" !@#$%^&*()_+={}[]:;<>,/?~`|\\"),
 )
 
 func fileExist(value interface{}, _ string) ([]string, []error) {