weiguoqiang/cert-manager
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

fix incorrect comments and error messages

Browse Source 
Signed-off-by: Tim Ramlot <42113979+inteon@users.noreply.github.com>
This commit is contained in:
Tim Ramlot 2024-02-16 19:49:28 +01:00
parent 205067b834
commit 0ed660873e
No known key found for this signature in database
GPG Key ID: 47428728E0C2878D
2 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
pkg/apis/acme/v1/types_issuer.go
View File

@ -48,8 +48,9 @@ type ACMEIssuer struct {
// endpoint.
// For example, for Let's Encrypt's DST crosssign you would use:
// "DST Root CA X3" or "ISRG Root X1" for the newer Let's Encrypt root CA.
// This value picks the first certificate bundle in the ACME alternative
// chains that has a certificate with this value as its issuer's CN
// This value picks the first certificate bundle in the combined set of
// ACME default and alternative chains that has a root-most certificate with
// this value as its issuer's commonname.
// +optional
// +kubebuilder:validation:MaxLength=64
PreferredChain string `json:"preferredChain,omitempty"`

2
pkg/controller/acmeorders/sync.go
View File

@ -588,7 +588,7 @@ func (c *controller) finalizeOrder(ctx context.Context, cl acmecl.Interface, o *
preferredChainName := issuer.GetSpec().ACME.PreferredChain
found, preferredCertChain, err := getPreferredCertChain(ctx, cl, certURL, certSlice, preferredChainName)
if err != nil {
return fmt.Errorf("error retrieving alternate chain: %w", err)
return fmt.Errorf("error retrieving preferred chain: %w", err)
}
if found {
return c.storeCertificateOnStatus(ctx, o, preferredCertChain)