CBG-3713 use atomic pointers for loggers

base/logger.go

@@ -13,6 +13,7 @@ package base
 import (
 	"log"
 	"math"
+	"slices"
 	"strings"
 	"sync"
 	"time"
@@ -21,27 +22,34 @@ import (
 var flushLogBuffersWaitGroup sync.WaitGroup
 var flushLogMutex sync.Mutex
 
+// getFileLoggers returns a slice of all non-nil file loggers.
+func getFileLoggers() []*FileLogger {
+	loggers := []*FileLogger{
+		traceLogger.Load(),
+		debugLogger.Load(),
+		infoLogger.Load(),
+		warnLogger.Load(),
+		errorLogger.Load(),
+		statsLogger.Load(),
+	}
+	rawAuditLogger := auditLogger.Load()
+	if rawAuditLogger != nil {
+		loggers = append(loggers, &rawAuditLogger.FileLogger)
+	}
+	rawConsoleLogger := consoleLogger.Load()
+	if rawConsoleLogger != nil {
+		loggers = append(loggers, &rawConsoleLogger.FileLogger)
+	}
+	return slices.DeleteFunc(loggers, func(l *FileLogger) bool { return l == nil })
+}
+
 // FlushLogBuffers will cause all log collation buffers to be flushed to the output before returning.
 func FlushLogBuffers() {
 	flushLogMutex.Lock()
 	defer flushLogMutex.Unlock()
 
-	loggers := []*FileLogger{
-		traceLogger,
-		debugLogger,
-		infoLogger,
-		warnLogger,
-		errorLogger,
-		statsLogger,
-		&consoleLogger.FileLogger,
-	}
-
-	if auditLogger != nil {
-		loggers = append(loggers, &auditLogger.FileLogger)
-	}
-
-	for _, logger := range loggers {
-		if logger != nil && cap(logger.collateBuffer) > 1 {
+	for _, logger := range getFileLoggers() {
+		if cap(logger.collateBuffer) > 1 {
 			logger.collateBufferWg.Wait()
 			flushLogBuffersWaitGroup.Add(1)
 			logger.flushChan <- struct{}{}

base/logger_audit.go

@@ -117,32 +117,33 @@ func (f *AuditFields) merge(ctx context.Context, overwrites AuditFields) {
 func Audit(ctx context.Context, id AuditID, additionalData AuditFields) {
 	var fields AuditFields
 
+	logger := auditLogger.Load()
 	if IsDevMode() {
 		// NOTE: This check is expensive and indicates a dev-time mistake that needs addressing.
 		// Don't bother in production code, but also delay expandFields until we know we will log.
-		fields = expandFields(id, ctx, auditLogger.globalFields, additionalData)
+		fields = expandFields(id, ctx, logger.globalFields, additionalData)
 		id.MustValidateFields(fields)
 	}
 
-	if !auditLogger.shouldLog(id, ctx) {
+	if !logger.shouldLog(id, ctx) {
 		return
 	}
 
 	// delayed expansion until after enabled checks in non-dev mode
 	if fields == nil {
-		fields = expandFields(id, ctx, auditLogger.globalFields, additionalData)
+		fields = expandFields(id, ctx, logger.globalFields, additionalData)
 	}
 	fieldsJSON, err := JSONMarshalCanonical(fields)
 	if err != nil {
 		AssertfCtx(ctx, "failed to marshal audit fields: %v", err)
 		return
 	}
-	auditLogger.logf(string(fieldsJSON))
+	logger.logf(string(fieldsJSON))
 }
 
 // IsAuditEnabled checks if auditing is enabled for the SG node
 func IsAuditEnabled() bool {
-	return auditLogger.FileLogger.shouldLog(LevelNone)
+	return auditLogger.Load().FileLogger.shouldLog(LevelNone)
 }
 
 // AuditLogger is a file logger with audit-specific behaviour.
@@ -221,7 +222,7 @@ func NewAuditLogger(ctx context.Context, config *AuditLoggerConfig, logFilePath
 }
 
 func (al *AuditLogger) shouldLog(id AuditID, ctx context.Context) bool {
-	if !auditLogger.FileLogger.shouldLog(LevelNone) {
+	if !al.FileLogger.shouldLog(LevelNone) {
 		return false
 	}
 

base/logger_audit_test.go

@@ -13,6 +13,7 @@ import (
 	"maps"
 	"testing"
 
+	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 )
 
@@ -133,9 +134,11 @@ func TestAuditLoggerGlobalFields(t *testing.T) {
 			if testCase.contextFields != nil {
 				ctx = AuditLogCtx(ctx, testCase.contextFields)
 			}
-			var err error
-			auditLogger, err = NewAuditLogger(ctx, &AuditLoggerConfig{FileLoggerConfig: FileLoggerConfig{Enabled: BoolPtr(true)}}, tmpdir, 0, nil, testCase.globalFields)
+			logger, err := NewAuditLogger(ctx, &AuditLoggerConfig{FileLoggerConfig: FileLoggerConfig{Enabled: BoolPtr(true)}}, tmpdir, 0, nil, testCase.globalFields)
 			require.NoError(t, err)
+			defer assert.NoError(t, logger.Close())
+			auditLogger.Store(logger)
+
 			startWarnCount := SyncGatewayStats.GlobalStats.ResourceUtilizationStats().WarnCount.Value()
 			output := AuditLogContents(t, func(tb testing.TB) {
 				// Test basic audit event

base/logger_external.go

@@ -47,7 +47,8 @@ func initExternalLoggers() {
 
 func updateExternalLoggers() {
 	// use context.Background() since this is called from init or to reset test logging
-	if consoleLogger != nil && consoleLogger.shouldLog(context.Background(), LevelDebug, KeyWalrus) {
+	logger := consoleLogger.Load()
+	if logger.shouldLog(context.Background(), LevelDebug, KeyWalrus) {
 		rosmar.SetLogLevel(rosmar.LevelDebug)
 	} else {
 		rosmar.SetLogLevel(rosmar.LevelInfo)
@@ -170,10 +171,11 @@ func (CBGoUtilsLogger) SetLevel(l logging.Level) {
 // CBGoUtilsLogger.Level returns a compatible go-couchbase/golog Log Level for
 // the given logging config LogLevel
 func (CBGoUtilsLogger) Level() logging.Level {
-	if consoleLogger == nil || consoleLogger.LogLevel == nil {
+	logger := consoleLogger.Load()
+	if logger == nil || logger.LogLevel == nil {
 		return logging.INFO
 	}
-	switch *consoleLogger.LogLevel {
+	switch *logger.LogLevel {
 	case LevelTrace:
 		return logging.TRACE
 	case LevelDebug:

base/logger_file.go

@@ -183,6 +183,13 @@ func (l *FileLogger) logf(format string, args ...interface{}) {
 	}
 }
 
+// conditionalPrintf will log the message if the log level is enabled.
+func (l *FileLogger) conditionalPrintf(logLevel LogLevel, format string, args ...interface{}) {
+	if l.shouldLog(logLevel) {
+		l.logf(format, args...)
+	}
+}
+
 // shouldLog returns true if we can log.
 func (l *FileLogger) shouldLog(logLevel LogLevel) bool {
 	return l != nil && l.logger != nil &&