From 0c96c54416a78e35388ba944554de876588ff960 Mon Sep 17 00:00:00 2001 From: joshvanl Date: Fri, 3 Sep 2021 13:47:43 +0100 Subject: [PATCH] Adds the ACME client middleware when create an ACME account Signed-off-by: joshvanl --- pkg/acme/accounts/BUILD.bazel | 1 + pkg/acme/accounts/client.go | 5 +++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/pkg/acme/accounts/BUILD.bazel b/pkg/acme/accounts/BUILD.bazel index 4d5bfa879..d498151ae 100644 --- a/pkg/acme/accounts/BUILD.bazel +++ b/pkg/acme/accounts/BUILD.bazel @@ -10,6 +10,7 @@ go_library( visibility = ["//visibility:public"], deps = [ "//pkg/acme/client:go_default_library", + "//pkg/acme/client/middleware:go_default_library", "//pkg/acme/util:go_default_library", "//pkg/apis/acme/v1:go_default_library", "//pkg/metrics:go_default_library", diff --git a/pkg/acme/accounts/client.go b/pkg/acme/accounts/client.go index ba5c32ea1..d7bb50931 100644 --- a/pkg/acme/accounts/client.go +++ b/pkg/acme/accounts/client.go @@ -26,6 +26,7 @@ import ( acmeapi "golang.org/x/crypto/acme" acmecl "github.com/jetstack/cert-manager/pkg/acme/client" + "github.com/jetstack/cert-manager/pkg/acme/client/middleware" acmeutil "github.com/jetstack/cert-manager/pkg/acme/util" cmacme "github.com/jetstack/cert-manager/pkg/apis/acme/v1" "github.com/jetstack/cert-manager/pkg/metrics" @@ -39,13 +40,13 @@ var _ NewClientFunc = NewClient // NewClient is an implementation of NewClientFunc that returns a real ACME client. func NewClient(client *http.Client, config cmacme.ACMEIssuer, privateKey *rsa.PrivateKey) acmecl.Interface { - return &acmeapi.Client{ + return middleware.NewLogger(&acmeapi.Client{ Key: privateKey, HTTPClient: client, DirectoryURL: config.Server, UserAgent: util.CertManagerUserAgent, RetryBackoff: acmeutil.RetryBackoff, - } + }) } // BuildHTTPClient returns a instrumented HTTP client to be used by the ACME