From 1548ac32b7d95f556ae7c8606704f1edaa215e5a Mon Sep 17 00:00:00 2001
From: Jesse Schmidt <jschmidt@aerospike.com>
Date: Wed, 3 Jul 2024 09:08:43 -0700
Subject: [PATCH] review changes

---
 cmd/flags/tls.go |  4 +--
 cmd/utils.go     | 94 ------------------------------------------------
 2 files changed, 2 insertions(+), 96 deletions(-)

diff --git a/cmd/flags/tls.go b/cmd/flags/tls.go
index 30927c4..837f2c7 100644
--- a/cmd/flags/tls.go
+++ b/cmd/flags/tls.go
@@ -23,8 +23,8 @@ func NewTLSFlags() *TLSFlags {
 	}
 }
 
-// NewAerospikeFlagSet returns a new pflag.FlagSet with Aerospike flags defined.
-// Values set in the returned FlagSet will be stored in the AerospikeFlags argument.
+// NewTLSFlagSet returns a new pflag.FlagSet with TLS flags defined. Values
+// are stored in the TLSFlags struct.
 func (tf *TLSFlags) NewTLSFlagSet(fmtUsage commonFlags.UsageFormatter) *pflag.FlagSet {
 	f := &pflag.FlagSet{}
 
diff --git a/cmd/utils.go b/cmd/utils.go
index 46ce4d1..71a062b 100644
--- a/cmd/utils.go
+++ b/cmd/utils.go
@@ -3,9 +3,6 @@ package cmd
 import (
 	"asvec/cmd/flags"
 	"context"
-	"crypto/tls"
-	"crypto/x509"
-	"encoding/pem"
 	"fmt"
 	"log/slog"
 	"os"
@@ -57,97 +54,6 @@ func createClientFromFlags(clientFlags *flags.ClientFlags, connectTimeout time.D
 
 	return adminClient, nil
 }
-
-func newTLSConfig(rootCA [][]byte, cert []byte, key []byte, keyPass []byte, tlsProtoMin int, tlsProtoMax int) (*tls.Config, error) {
-	if len(rootCA) == 0 && len(cert) == 0 && len(key) == 0 {
-		return nil, nil
-	}
-
-	var (
-		clientPool []tls.Certificate
-		serverPool *x509.CertPool
-		err        error
-	)
-
-	serverPool = loadCACerts(rootCA)
-
-	if len(cert) > 0 || len(key) > 0 {
-		clientPool, err = loadServerCertAndKey(cert, key, keyPass)
-		if err != nil {
-			return nil, fmt.Errorf("failed to load client authentication certificate and key `%s`", err)
-		}
-	}
-
-	tlsConfig := &tls.Config{ //nolint:gosec // aerospike default tls version is TLSv1.2
-		Certificates:             clientPool,
-		RootCAs:                  serverPool,
-		InsecureSkipVerify:       false,
-		PreferServerCipherSuites: true,
-		MinVersion:               uint16(tlsProtoMin),
-		MaxVersion:               uint16(tlsProtoMax),
-	}
-
-	return tlsConfig, nil
-}
-
-// loadCACerts returns CA set of certificates (cert pool)
-// reads CA certificate based on the certConfig and adds it to the pool
-func loadCACerts(certsBytes [][]byte) *x509.CertPool {
-	certificates, err := x509.SystemCertPool()
-	if certificates == nil || err != nil {
-		certificates = x509.NewCertPool()
-	}
-
-	for _, cert := range certsBytes {
-		if len(cert) > 0 {
-			certificates.AppendCertsFromPEM(cert)
-		}
-	}
-
-	return certificates
-}
-
-// loadServerCertAndKey reads server certificate and associated key file based on certConfig and keyConfig
-// returns parsed server certificate
-// if the private key is encrypted, it will be decrypted using key file passphrase
-func loadServerCertAndKey(certFileBytes, keyFileBytes, keyPassBytes []byte) ([]tls.Certificate, error) {
-	var certificates []tls.Certificate
-
-	// Decode PEM data
-	keyBlock, _ := pem.Decode(keyFileBytes)
-
-	if keyBlock == nil {
-		return nil, fmt.Errorf("failed to decode PEM data for key or certificate")
-	}
-
-	// Check and Decrypt the Key Block using passphrase
-	if x509.IsEncryptedPEMBlock(keyBlock) { //nolint:staticcheck,lll // This needs to be addressed by aerospike as multiple projects require this functionality
-		decryptedDERBytes, err := x509.DecryptPEMBlock(keyBlock, keyPassBytes) //nolint:staticcheck,lll // This needs to be addressed by aerospike as multiple projects require this functionality
-		if err != nil {
-			return nil, fmt.Errorf("failed to decrypt PEM Block: `%s`", err)
-		}
-
-		keyBlock.Bytes = decryptedDERBytes
-		keyBlock.Headers = nil
-	}
-
-	// Encode PEM data
-	keyPEM := pem.EncodeToMemory(keyBlock)
-
-	if keyPEM == nil {
-		return nil, fmt.Errorf("failed to encode PEM data for key or certificate")
-	}
-
-	cert, err := tls.X509KeyPair(certFileBytes, keyPEM)
-	if err != nil {
-		return nil, fmt.Errorf("failed to add certificate and key to the pool: `%s`", err)
-	}
-
-	certificates = append(certificates, cert)
-
-	return certificates, nil
-}
-
 func parseBothHostSeedsFlag(seeds *flags.SeedsSliceFlag, host *flags.HostPortFlag) (avs.HostPortSlice, bool) {
 	isLoadBalancer := false
 	hosts := avs.HostPortSlice{}