Skip to content

Commit bb7a5cf

Browse files
committed
refactor(auth): replace log.Printf with structured slog logging
1 parent 056e199 commit bb7a5cf

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

auth.go

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import (
55
"encoding/json"
66
"errors"
77
"fmt"
8-
"log"
8+
"log/slog"
99
"net/http"
1010
"strings"
1111
"time"
@@ -82,7 +82,7 @@ func handleLogin(fetcher UserFetcher, secret []byte) http.HandlerFunc {
8282
writeJSON(w, http.StatusUnauthorized, map[string]string{"error": "invalid email or password"})
8383
return
8484
}
85-
log.Printf("login: database error: %v", err)
85+
slog.Error("login: database error", "error", err)
8686
writeJSON(w, http.StatusInternalServerError, map[string]string{"error": "internal server error"})
8787
return
8888
}
@@ -93,14 +93,14 @@ func handleLogin(fetcher UserFetcher, secret []byte) http.HandlerFunc {
9393
writeJSON(w, http.StatusUnauthorized, map[string]string{"error": "invalid email or password"})
9494
return
9595
}
96-
log.Printf("login: bcrypt error: %v", err)
96+
slog.Error("login: bcrypt error", "error", err)
9797
writeJSON(w, http.StatusInternalServerError, map[string]string{"error": "internal server error"})
9898
return
9999
}
100100

101101
tokenStr, err := generateJWT(user, secret)
102102
if err != nil {
103-
log.Printf("login: failed to generate JWT: %v", err)
103+
slog.Error("login: failed to generate JWT", "error", err)
104104
writeJSON(w, http.StatusInternalServerError, map[string]string{"error": "internal server error"})
105105
return
106106
}
@@ -148,7 +148,7 @@ func requireAuth(secret []byte, checker TokenChecker) func(http.Handler) http.Ha
148148
}, jwt.WithValidMethods([]string{"HS256"}), jwt.WithIssuer("vehicle-positions-api"))
149149

150150
if err != nil || !token.Valid {
151-
log.Printf("auth: token validation failed: %v", err)
151+
slog.Warn("auth: token validation failed", "error", err)
152152
writeJSON(w, http.StatusUnauthorized, map[string]string{"error": "invalid token"})
153153
return
154154
}
@@ -162,7 +162,7 @@ func requireAuth(secret []byte, checker TokenChecker) func(http.Handler) http.Ha
162162
if jti, ok := claims["jti"].(string); ok && jti != "" {
163163
revoked, err := checker.IsTokenRevoked(r.Context(), jti)
164164
if err != nil {
165-
log.Printf("auth: revocation check failed: %v", err)
165+
slog.Error("auth: revocation check failed", "error", err)
166166
writeJSON(w, http.StatusInternalServerError, map[string]string{"error": "internal server error"})
167167
return
168168
}
@@ -200,7 +200,7 @@ func handleLogout(revoker TokenRevoker) http.HandlerFunc {
200200
expiresAt := time.Unix(int64(expFloat), 0)
201201

202202
if err := revoker.RevokeToken(r.Context(), jti, expiresAt); err != nil {
203-
log.Printf("logout: failed to revoke token: %v", err)
203+
slog.Error("logout: failed to revoke token", "error", err)
204204
writeJSON(w, http.StatusInternalServerError, map[string]string{"error": "internal server error"})
205205
return
206206
}

0 commit comments

Comments
 (0)