fix: makes env vars for azure tenant-id and gcp credentials file more…

… consistent w/ flags.

Signed-off-by: ianhundere <[email protected]>

cmd/certificate_maker/certificate_maker.go

@@ -119,7 +119,7 @@ func runCreate(_ *cobra.Command, _ []string) error {
 	// Handle KMS provider options
 	switch config.Type {
 	case "gcpkms":
-		if credsFile := getConfigValue(kmsCredsFile, "KMS_CREDENTIALS_FILE"); credsFile != "" {
+		if credsFile := getConfigValue(kmsCredsFile, "GCP_CREDENTIALS_FILE"); credsFile != "" {
 			// Check if credentials file exists before trying to use it
 			if _, err := os.Stat(credsFile); err != nil {
 				if os.IsNotExist(err) {
@@ -130,7 +130,7 @@ func runCreate(_ *cobra.Command, _ []string) error {
 			config.Options["credentials-file"] = credsFile
 		}
 	case "azurekms":
-		if tenantID := getConfigValue(kmsTenantID, "KMS_TENANT_ID"); tenantID != "" {
+		if tenantID := getConfigValue(kmsTenantID, "AZURE_TENANT_ID"); tenantID != "" {
 			config.Options["tenant-id"] = tenantID
 		}
 	}

cmd/certificate_maker/certificate_maker_test.go

@@ -54,6 +54,20 @@ func TestGetConfigValue(t *testing.T) {
 			envValue:  "",
 			want:      "",
 		},
+		{
+			name:      "GCP credentials file from env",
+			flagValue: "",
+			envVar:    "GCP_CREDENTIALS_FILE",
+			envValue:  "/path/to/creds.json",
+			want:      "/path/to/creds.json",
+		},
+		{
+			name:      "Azure tenant ID from env",
+			flagValue: "",
+			envVar:    "AZURE_TENANT_ID",
+			envValue:  "tenant-123",
+			want:      "tenant-123",
+		},
 	}
 
 	for _, tt := range tests {