From baf40e9f54edc62c3d309c35335a0d74913c7db0 Mon Sep 17 00:00:00 2001
From: Graham Steffaniak <graham.steffaniak@autodesk.com>
Date: Fri, 17 Jan 2025 18:03:27 -0600
Subject: [PATCH] made sure only one stdout logger at a time

---
 backend/logger/setup.go    | 15 ++++++++++++---
 backend/settings/config.go |  3 ++-
 2 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/backend/logger/setup.go b/backend/logger/setup.go
index 242dac45..a59ddbb0 100644
--- a/backend/logger/setup.go
+++ b/backend/logger/setup.go
@@ -20,11 +20,14 @@ type Logger struct {
 	colors      bool
 }
 
+var stdOutLoggerExists bool
+
 // NewLogger creates a new Logger instance with separate file and stdout loggers
 func NewLogger(filepath string, levels, apiLevels []LogLevel, noColors bool) (*Logger, error) {
 	var fileWriter io.Writer = io.Discard
+	stdout := filepath == ""
 	// Configure file logging
-	if filepath != "" {
+	if !stdout {
 		file, err := os.OpenFile(filepath, os.O_CREATE|os.O_WRONLY|os.O_APPEND, 0666)
 		if err != nil {
 			return nil, fmt.Errorf("failed to open log file: %v", err)
@@ -39,6 +42,9 @@ func NewLogger(filepath string, levels, apiLevels []LogLevel, noColors bool) (*L
 	if filepath != "" {
 		logger = log.New(fileWriter, "", flags)
 	}
+	if stdout {
+		stdOutLoggerExists = true
+	}
 	return &Logger{
 		logger:      logger,
 		levels:      levels,
@@ -46,7 +52,7 @@ func NewLogger(filepath string, levels, apiLevels []LogLevel, noColors bool) (*L
 		disabled:    slices.Contains(levels, DISABLED),
 		disabledAPI: slices.Contains(apiLevels, DISABLED),
 		colors:      !noColors,
-		stdout:      filepath == "",
+		stdout:      stdout,
 	}, nil
 }
 
@@ -100,7 +106,10 @@ func SetupLogger(output, levels, apiLevels string, noColors bool) error {
 	if outputStdout == "STDOUT" {
 		output = ""
 	}
-
+	if output == "" && stdOutLoggerExists {
+		// stdout logger already exists... don't create another
+		return fmt.Errorf("stdout logger already exists, could not set config levels=[%v] apiLevels=[%v] noColors=[%v]", levels, apiLevels, noColors)
+	}
 	// Create the logger
 	logger, err := NewLogger(output, upperLevels, upperApiLevels, noColors)
 	if err != nil {
diff --git a/backend/settings/config.go b/backend/settings/config.go
index 1478da46..9468f359 100644
--- a/backend/settings/config.go
+++ b/backend/settings/config.go
@@ -2,6 +2,7 @@ package settings
 
 import (
 	"fmt"
+	"log"
 	"os"
 	"path/filepath"
 	"strings"
@@ -82,7 +83,7 @@ func Initialize(configFile string) {
 			logConfig.NoColors,
 		)
 		if err != nil {
-			logger.Error(fmt.Sprintf("Failed to set up logger: %v", err))
+			log.Println("[ERROR] Failed to set up logger:", err)
 		}
 	}
 }