fix(k8s): retain conjur-map key in SecretSelectedFields and tests

- Add conjur-map to SecretSelectedFields to allow mapping secrets to Conjur
- Update tests to verify conjur-map is retained and extra keys are removed
- Reference Conjur documentation for context on conjur-map usage

Fixes: https://venafi.atlassian.net/browse/VC-43052

Signed-off-by: Richard Wall <[email protected]>

Author: wallrj-cyberark

pkg/datagatherer/k8s/fieldfilter.go

@@ -9,6 +9,9 @@ import (
 // The `data` is redacted, to prevent private keys or sensitive data being
 // collected. Only the following none-sensitive keys are retained: tls.crt,
 // ca.crt. These keys are assumed to always contain public TLS certificates.
+// The `conjur-map` key is also retained, as it is used to map Secrets to
+// Conjur variables, and is not considered sensitive.
+// See https://docs.cyberark.com/conjur-open-source/latest/en/content/integrations/k8s-ocp/cjr-secrets-provider-lp.htm
 var SecretSelectedFields = []FieldPath{
 	{"kind"},
 	{"apiVersion"},
@@ -26,6 +29,7 @@ var SecretSelectedFields = []FieldPath{
 	{"type"},
 	{"data", "tls.crt"},
 	{"data", "ca.crt"},
+	{"data", "conjur-map"},
 }
 
 // RouteSelectedFields is the list of fields sent from OpenShift Route objects to the

pkg/datagatherer/k8s/fieldfilter_test.go

@@ -35,8 +35,10 @@ func TestSelect(t *testing.T) {
 			},
 			"type": "kubernetes.io/tls",
 			"data": map[string]interface{}{
-				"tls.crt": "cert data",
-				"tls.key": "secret",
+				"tls.crt":    "cert data",
+				"tls.key":    "secret",
+				"extra":      "should be removed",
+				"conjur-map": "should be kept",
 			},
 		},
 		SecretSelectedFields,
@@ -61,7 +63,8 @@ func TestSelect(t *testing.T) {
 			"type": "kubernetes.io/tls",
 			"data": map[string]interface{}{
 				// The "tls.key" is ignored.
-				"tls.crt": "cert data",
+				"tls.crt":    "cert data",
+				"conjur-map": "should be kept",
 			},
 		},
 	))