fix proxy config override and removes comments

Venafi · Sep 23, 2024 · d64ceba · d64ceba
1 parent 2869d57
commit d64ceba
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/pkg/playbook/app/vcertutil/vcertutil.go b/pkg/playbook/app/vcertutil/vcertutil.go
@@ -122,12 +122,12 @@ func buildClient(config domain.Config, zone string, timeout int) (endpoint.Conne
 		if !connectionTrustBundle.AppendCertsFromPEM([]byte(vcertConfig.ConnectionTrust)) {
 			return nil, fmt.Errorf("%w: failed to parse PEM trust bundle", verror.UserDataError)
 		}
-		vcertConfig.Client.Transport = &http.Transport{
-			TLSClientConfig: &tls.Config{
-				RootCAs:    connectionTrustBundle,
-				MinVersion: tls.VersionTLS12,
-			},
+		netTransport.TLSClientConfig = &tls.Config{
+			RootCAs:    connectionTrustBundle,
+			MinVersion: tls.VersionTLS12,
 		}
+
+		vcertConfig.Client.Transport = netTransport
 	}
 
 	// build Authentication object

diff --git a/pkg/venafi/cloud/connector.go b/pkg/venafi/cloud/connector.go
@@ -1267,8 +1267,6 @@ type managedCertificate struct {
 }
 
 func (c *Connector) getCertificate(certificateId string) (*managedCertificate, error) {
-	// Flow renew certificate
-	//var err error
 	url := c.getURL(urlResourceCertificateByID)
 	url = fmt.Sprintf(url, certificateId)