-
Notifications
You must be signed in to change notification settings - Fork 1
Print Log Error Swallowing #17
Copy link
Copy link
Closed
Labels
good first issueGood for newcomersGood for newcomers
Description
The base.go printLog() method does not return an error.
- Issue: If j.write(...) to the os.File fails (disk full, permissions lost), the error is silently ignored.
- Fix: While logging libraries usually don't crash the app on failure, it is good practice to have a fallback (like writing to stderr if the primary log file fails) or at least an internal error reporting mechanism.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomers