From d489930f6ae798013289270726ffef78e1831ec4 Mon Sep 17 00:00:00 2001
From: Tiago Silva <tiago.silva@goteleport.com>
Date: Wed, 28 Aug 2024 15:57:39 +0100
Subject: [PATCH] disable JamfSpec secrets validation (#45959)

---
 api/types/jamf.go      |  7 -------
 api/types/jamf_test.go | 32 --------------------------------
 2 files changed, 39 deletions(-)

diff --git a/api/types/jamf.go b/api/types/jamf.go
index 3bad0084603e..0d2074a1f515 100644
--- a/api/types/jamf.go
+++ b/api/types/jamf.go
@@ -44,13 +44,6 @@ func ValidateJamfSpecV1(s *JamfSpecV1) error {
 		return trace.BadParameter("spec required")
 	}
 
-	// Jamf can handle both credential sets being present, so we let it pass.
-	hasUserPass := s.Username != "" && s.Password != ""
-	hasAPICreds := s.ClientId != "" && s.ClientSecret != ""
-	if !hasUserPass && !hasAPICreds {
-		return trace.BadParameter("either username+password or clientID+clientSecret must be provided")
-	}
-
 	switch u, err := url.Parse(s.ApiEndpoint); {
 	case err != nil:
 		return trace.BadParameter("invalid API endpoint: %v", err)
diff --git a/api/types/jamf_test.go b/api/types/jamf_test.go
index d997cc3e518f..e4a230730642 100644
--- a/api/types/jamf_test.go
+++ b/api/types/jamf_test.go
@@ -125,38 +125,6 @@ func TestValidateJamfSpecV1(t *testing.T) {
 			}),
 			wantErr: "missing hostname",
 		},
-		{
-			name: "username empty",
-			spec: modify(func(spec *types.JamfSpecV1) {
-				spec.Username = ""
-			}),
-			wantErr: "username",
-		},
-		{
-			name: "password empty",
-			spec: modify(func(spec *types.JamfSpecV1) {
-				spec.Password = ""
-			}),
-			wantErr: "password",
-		},
-		{
-			name: "client_id empty",
-			spec: modify(func(spec *types.JamfSpecV1) {
-				spec.Username = ""
-				spec.Password = ""
-				spec.ClientSecret = "supersecretsecret!!1!"
-			}),
-			wantErr: "clientID",
-		},
-		{
-			name: "client_secret empty",
-			spec: modify(func(spec *types.JamfSpecV1) {
-				spec.Username = ""
-				spec.Password = ""
-				spec.ClientId = "llama-UUID"
-			}),
-			wantErr: "clientSecret",
-		},
 		{
 			name: "inventory nil entry",
 			spec: modify(func(spec *types.JamfSpecV1) {