From 1ca9b48b67d1532527e29cb93e72f4015eccc4d1 Mon Sep 17 00:00:00 2001
From: Gavin Nishizawa <gavin@terrastruct.com>
Date: Tue, 5 Dec 2023 15:16:52 -0800
Subject: [PATCH] update slog usage

---
 lib/log/log.go | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/lib/log/log.go b/lib/log/log.go
index 3c0ce7113..aa29fbc8c 100644
--- a/lib/log/log.go
+++ b/lib/log/log.go
@@ -47,32 +47,35 @@ func WithTB(ctx context.Context, t testing.TB, opts *slogtest.Options) context.C
 	return With(ctx, l)
 }
 
-func Debug(ctx context.Context, msg string, fields ...slog.Field) {
+// see slog.Logger's Info; it is now possible to do the following
+// log.Debug(ctx, msg, keyString1, valueAny1, slog.Field, keyString2, valueAny2, slog.Map)
+// Warning: it will panic when using key, value as separate args if key is not a string or value arg is missing
+func Debug(ctx context.Context, msg string, fields ...any) {
 	slog.Helper()
 	from(ctx).Debug(ctx, msg, fields...)
 }
 
-func Info(ctx context.Context, msg string, fields ...slog.Field) {
+func Info(ctx context.Context, msg string, fields ...any) {
 	slog.Helper()
 	from(ctx).Info(ctx, msg, fields...)
 }
 
-func Warn(ctx context.Context, msg string, fields ...slog.Field) {
+func Warn(ctx context.Context, msg string, fields ...any) {
 	slog.Helper()
 	from(ctx).Warn(ctx, msg, fields...)
 }
 
-func Error(ctx context.Context, msg string, fields ...slog.Field) {
+func Error(ctx context.Context, msg string, fields ...any) {
 	slog.Helper()
 	from(ctx).Error(ctx, msg, fields...)
 }
 
-func Critical(ctx context.Context, msg string, fields ...slog.Field) {
+func Critical(ctx context.Context, msg string, fields ...any) {
 	slog.Helper()
 	from(ctx).Critical(ctx, msg, fields...)
 }
 
-func Fatal(ctx context.Context, msg string, fields ...slog.Field) {
+func Fatal(ctx context.Context, msg string, fields ...any) {
 	slog.Helper()
 	from(ctx).Fatal(ctx, msg, fields...)
 }