Try logging just the message and not the args

Author: joel-rieke

sql/analyzer/analyzer.go

@@ -295,31 +295,15 @@ func (a *Analyzer) Log(msg string, args ...interface{}) {
 		return
 	}
 
-	// Since this is debug-only code, we'll simplify to avoid the analyzer warnings
-	// First, handle any sanitization needed for sensitive data
-	safeArgs := make([]interface{}, len(args))
-	for i, arg := range args {
-		if s, ok := arg.(string); ok && isSensitiveString(s) {
-			safeArgs[i] = "[REDACTED]"
-		} else if _, ok := arg.(plan.AuthenticationMysqlNativePassword); ok {
-			safeArgs[i] = "[PASSWORD_REDACTED]"
-		} else {
-			safeArgs[i] = arg
-		}
-	}
-
 	// Format the message with context prefix if needed
 	if len(a.contextStack) > 0 {
 		ctx := strings.Join(a.contextStack, "/")
 		// Create a string with the context prefix
-		prefixedMsg := fmt.Sprintf("%s: %s", ctx, msg)
 		// Format the message and log it directly, avoiding variadic expansion in log calls
-		formattedMsg := fmt.Sprintf(prefixedMsg, safeArgs...)
-		log.Info(formattedMsg)
+		log.Info(fmt.Sprintf("%s: %s", ctx, msg))
 	} else {
 		// Format the message and log it directly
-		formattedMsg := fmt.Sprintf(msg, safeArgs...)
-		log.Info(formattedMsg)
+		log.Info(msg)
 	}
 }