Skip to content

Commit

Permalink
- remove accidental unnecessary print
Browse files Browse the repository at this point in the history
- update logic of `RetrieveCertificate()` to accommodate eNum changes for `Request.Format`
  • Loading branch information
eyalle committed Mar 26, 2023
1 parent ddfb2d6 commit 1d7d42b
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions pkg/venafi/tpp/connector.go
Original file line number Diff line number Diff line change
Expand Up @@ -1203,11 +1203,8 @@ func (c *Connector) RetrieveCertificate(req *certificate.Request) (certificates
rootFirstOrder := includeChain && req.ChainOption == certificate.ChainOptionRootFirst

// if Request doesn't contain a Format, use defaults
if req.Format.String() == "" {
req.Format = certificate.CertFormatBase64
if req.KeyType == certificate.KeyTypeRSA {
req.Format = certificate.CertFormatBase64PKCS8
}
if req.KeyType == certificate.KeyTypeRSA {
req.Format = certificate.CertFormatBase64PKCS8
}

if req.PickupID == "" && req.Thumbprint != "" {
Expand All @@ -1232,7 +1229,6 @@ func (c *Connector) RetrieveCertificate(req *certificate.Request) (certificates
Format: req.Format.String(),
}

fmt.Println("\n\n", certReq)
if req.CsrOrigin == certificate.ServiceGeneratedCSR || req.FetchPrivateKey {
certReq.IncludePrivateKey = true
certReq.Password = req.KeyPassword
Expand Down

0 comments on commit 1d7d42b

Please sign in to comment.