Skip to content

Commit

Permalink
Add missing slog parameters
Browse files Browse the repository at this point in the history
Signed-off-by: M Essam Hamed <[email protected]>
  • Loading branch information
mohamed-essam committed Jan 25, 2024
1 parent 08b7c6d commit dfab9ad
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions pkg/controller/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -72,14 +72,14 @@ func httpserver(cp certProvider, sc secretChecker, sr secretRotator, burst int,
mux.Handle("/v1/rotate", Instrument("/v1/rotate", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
content, err := io.ReadAll(r.Body)
if err != nil {
slog.Error("Error handling /v1/rotate request: %v", err)
slog.Error("Error handling /v1/rotate request", "error", err)
w.WriteHeader(http.StatusBadRequest)
return
}

newSecret, err := sr(content)
if err != nil {
slog.Error("Error rotating secret: %v", err)
slog.Error("Error rotating secret", "error", err)
w.WriteHeader(http.StatusInternalServerError)
return
}
Expand All @@ -92,7 +92,7 @@ func httpserver(cp certProvider, sc secretChecker, sr secretRotator, burst int,
mux.Handle("/v1/cert.pem", Instrument("/v1/cert.pem", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
certs, err := cp()
if err != nil {
slog.Error("cannot get certificates: %v", err)
slog.Error("cannot get certificates", "error", err)
http.Error(w, "cannot get certificate", http.StatusInternalServerError)
return
}
Expand All @@ -114,7 +114,7 @@ func httpserver(cp certProvider, sc secretChecker, sr secretRotator, burst int,
slog.Info("HTTP server serving", "addr", server.Addr)
go func() {
err := server.ListenAndServe()
slog.Error("HTTP server exiting: %v", err)
slog.Error("HTTP server exiting", "error", err)
}()
return &server
}
Expand All @@ -134,7 +134,7 @@ func httpserverMetrics() *http.Server {
slog.Info("HTTP metrics server serving", "addr", server.Addr)
go func() {
err := server.ListenAndServe()
slog.Error("HTTP metrics server exiting: %v", err)
slog.Error("HTTP metrics server exiting", "error", err)
}()
return &server
}
Expand Down

0 comments on commit dfab9ad

Please sign in to comment.