otellogr: Fix nil context panic

CHANGELOG.md

@@ -14,6 +14,7 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 
 ### Fixed
 
+- Use `context.Background()` as default context instead of nil in `go.opentelemetry.io/contrib/bridges/otellogr`. (#6527)
 - Fix error logged by Jaeger remote sampler on empty or unset `OTEL_TRACES_SAMPLER_ARG` environment variable (#6511)
 
 ## [1.33.0/0.58.0/0.27.0/0.13.0/0.8.0/0.6.0/0.5.0] - 2024-12-12

bridges/otellogr/logsink.go

@@ -65,10 +65,9 @@ import (
 )
 
 type config struct {
-	provider  log.LoggerProvider
-	version   string
-	schemaURL string
-
+	provider      log.LoggerProvider
+	version       string
+	schemaURL     string
 	levelSeverity func(int) log.Severity
 }
 
@@ -176,6 +175,7 @@ func NewLogSink(name string, options ...Option) *LogSink {
 		logger:        c.provider.Logger(name, opts...),
 		levelSeverity: c.levelSeverity,
 		opts:          opts,
+		ctx:           context.Background(),
 	}
 }
 
@@ -201,9 +201,8 @@ var _ logr.LogSink = (*LogSink)(nil)
 // For example, commandline flags might be used to set the logging
 // verbosity and disable some info logs.
 func (l *LogSink) Enabled(level int) bool {
-	ctx := context.Background()
 	param := log.EnabledParameters{Severity: l.levelSeverity(level)}
-	return l.logger.Enabled(ctx, param)
+	return l.logger.Enabled(context.Background(), param)
 }
 
 // Error logs an error, with the given message and key/value pairs.
@@ -240,7 +239,7 @@ func (l *LogSink) Info(level int, msg string, keysAndValues ...any) {
 
 // Init receives optional information about the logr library this
 // implementation does not use it.
-func (l *LogSink) Init(info logr.RuntimeInfo) {
+func (l *LogSink) Init(logr.RuntimeInfo) {
 	// We don't need to do anything here.
 	// CallDepth is used to calculate the caller's PC.
 	// PC is dropped as part of the conversion to the OpenTelemetry log.Record.

bridges/otellogr/logsink_test.go

@@ -30,44 +30,37 @@ func TestNewConfig(t *testing.T) {
 	customLoggerProvider := mockLoggerProvider{}
 
 	for _, tt := range []struct {
-		name    string
-		options []Option
-
-		wantConfig config
+		name     string
+		options  []Option
+		wantFunc func(config)
 	}{
 		{
 			name: "with no options",
-
-			wantConfig: config{
-				provider: global.GetLoggerProvider(),
+			wantFunc: func(c config) {
+				assert.Equal(t, global.GetLoggerProvider(), c.provider)
 			},
 		},
 		{
 			name: "with a custom instrumentation scope",
 			options: []Option{
 				WithVersion("42.0"),
 			},
-
-			wantConfig: config{
-				version:  "42.0",
-				provider: global.GetLoggerProvider(),
+			wantFunc: func(c config) {
+				assert.Equal(t, "42.0", c.version)
 			},
 		},
 		{
 			name: "with a custom logger provider",
 			options: []Option{
 				WithLoggerProvider(customLoggerProvider),
 			},
-
-			wantConfig: config{
-				provider: customLoggerProvider,
+			wantFunc: func(c config) {
+				assert.Equal(t, customLoggerProvider, c.provider)
 			},
 		},
 	} {
 		t.Run(tt.name, func(t *testing.T) {
-			config := newConfig(tt.options)
-			config.levelSeverity = nil // Ignore asserting level severity function, assert.Equal does not support function comparison
-			assert.Equal(t, tt.wantConfig, config)
+			tt.wantFunc(newConfig(tt.options))
 		})
 	}
 }