Using global logger fails to bind values causing incorrect context values about kratos HOT 3 OPEN

package main

import (
	"context"

	"github.com/go-kratos/kratos/v2/log"
)

func main() {
	ctx := context.Background()
	ctx = context.WithValue(ctx, traceIdKey{}, "123456")


	logger := log.DefaultLogger
	newLogger := log.With(logger, "traceId", TraceIdValue())

	log.SetLogger(newLogger) // setting logger with valuer as global

	helper := log.NewHelper(newLogger)

	helper.WithContext(ctx).Info("This prints the traceID correctly")


	globalLogger := log.NewHelper(log.GetLogger()) // getting the global logger

	globalLogger.WithContext(ctx).Info("This does not print the traceID")

	// INFO traceId=123456 msg=This prints the traceID correctly
	// INFO traceId=<nil> msg=This does not print the traceID
}

type traceIdKey struct{}

func TraceIdValue() log.Valuer {
	return func(ctx context.Context) any {
		return ctx.Value(traceIdKey{})
	}
}

package main

import (
	"context"

	"github.com/go-kratos/kratos/v2/log"
)

func main() {
	ctx := context.Background()
	ctx = context.WithValue(ctx, traceIdKey{}, "123456")

	global := log.GetLogger()
	logger := log.WithContext(ctx, global)
	// logger = log.With(logger, "traceId", TraceIdValue()) // 看后面的说明
	helper := log.NewHelper(logger)

	helper.Infow("msg", "用户登录", "user", "张三")

	// 期望输出
	// INFO traceId=123456 msg=用户登录 user=张三
	// 实际输出
	// INFO msg=用户登录 user=张三
}

type traceIdKey struct{}

// 看后面的说明
func TraceIdValue() log.Valuer {
	return func(ctx context.Context) any {
		return ctx.Value(traceIdKey{})
	}
}

package main

import (
	"context"
	"os"

	"github.com/go-kratos/kratos/v2/log"
)

func main() {
	l1 := log.NewStdLogger(os.Stderr)            // *log.stdLogger
	l2 := log.With(l1, "traceId", traceIdValuer) // *log.logger ctx=Background

	log.SetLogger(l2)     // log.global.Logger = l2
	l3 := log.GetLogger() // log.global not l2 *log.loggerAppliance

	ctx := context.TODO()
	ctx = context.WithValue(ctx, traceIdKey{}, "123456")

	// log.WithContext(ctx,l) only "merge" ctx when l is a *log.logger.
	// and l3 is not.
	l4 := log.WithContext(ctx, l3) // *log.logger (not l2) ctx=ctx
	l4.Log(log.LevelInfo, "msg", "m")

	// want:
	// INFO traceId=123456 msg=m
	// got:
	// INFO traceId=<nil> msg=m

	// solution: Let log.global be a place that only stores values, rather than a Logger.
}

type traceIdKey struct{}

var traceIdValuer log.Valuer = func(ctx context.Context) any {
	return ctx.Value(traceIdKey{})
}