diff options
| author | Matt Holt <mholt@users.noreply.github.com> | 2023-08-23 20:47:54 -0600 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-08-23 20:47:54 -0600 |
| commit | b377208ededa964893dedd5660734e9616f998f7 (patch) | |
| tree | 977e8b5621c910e0f369936a77894bece48f3270 /modules/caddyhttp | |
| parent | 4776f62caa36c580d24be8e55ebc6a61ae129f51 (diff) | |
chore: Appease gosec linter (#5777)
These happen to be harmless memory aliasing
but I guess the linter can't know that and we
can't really prove it in general.
Diffstat (limited to 'modules/caddyhttp')
| -rw-r--r-- | modules/caddyhttp/reverseproxy/httptransport.go | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/modules/caddyhttp/reverseproxy/httptransport.go b/modules/caddyhttp/reverseproxy/httptransport.go index bcce67b..9290f7e 100644 --- a/modules/caddyhttp/reverseproxy/httptransport.go +++ b/modules/caddyhttp/reverseproxy/httptransport.go @@ -529,7 +529,8 @@ func (t TLSConfig) MakeTLSClientConfig(ctx caddy.Context) (*tls.Config, error) { certs := caddytls.AllMatchingCertificates(t.ClientCertificateAutomate) var err error for _, cert := range certs { - err = cri.SupportsCertificate(&cert.Certificate) + certCertificate := cert.Certificate // avoid taking address of iteration variable (gosec warning) + err = cri.SupportsCertificate(&certCertificate) if err == nil { return &cert.Certificate, nil } |