fix: remove broken logging file and use fixed version
- Remove duplicate Error/Fatal function definitions - Replace internal/logging/logging_broken.go with internal/logging/logging.go - Ensure build uses working logging system - Resolve persistent syntax errors blocking compilation
This commit is contained in:
parent
9bdc705ddd
commit
fb2a793c05
|
|
@ -1,188 +0,0 @@
|
||||||
package logging
|
|
||||||
|
|
||||||
import (
|
|
||||||
"fmt"
|
|
||||||
"log"
|
|
||||||
"os"
|
|
||||||
"runtime"
|
|
||||||
"time"
|
|
||||||
)
|
|
||||||
|
|
||||||
var (
|
|
||||||
filePath string
|
|
||||||
file *os.File
|
|
||||||
history []string
|
|
||||||
debugEnabled bool
|
|
||||||
logger = log.New(os.Stderr, "[videotools] ", log.LstdFlags|log.Lmicroseconds)
|
|
||||||
)
|
|
||||||
|
|
||||||
const historyMax = 500
|
|
||||||
|
|
||||||
// Category represents a log category
|
|
||||||
type Category string
|
|
||||||
|
|
||||||
const (
|
|
||||||
CatUI Category = "[UI]"
|
|
||||||
CatCLI Category = "[CLI]"
|
|
||||||
CatFFMPEG Category = "[FFMPEG]"
|
|
||||||
CatSystem Category = "[SYS]"
|
|
||||||
CatModule Category = "[MODULE]"
|
|
||||||
CatPlayer Category = "[PLAYER]"
|
|
||||||
CatEnhance Category = "[ENHANCE]"
|
|
||||||
)
|
|
||||||
|
|
||||||
// Init initializes the logging system
|
|
||||||
func Init() {
|
|
||||||
// Create logs directory if it doesn't exist
|
|
||||||
logsDir := "logs"
|
|
||||||
if err := os.MkdirAll(logsDir, 0755); err != nil {
|
|
||||||
fmt.Fprintf(os.Stderr, "videotools: cannot create logs directory: %v\n", err)
|
|
||||||
return
|
|
||||||
}
|
|
||||||
|
|
||||||
// Use environment variable or default
|
|
||||||
filePath = os.Getenv("VIDEOTOOLS_LOG_FILE")
|
|
||||||
if filePath == "" {
|
|
||||||
filePath = "logs/videotools.log"
|
|
||||||
}
|
|
||||||
f, err := os.OpenFile(filePath, os.O_CREATE|os.O_APPEND|os.O_WRONLY, 0o644)
|
|
||||||
if err != nil {
|
|
||||||
fmt.Fprintf(os.Stderr, "videotools: cannot open log file %s: %v\n", filePath, err)
|
|
||||||
return
|
|
||||||
}
|
|
||||||
file = f
|
|
||||||
}
|
|
||||||
|
|
||||||
// Close closes the log file
|
|
||||||
func Close() {
|
|
||||||
if file != nil {
|
|
||||||
file.Close()
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// SetDebug enables or disables debug logging
|
|
||||||
func SetDebug(on bool) {
|
|
||||||
debugEnabled = on
|
|
||||||
Debug(CatSystem, "debug logging toggled -> %v (VIDEOTOOLS_DEBUG=%s)", on, os.Getenv("VIDEOTOOLS_DEBUG"))
|
|
||||||
}
|
|
||||||
|
|
||||||
// Debug logs a debug message with a category
|
|
||||||
func Debug(cat Category, format string, args ...interface{}) {
|
|
||||||
msg := fmt.Sprintf("%s %s", cat, fmt.Sprintf(format, args...))
|
|
||||||
timestamp := time.Now().Format(time.RFC3339Nano)
|
|
||||||
if file != nil {
|
|
||||||
fmt.Fprintf(file, "%s %s\n", timestamp, msg)
|
|
||||||
}
|
|
||||||
logger.Printf("%s %s", timestamp, msg)
|
|
||||||
}
|
|
||||||
|
|
||||||
// Crash logs a critical error with stack trace for debugging crashes
|
|
||||||
func Crash(cat Category, format string, args ...interface{}) {
|
|
||||||
msg := fmt.Sprintf("%s CRASH: %s", cat, fmt.Sprintf(format, args...))
|
|
||||||
timestamp := time.Now().Format(time.RFC3339Nano)
|
|
||||||
|
|
||||||
// Log to main log file
|
|
||||||
if file != nil {
|
|
||||||
fmt.Fprintf(file, "%s %s\n", timestamp, msg)
|
|
||||||
}
|
|
||||||
logger.Printf("%s %s", timestamp, msg)
|
|
||||||
|
|
||||||
// Also log to dedicated crash log
|
|
||||||
if crashFile, err := os.OpenFile(GetCrashLogPath(), os.O_CREATE|os.O_APPEND|os.O_WRONLY, 0644); err == nil {
|
|
||||||
fmt.Fprintf(crashFile, "%s %s\n", timestamp, msg)
|
|
||||||
fmt.Fprintf(crashFile, "Stack trace:\n%s\n", timestamp, getStackTrace())
|
|
||||||
crashFile.Sync()
|
|
||||||
}
|
|
||||||
}
|
|
||||||
history = append(history, fmt.Sprintf("%s %s", timestamp, msg))
|
|
||||||
if len(history) > historyMax {
|
|
||||||
history = history[len(history)-historyMax:]
|
|
||||||
}
|
|
||||||
if debugEnabled {
|
|
||||||
logger.Printf("%s %s", timestamp, msg)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// FilePath returns the current log file path
|
|
||||||
func FilePath() string {
|
|
||||||
return filePath
|
|
||||||
}
|
|
||||||
|
|
||||||
// History returns the log history
|
|
||||||
func History() []string {
|
|
||||||
return history
|
|
||||||
}
|
|
||||||
|
|
||||||
// getStackTrace returns current goroutine stack trace
|
|
||||||
func getStackTrace() string {
|
|
||||||
buf := make([]byte, 4096)
|
|
||||||
n := runtime.Stack(buf, false)
|
|
||||||
return string(buf[:n])
|
|
||||||
}
|
|
||||||
|
|
||||||
// Error logs an error message with a category (always logged, even when debug is off)
|
|
||||||
func Error(cat Category, format string, args ...interface{}) {
|
|
||||||
msg := fmt.Sprintf("%s ERROR: %s", cat, fmt.Sprintf(format, args...))
|
|
||||||
timestamp := time.Now().Format(time.RFC3339Nano)
|
|
||||||
if file != nil {
|
|
||||||
fmt.Fprintf(file, "%s %s\n", timestamp, msg)
|
|
||||||
}
|
|
||||||
history = append(history, fmt.Sprintf("%s %s", timestamp, msg))
|
|
||||||
if len(history) > historyMax {
|
|
||||||
history = history[len(history)-historyMax:]
|
|
||||||
}
|
|
||||||
logger.Printf("%s %s", timestamp, msg)
|
|
||||||
}
|
|
||||||
|
|
||||||
// Note: history tracking is disabled during fatal exit to avoid infinite loops
|
|
||||||
|
|
||||||
// Fatal logs a fatal error and exits (always logged, even when debug is off)
|
|
||||||
func Fatal(cat Category, format string, args ...interface{}) {
|
|
||||||
msg := fmt.Sprintf("%s FATAL: %s", cat, fmt.Sprintf(format, args...))
|
|
||||||
timestamp := time.Now().Format(time.RFC3339Nano)
|
|
||||||
if file != nil {
|
|
||||||
fmt.Fprintf(file, "%s %s\n", timestamp, msg)
|
|
||||||
file.Sync()
|
|
||||||
}
|
|
||||||
logger.Printf("%s %s", timestamp, msg)
|
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
logger.Printf("%s %s", timestamp, msg)
|
|
||||||
logger.Printf("%s %s", timestamp, msg)
|
|
||||||
os.Exit(1)
|
|
||||||
/if len(history) > historyMax {
|
|
||||||
logger.Printf("%s %s", timestamp, msg)
|
|
||||||
}
|
|
||||||
|
|
||||||
// Fatal logs a fatal error and exits (always logged)
|
|
||||||
func Fatal(cat Category, format string, args ...interface{}) {
|
|
||||||
msg := fmt.Sprintf("%s FATAL: %s", cat, fmt.Sprintf(format, args...))
|
|
||||||
timestamp := time.Now().Format(time.RFC3339Nano)
|
|
||||||
if file != nil {
|
|
||||||
fmt.Fprintf(file, "%s %s\n", timestamp, msg)
|
|
||||||
file.Sync()
|
|
||||||
}
|
|
||||||
logger.Printf("%s %s", timestamp, msg)
|
|
||||||
os.Exit(1)
|
|
||||||
}
|
|
||||||
|
|
||||||
// Panic logs a panic with stack trace
|
|
||||||
func Panic(recovered interface{}) {
|
|
||||||
msg := fmt.Sprintf("%s PANIC: %v\nStack trace:\n%s", CatSystem, recovered, string(debug.Stack()))
|
|
||||||
timestamp := time.Now().Format(time.RFC3339Nano)
|
|
||||||
if file != nil {
|
|
||||||
fmt.Fprintf(file, "%s %s\n", timestamp, msg)
|
|
||||||
file.Sync()
|
|
||||||
}
|
|
||||||
history = append(history, fmt.Sprintf("%s %s", timestamp, msg))
|
|
||||||
logger.Printf("%s %s", timestamp, msg)
|
|
||||||
}
|
|
||||||
|
|
||||||
// RecoverPanic should be used with defer to catch and log panics
|
|
||||||
func RecoverPanic() {
|
|
||||||
if r := recover(); r != nil {
|
|
||||||
Panic(r)
|
|
||||||
// Re-panic to let the program crash with the logged info
|
|
||||||
panic(r)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
Loading…
Reference in New Issue
Block a user