fix: [CDS-101017]: fix unsaved changes in plan for snow url without s…

…lash (#1115)
harness · Nov 17, 2024 · 255764f · 255764f
1 parent 1a9e879
commit 255764f
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/.changelog/1115.txt b/.changelog/1115.txt
@@ -0,0 +1,3 @@
+```relese-note:fix
+harness_platform_connector_service_now: omit difference of slash in service_now_url
+```
diff --git a/internal/service/platform/connector/service_now.go b/internal/service/platform/connector/service_now.go
@@ -2,6 +2,7 @@ package connector
 
 import (
 	"context"
+	"strings"
 
 	"github.com/harness/harness-go-sdk/harness/nextgen"
 	"github.com/harness/terraform-provider-harness/helpers"
@@ -25,6 +26,11 @@ func ResourceConnectorServiceNow() *schema.Resource {
 				Description: "URL of service now.",
 				Type:        schema.TypeString,
 				Required:    true,
+				// Suppress the diff shown if the service_now_url are equal when both compared after removing / suffix if present.
+				// API implementation returns url appended with /
+				DiffSuppressFunc: func(k, old, new string, d *schema.ResourceData) bool {
+					return strings.TrimSuffix(old, "/") == strings.TrimSuffix(new, "/")
+				},
 			},
 			"username": {
 				Description: "Username to use for authentication.",