Merge pull request #162 from scribd/maksimt/DEVPLAT-6703/bump-go

[DEVPLAT-6703] Bump go version to v1.26.2

Author: Neurostep

Dockerfile

@@ -2,7 +2,7 @@
 # build stage
 # =============================================================================
 
-FROM golang:1.25.5-alpine AS builder
+FROM golang:1.26.2-alpine AS builder
 
 WORKDIR /sdk
 
@@ -33,15 +33,15 @@ FROM builder AS linter
 
 # binary will be $(go env GOPATH)/bin/golangci-lint
 RUN curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh \
-	| sh -s -- -b $(go env GOPATH)/bin v2.4.0
+	| sh -s -- -b $(go env GOPATH)/bin v2.11.4
 
 # install goimports
-RUN go install golang.org/x/tools/cmd/goimports@v0.36.0
+RUN go install golang.org/x/tools/cmd/goimports@v0.44.0
 
 # =============================================================================
 # development stage
 # =============================================================================
 
 FROM linter AS development
 
-RUN go install github.com/go-delve/delve/cmd/dlv@v1.25.1
+RUN go install github.com/go-delve/delve/cmd/dlv@v1.26.2

README.md

@@ -67,7 +67,7 @@ SDK, the Go version.
 
 ## Prerequisites
 
-* [Go](https://golang.org) (version `1.25.0`).
+* [Go](https://golang.org) (version `1.26.2`).
 * [Docker](https://www.docker.com/) (version `19.03.2`).
 
 ## SDK functionality
@@ -1743,7 +1743,7 @@ You can enter the docker environment to build, run and debug your service:
 ```
 $ docker-compose run --rm sdk /bin/bash
 root@1f31fa8e5c49:/sdk# go version
-go version go1.25.0 linux/amd64
+go version go1.26.2 linux/amd64
 ```
 
 Refer to the

go.mod

@@ -1,6 +1,6 @@
 module github.com/scribd/go-sdk
 
-go 1.25.5
+go 1.26.2
 
 require (
 	github.com/DATA-DOG/go-sqlmock v1.5.2

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,