Skip to content

Commit

Permalink
cleanup: client cert decoding is not required
Browse files Browse the repository at this point in the history
Signed-off-by: Praveen M <m.praveen@ibm.com>
  • Loading branch information
iPraveenParihar authored and mergify[bot] committed Apr 15, 2024
1 parent c1dd15a commit 29f1fbe
Showing 1 changed file with 1 addition and 10 deletions.
11 changes: 1 addition & 10 deletions internal/kms/azure_vault.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ package kms

import (
"context"
"encoding/base64"
"errors"
"fmt"

Expand Down Expand Up @@ -106,19 +105,11 @@ func initAzureKeyVaultKMS(args ProviderInitArgs) (EncryptionKMS, error) {
return nil, fmt.Errorf("failed to get secrets for %T, %w", kms, err)
}

var encodedClientCertificate string
err = setConfigString(&encodedClientCertificate, secrets, azureClientCertificate)
err = setConfigString(&kms.clientCertificate, secrets, azureClientCertificate)
if err != nil {
return nil, err
}

clientCertificate, err := base64.StdEncoding.DecodeString(encodedClientCertificate)
if err != nil {
return nil, fmt.Errorf("failed to decode client certificate: %w", err)
}

kms.clientCertificate = string(clientCertificate)

return kms, nil
}

Expand Down

0 comments on commit 29f1fbe

Please sign in to comment.