diff options
author | Dave Henderson <dhenderson@gmail.com> | 2020-09-17 23:46:24 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-09-17 21:46:24 -0600 |
commit | d16ede358a2ec049bda28bf37e79c9cfcaa64c29 (patch) | |
tree | 9304f4b06287fa2cbeafda019a70556a00e119ee /modules/caddyhttp/app.go | |
parent | c82c231ba7ec7c400e4d48ab943f57a6b29a2a2a (diff) |
metrics: Fix hidden panic while observing with bad exemplars (#3733)
* metrics: Fixing panic while observing with bad exemplars
Signed-off-by: Dave Henderson <dhenderson@gmail.com>
* Minor cleanup
The server is already added to the context. So, we can simply use that
to get the server name, which is a field on the server.
* Add integration test for auto HTTP->HTTPS redirects
A test like this would have caught the problem in the first place
Co-authored-by: Matthew Holt <mholt@users.noreply.github.com>
Diffstat (limited to 'modules/caddyhttp/app.go')
-rw-r--r-- | modules/caddyhttp/app.go | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/modules/caddyhttp/app.go b/modules/caddyhttp/app.go index 375ca4d..41820ea 100644 --- a/modules/caddyhttp/app.go +++ b/modules/caddyhttp/app.go @@ -155,6 +155,7 @@ func (app *App) Provision(ctx caddy.Context) error { // prepare each server for srvName, srv := range app.Servers { + srv.name = srvName srv.tlsApp = app.tlsApp srv.logger = app.logger.Named("log") srv.errorLogger = app.logger.Named("log.error") |