Merge pull request #12481 from kalexmills/fix-defer-log

fix: pass argument url in defer to avoid loopclosure
This commit is contained in:
Piotr Tabor 2021-01-31 23:20:32 +01:00 committed by GitHub
commit 958f6f9878
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -589,7 +589,7 @@ func configureClientListeners(cfg *Config) (sctxs map[string]*serveCtx, err erro
}
}
defer func() {
defer func(u url.URL) {
if err == nil {
return
}
@ -599,7 +599,7 @@ func configureClientListeners(cfg *Config) (sctxs map[string]*serveCtx, err erro
zap.String("address", u.Host),
zap.Error(err),
)
}()
}(u)
for k := range cfg.UserHandlers {
sctx.userHandlers[k] = cfg.UserHandlers[k]
}