Improvements after testing

minio · Jan 13, 2025 · 5e86735 · 5e86735
1 parent 9f2be64
commit 5e86735
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 22 deletions.
diff --git a/internal/http/log.go b/internal/http/log.go
@@ -19,27 +19,30 @@ func (lt *LoggingTransport) RoundTrip(req *http.Request) (*http.Response, error)
 	}
 
 	start := time.Now()
-
 	resp, err := rt.RoundTrip(req)
-	switch {
-	case err != nil:
-		slog.Info("HTTP error",
-			slog.String("method", req.Method),
-			slog.String("url", req.URL.String()),
-			slog.Duration("duration", time.Since(start)),
-			slog.String("error", err.Error()))
-	case resp.StatusCode < 300:
-		slog.Info("HTTP error response",
-			slog.String("method", req.Method),
-			slog.String("url", req.URL.String()),
-			slog.Duration("duration", time.Since(start)),
-			slog.String("status", resp.Status))
-	default:
-		slog.Debug("HTTP success response",
-			slog.String("method", req.Method),
-			slog.String("url", req.URL.String()),
-			slog.Duration("duration", time.Since(start)),
-			slog.String("status", resp.Status))
+
+	// don't log health checks
+	if req.URL.Path != "/v1/sys/health" {
+		switch {
+		case err != nil:
+			slog.Info("HTTP error",
+				slog.String("method", req.Method),
+				slog.String("url", req.URL.String()),
+				slog.Duration("duration", time.Since(start)),
+				slog.String("error", err.Error()))
+		case resp.StatusCode >= 300:
+			slog.Info("HTTP error response",
+				slog.String("method", req.Method),
+				slog.String("url", req.URL.String()),
+				slog.Duration("duration", time.Since(start)),
+				slog.String("status", resp.Status))
+		default:
+			slog.Debug("HTTP success response",
+				slog.String("method", req.Method),
+				slog.String("url", req.URL.String()),
+				slog.Duration("duration", time.Since(start)),
+				slog.String("status", resp.Status))
+		}
 	}
 
 	return resp, err

diff --git a/internal/keystore/vault/config.go b/internal/keystore/vault/config.go
@@ -249,5 +249,6 @@ func (c *Config) Clone() *Config {
 		PrivateKey:      c.PrivateKey,
 		Certificate:     c.Certificate,
 		CAPath:          c.CAPath,
+		Verbose:         c.Verbose,
 	}
 }
diff --git a/internal/keystore/vault/vault.go b/internal/keystore/vault/vault.go
@@ -142,7 +142,7 @@ func Connect(ctx context.Context, c *Config) (*Store, error) {
 
 	// log authentication events
 	lastAuthSuccess := false
-	authenticate = func(ctx context.Context) (*vaultapi.Secret, error) {
+	authenticateLogged := func(ctx context.Context) (*vaultapi.Secret, error) {
 		secret, err := authenticate(ctx)
 		if err != nil {
 			if lastAuthSuccess {
@@ -164,7 +164,7 @@ func Connect(ctx context.Context, c *Config) (*Store, error) {
 		return secret, err
 	}
 
-	auth, err := authenticate(ctx)
+	auth, err := authenticateLogged(ctx)
 	if err != nil {
 		return nil, err
 	}