From 39247d5db7e97bd9e95b2a97028bd15db630191a Mon Sep 17 00:00:00 2001
From: Christopher LaPointe <cmlapointe11@gmail.com>
Date: Sun, 28 Jul 2024 21:48:41 -0400
Subject: [PATCH 1/2] Fix logger race. Resolves #105

---
 pkg/logger/logger.go | 29 +++++++++++++++++++++++++++--
 1 file changed, 27 insertions(+), 2 deletions(-)

diff --git a/pkg/logger/logger.go b/pkg/logger/logger.go
index e0b48318..7bdb2a2e 100644
--- a/pkg/logger/logger.go
+++ b/pkg/logger/logger.go
@@ -2,14 +2,15 @@ package logger
 
 import (
 	"bytes"
-	"fmt"
 	"log"
 	"os"
+	"sync"
 )
 
 // ErrLog is the logger that is controlled by this log controller
 var logger *log.Logger
 var logBuffer *bytes.Buffer
+var mux sync.Mutex
 
 const logPrefix = "[Log] "
 
@@ -22,6 +23,9 @@ func init() {
 
 // DeferLogs enables the log-buffer and defers any logs from printing to the screen
 func DeferLogs() {
+	mux.Lock()
+	defer mux.Unlock()
+
 	if logBuffer == nil {
 		logBuffer = new(bytes.Buffer)
 		logger = log.New(logBuffer, logPrefix, 0)
@@ -30,6 +34,9 @@ func DeferLogs() {
 
 // ImmediateLogs flushes logs and puts logging back into immediate mode
 func ImmediateLogs() {
+	mux.Lock()
+	defer mux.Unlock()
+
 	if logBuffer != nil {
 		os.Stderr.Write(logBuffer.Bytes())
 		resetLogger()
@@ -42,28 +49,46 @@ func resetLogger() {
 }
 
 func Fatalln(code int, s interface{}) {
+	mux.Lock()
+	defer mux.Unlock()
+
 	logger.Println(s)
 	OsExit(code)
 }
 
 func Fatal(code int, v ...interface{}) {
+	mux.Lock()
+	defer mux.Unlock()
+
 	logger.Print(v...)
 	OsExit(code)
 }
 
 func Fatalf(code int, s string, args ...interface{}) {
+	mux.Lock()
+	defer mux.Unlock()
+
 	logger.Printf(s, args...)
 	OsExit(code)
 }
 
 func Println(s interface{}) {
+	mux.Lock()
+	defer mux.Unlock()
+
 	logger.Println(s)
 }
 
 func Print(v ...interface{}) {
+	mux.Lock()
+	defer mux.Unlock()
+
 	logger.Print(v...)
 }
 
 func Printf(s string, args ...interface{}) {
-	Println(fmt.Sprintf(s, args...))
+	mux.Lock()
+	defer mux.Unlock()
+
+	logger.Printf(s, args...)
 }

From 8edca64016ca86000cd753bbc6b80c73577171e9 Mon Sep 17 00:00:00 2001
From: Christopher LaPointe <cmlapointe11@gmail.com>
Date: Mon, 29 Jul 2024 20:03:33 -0400
Subject: [PATCH 2/2] Switch to RWLock

---
 pkg/logger/logger.go | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/pkg/logger/logger.go b/pkg/logger/logger.go
index 7bdb2a2e..8f7ae4b9 100644
--- a/pkg/logger/logger.go
+++ b/pkg/logger/logger.go
@@ -10,7 +10,7 @@ import (
 // ErrLog is the logger that is controlled by this log controller
 var logger *log.Logger
 var logBuffer *bytes.Buffer
-var mux sync.Mutex
+var mux sync.RWMutex
 
 const logPrefix = "[Log] "
 
@@ -49,46 +49,46 @@ func resetLogger() {
 }
 
 func Fatalln(code int, s interface{}) {
-	mux.Lock()
-	defer mux.Unlock()
+	mux.RLock()
+	defer mux.RUnlock()
 
 	logger.Println(s)
 	OsExit(code)
 }
 
 func Fatal(code int, v ...interface{}) {
-	mux.Lock()
-	defer mux.Unlock()
+	mux.RLock()
+	defer mux.RUnlock()
 
 	logger.Print(v...)
 	OsExit(code)
 }
 
 func Fatalf(code int, s string, args ...interface{}) {
-	mux.Lock()
-	defer mux.Unlock()
+	mux.RLock()
+	defer mux.RUnlock()
 
 	logger.Printf(s, args...)
 	OsExit(code)
 }
 
 func Println(s interface{}) {
-	mux.Lock()
-	defer mux.Unlock()
+	mux.RLock()
+	defer mux.RUnlock()
 
 	logger.Println(s)
 }
 
 func Print(v ...interface{}) {
-	mux.Lock()
-	defer mux.Unlock()
+	mux.RLock()
+	defer mux.RUnlock()
 
 	logger.Print(v...)
 }
 
 func Printf(s string, args ...interface{}) {
-	mux.Lock()
-	defer mux.Unlock()
+	mux.RLock()
+	defer mux.RUnlock()
 
 	logger.Printf(s, args...)
 }