diff --git a/internal/logging/gin_logger.go b/internal/logging/gin_logger.go index 9bfef8ad..2dfbcfc2 100644 --- a/internal/logging/gin_logger.go +++ b/internal/logging/gin_logger.go @@ -73,17 +73,15 @@ func GinLogrusLogger() gin.HandlerFunc { method := c.Request.Method errorMessage := c.Errors.ByType(gin.ErrorTypePrivate).String() + if requestID == "" { + requestID = "--------" + } logLine := fmt.Sprintf("%3d | %13v | %15s | %-7s \"%s\"", statusCode, latency, clientIP, method, path) if errorMessage != "" { logLine = logLine + " | " + errorMessage } - var entry *log.Entry - if requestID != "" { - entry = log.WithField("request_id", requestID) - } else { - entry = log.WithField("request_id", "--------") - } + entry := log.WithField("request_id", requestID) switch { case statusCode >= http.StatusInternalServerError: diff --git a/internal/logging/global_logger.go b/internal/logging/global_logger.go index f27f726f..6ea1963a 100644 --- a/internal/logging/global_logger.go +++ b/internal/logging/global_logger.go @@ -40,22 +40,22 @@ func (m *LogFormatter) Format(entry *log.Entry) ([]byte, error) { timestamp := entry.Time.Format("2006-01-02 15:04:05") message := strings.TrimRight(entry.Message, "\r\n") - reqID := "" + reqID := "--------" if id, ok := entry.Data["request_id"].(string); ok && id != "" { reqID = id } - levelStr := fmt.Sprintf("%-5s", entry.Level.String()) + level := entry.Level.String() + if level == "warning" { + level = "warn" + } + levelStr := fmt.Sprintf("%-5s", level) var formatted string - if reqID != "" && entry.Caller != nil { - formatted = fmt.Sprintf("[%s] [%s] [%s:%d] | %s | %s\n", timestamp, levelStr, filepath.Base(entry.Caller.File), entry.Caller.Line, reqID, message) - } else if reqID != "" { - formatted = fmt.Sprintf("[%s] [%s] | %s | %s\n", timestamp, levelStr, reqID, message) - } else if entry.Caller != nil { - formatted = fmt.Sprintf("[%s] [%s] [%s:%d] %s\n", timestamp, levelStr, filepath.Base(entry.Caller.File), entry.Caller.Line, message) + if entry.Caller != nil { + formatted = fmt.Sprintf("[%s] [%s] [%s] [%s:%d] %s\n", timestamp, reqID, levelStr, filepath.Base(entry.Caller.File), entry.Caller.Line, message) } else { - formatted = fmt.Sprintf("[%s] [%s] %s\n", timestamp, levelStr, message) + formatted = fmt.Sprintf("[%s] [%s] [%s] %s\n", timestamp, reqID, levelStr, message) } buffer.WriteString(formatted)