Skip to content

Commit bb4ef17

Browse files
committed
Fix typos
1 parent b1ebcd7 commit bb4ef17

File tree

4 files changed

+6
-2
lines changed

4 files changed

+6
-2
lines changed

apis/concierge/config/v1alpha1/types_credentialissuer.go.tmpl

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// Copyright 2020-2023 the Pinniped contributors. All Rights Reserved.
1+
// Copyright 2020-2024 the Pinniped contributors. All Rights Reserved.
22
// SPDX-License-Identifier: Apache-2.0
33

44
package v1alpha1
@@ -88,6 +88,7 @@ const (
8888
//
8989
// If neither CertificateAuthorityData nor ca.crt is provided, no CA bundle will be advertised for
9090
// the impersonation proxy endpoint.
91+
// TODO: Improve the docs on the fields below to reference ca.crt vs CertificateAuthorityData
9192
type ImpersonationProxyTLSSpec struct {
9293
// X.509 Certificate Authority (base64-encoded PEM bundle).
9394
// Used to advertise the CA bundle for the impersonation proxy endpoint.

internal/controller/apicerts/certs_creator.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ func NewCertsCreatorController(
6161
) controllerlib.Controller {
6262
return controllerlib.New(
6363
controllerlib.Config{
64-
Name: "certs-manager-controller",
64+
Name: "certs-creator-controller",
6565
Syncer: &certsCreatorController{
6666
namespace: namespace,
6767
certsSecretResourceName: certsSecretResourceName,

internal/controller/apicerts/certs_observer_test.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -242,6 +242,8 @@ func TestObserverControllerSync(t *testing.T) {
242242

243243
actualCertChain, actualKey = dynamicCertProvider.CurrentCertKeyContent()
244244
r.True(strings.HasPrefix(string(actualCertChain), `-----BEGIN CERTIFICATE-----`), "not a cert:\n%s", string(actualCertChain))
245+
// Confirm that the embed worked successfully
246+
r.True(len(privateKeyPrefix) > 0, "privateKeyPrefix should be non-empty")
245247
r.True(strings.HasPrefix(string(actualKey), privateKeyPrefix), "not a key:\n%s", string(actualKey))
246248
})
247249
})
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
-----BEGIN PRIVATE KEY-----

0 commit comments

Comments
 (0)