chore: Run go fix

Author: Neurostep

internal/pkg/configuration/builder/viper.go

@@ -13,7 +13,7 @@ import (
 // ViperBuilder is a builder to streamline Viper configuration and building.
 type ViperBuilder struct {
 	vConf    *viper.Viper
-	defaults map[string]interface{}
+	defaults map[string]any
 	name     string
 }
 
@@ -32,7 +32,7 @@ func New(name string) *ViperBuilder {
 	return &ViperBuilder{
 		vConf:    vConf,
 		name:     name,
-		defaults: make(map[string]interface{}),
+		defaults: make(map[string]any),
 	}
 }
 
@@ -45,7 +45,7 @@ func (vb *ViperBuilder) ConfigPath(path string) *ViperBuilder {
 // SetDefault sets a default value for a configuration key.
 // Any default value set will be available in the `viper.Viper` configuration
 // instance that is returned after calling the `Build()` function.
-func (vb *ViperBuilder) SetDefault(key string, value interface{}) *ViperBuilder {
+func (vb *ViperBuilder) SetDefault(key string, value any) *ViperBuilder {
 	vb.defaults[key] = value
 	return vb
 }

pkg/app/config.go

@@ -58,7 +58,7 @@ func (c *Config) String(key string) string {
 }
 
 // StringMap returns a key's value as map[string]interface{}.
-func (c *Config) StringMap(key string) map[string]interface{} {
+func (c *Config) StringMap(key string) map[string]any {
 	return c.vConf.GetStringMap(key)
 }
 
@@ -83,7 +83,7 @@ func (c *Config) Duration(key string) time.Duration {
 }
 
 // Set sets a value to a key.
-func (c *Config) Set(key string, value interface{}) {
+func (c *Config) Set(key string, value any) {
 	c.vConf.Set(key, value)
 }
 
@@ -93,6 +93,6 @@ func (c *Config) IsSet(key string) bool {
 }
 
 // AllSettings returns all settings as map.
-func (c *Config) AllSettings() map[string]interface{} {
+func (c *Config) AllSettings() map[string]any {
 	return c.vConf.AllSettings()
 }

pkg/aws/aws_test.go

@@ -282,7 +282,6 @@ func TestBuilder(t *testing.T) {
 	}
 
 	for _, tc := range tests {
-		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
 			tc.fn(t)
 		})

pkg/context/logger/context.go

@@ -3,6 +3,7 @@ package logger
 import (
 	"context"
 	"fmt"
+	"maps"
 
 	grpcctxtags "github.com/grpc-ecosystem/go-grpc-middleware/tags"
 
@@ -26,9 +27,7 @@ func AddFields(ctx context.Context, fields sdklogger.Fields) {
 	if !ok || l == nil {
 		return
 	}
-	for k, v := range fields {
-		l.fields[k] = v
-	}
+	maps.Copy(l.fields, fields)
 }
 
 // Extract takes the call-scoped sdklogger.Logger from the context.
@@ -43,14 +42,10 @@ func Extract(ctx context.Context) (sdklogger.Logger, error) {
 
 	// Add grpcctxtags tags metadata until now.
 	tags := grpcctxtags.Extract(ctx)
-	for k, v := range tags.Values() {
-		fields[k] = v
-	}
+	maps.Copy(fields, tags.Values())
 
 	// Add sdklogger fields added until now.
-	for k, v := range l.fields {
-		fields[k] = v
-	}
+	maps.Copy(fields, l.fields)
 
 	return l.logger.WithFields(fields), nil
 }

pkg/instrumentation/kafka/kafka_test.go

@@ -130,7 +130,7 @@ func TestNewClient(t *testing.T) {
 	assert.Len(t, spans, 7)
 
 	// produce
-	for i := 0; i < 4; i++ {
+	for i := range 4 {
 		s := spans[i]
 		assert.Equal(t, "kafka.produce", s.OperationName())
 		assert.Equal(t, "kafka", s.Tag(ext.ServiceName))

pkg/interceptors/database.go

@@ -14,10 +14,10 @@ import (
 func DatabaseUnaryServerInterceptor(db *gorm.DB) grpc.UnaryServerInterceptor {
 	return func(
 		ctx context.Context,
-		req interface{},
+		req any,
 		info *grpc.UnaryServerInfo,
 		handler grpc.UnaryHandler,
-	) (interface{}, error) {
+	) (any, error) {
 		instrumentedDB := db.WithContext(ctx)
 		newCtx := sdkcontext.ToContext(ctx, instrumentedDB)
 
@@ -28,7 +28,7 @@ func DatabaseUnaryServerInterceptor(db *gorm.DB) grpc.UnaryServerInterceptor {
 // DatabaseStreamServerInterceptor returns a streaming server interceptor that adds gorm.DB to the context.
 func DatabaseStreamServerInterceptor(db *gorm.DB) grpc.StreamServerInterceptor {
 	return func(
-		srv interface{},
+		srv any,
 		stream grpc.ServerStream,
 		info *grpc.StreamServerInfo,
 		handler grpc.StreamHandler,

pkg/interceptors/database_logging.go

@@ -19,10 +19,10 @@ import (
 func DatabaseLoggingUnaryServerInterceptor() grpc.UnaryServerInterceptor {
 	return func(
 		ctx context.Context,
-		req interface{},
+		req any,
 		_ *grpc.UnaryServerInfo,
 		handler grpc.UnaryHandler,
-	) (interface{}, error) {
+	) (any, error) {
 		db, err := sdkdatabasecontext.Extract(ctx)
 		if err != nil {
 			return nil, err
@@ -50,7 +50,7 @@ func DatabaseLoggingUnaryServerInterceptor() grpc.UnaryServerInterceptor {
 // meta-information using the logger.
 func DatabaseLoggingStreamServerInterceptor() grpc.StreamServerInterceptor {
 	return func(
-		srv interface{},
+		srv any,
 		stream grpc.ServerStream,
 		_ *grpc.StreamServerInfo,
 		handler grpc.StreamHandler,

pkg/interceptors/database_logging_test.go

@@ -93,7 +93,7 @@ func TestDatabaseLoggingUnaryServerInterceptor(t *testing.T) {
 	assert.Nil(t, err)
 
 	// read first log entry
-	var fieldsUnary map[string]interface{}
+	var fieldsUnary map[string]any
 	dec := json.NewDecoder(bytes.NewReader(buffer.Bytes()))
 	err = dec.Decode(&fieldsUnary)
 	require.Nil(t, err)
@@ -177,18 +177,18 @@ func TestDatabaseLoggingStreamServerInterceptors(t *testing.T) {
 	assert.Nil(t, err)
 
 	// read first log entry
-	var fieldsStream map[string]interface{}
+	var fieldsStream map[string]any
 	dec := json.NewDecoder(bytes.NewReader(buffer.Bytes()))
 	err = dec.Decode(&fieldsStream)
 	require.Nil(t, err)
 
 	checkGormLoggerFields(t, fieldsStream)
 }
 
-func checkGormLoggerFields(t *testing.T, fields map[string]interface{}) {
+func checkGormLoggerFields(t *testing.T, fields map[string]any) {
 	assert.NotEmpty(t, fields["sql"])
 
-	dbFields, ok := (fields["sql"]).(map[string]interface{})
+	dbFields, ok := (fields["sql"]).(map[string]any)
 	assert.True(t, ok, "%s not found in log fields", "trace")
 	assert.NotEmpty(t, dbFields)
 

pkg/interceptors/logger.go

@@ -2,7 +2,6 @@ package interceptors
 
 import (
 	"context"
-	"fmt"
 	"path"
 	"time"
 
@@ -21,10 +20,10 @@ import (
 func LoggerUnaryServerInterceptor(logger sdklogger.Logger) grpc.UnaryServerInterceptor {
 	return func(
 		ctx context.Context,
-		req interface{},
+		req any,
 		info *grpc.UnaryServerInfo,
 		handler grpc.UnaryHandler,
-	) (interface{}, error) {
+	) (any, error) {
 		startTime := time.Now()
 		newCtx := newLoggerForCall(ctx, logger, info.FullMethod, startTime)
 
@@ -39,7 +38,7 @@ func LoggerUnaryServerInterceptor(logger sdklogger.Logger) grpc.UnaryServerInter
 // LoggerStreamServerInterceptor returns a streaming server interceptor that adds the sdklogger.Logger to the context.
 func LoggerStreamServerInterceptor(logger sdklogger.Logger) grpc.StreamServerInterceptor {
 	return func(
-		srv interface{},
+		srv any,
 		stream grpc.ServerStream,
 		info *grpc.StreamServerInfo,
 		handler grpc.StreamHandler,
@@ -110,24 +109,23 @@ func log(ctx context.Context, err error, startTime time.Time) {
 		"grpc.time_ms": float32(time.Since(startTime).Nanoseconds()/1000) / 1000,
 	}
 
-	msg := fmt.Sprintf("finished gRPC call with code %s", code.String())
 	l, extractErr := sdkcontext.Extract(ctx)
 	if extractErr == nil {
 		l = l.WithFields(fields)
 
 		switch level {
 		case sdklogger.Debug:
-			l.Debugf(msg)
+			l.Debugf("finished gRPC call with code %s", code.String())
 		case sdklogger.Info:
-			l.Infof(msg)
+			l.Infof("finished gRPC call with code %s", code.String())
 		case sdklogger.Warn:
-			l.Warnf(msg)
+			l.Warnf("finished gRPC call with code %s", code.String())
 		case sdklogger.Error:
-			l.Errorf(msg)
+			l.Errorf("finished gRPC call with code %s", code.String())
 		case sdklogger.Fatal:
-			l.Fatalf(msg)
+			l.Fatalf("finished gRPC call with code %s", code.String())
 		case sdklogger.Panic:
-			l.Panicf(msg)
+			l.Panicf("finished gRPC call with code %s", code.String())
 		}
 	}
 }

pkg/interceptors/logger_test.go

@@ -65,7 +65,7 @@ func TestLoggerUnaryServerInterceptors(t *testing.T) {
 	_, err = client.Ping(context.Background(), &mwitkow_testproto.PingRequest{Value: "test"})
 	assert.Nil(t, err)
 
-	var fieldsUnary map[string]interface{}
+	var fieldsUnary map[string]any
 	err = json.Unmarshal(buffer.Bytes(), &fieldsUnary)
 	require.Nil(t, err)
 
@@ -125,7 +125,7 @@ func TestLoggerStreamServerInterceptors(t *testing.T) {
 	}
 	assert.Nil(t, err)
 
-	var fieldsStream map[string]interface{}
+	var fieldsStream map[string]any
 	err = json.Unmarshal(buffer.Bytes(), &fieldsStream)
 	require.Nil(t, err)
 
@@ -143,7 +143,7 @@ func getLogger(logLevel string, buf *bytes.Buffer) (sdklogger.Logger, error) {
 	return sdklogger.NewBuilder(config).BuildTestLogger(buf)
 }
 
-func checkLoggerFields(t *testing.T, fields map[string]interface{}) {
+func checkLoggerFields(t *testing.T, fields map[string]any) {
 	assert.NotEmpty(t, fields["message"])
 	assert.Equal(t, "info", fields["level"])
 	assert.NotEmpty(t, fields["timestamp"])
@@ -156,7 +156,7 @@ func checkLoggerFields(t *testing.T, fields map[string]interface{}) {
 	assert.NotEmpty(t, fields["grpc.time_ms"])
 	assert.NotEmpty(t, fields["grpc.request_id"])
 
-	var dd = (fields["dd"]).(map[string]interface{})
+	var dd = (fields["dd"]).(map[string]any)
 
 	assert.NotEmpty(t, dd["trace_id"])
 	assert.NotEmpty(t, dd["span_id"])