Update dependencies and fix new linter issues

Author: nicolaasuni

.golangci.yml

@@ -20,6 +20,7 @@ linters:
     - tagliatelle
     - testpackage
     - varnamelen
+    - wsl
   settings:
     depguard:
       rules:

Makefile

@@ -149,7 +149,7 @@ GOFMT=$(shell which gofmt)
 GOTEST=GOPATH=$(GOPATH) $(shell which gotest)
 GODOC=GOPATH=$(GOPATH) $(shell which godoc)
 GOLANGCILINT=$(BINUTIL)/golangci-lint
-GOLANGCILINTVERSION=v2.1.6
+GOLANGCILINTVERSION=v2.2.1
 DOCKERIZEVERSION=v0.9.2
 
 # Current operating system and architecture as one string.

VERSION

@@ -1 +1 @@
-5.4.1
+5.4.2

cmd/main.go

@@ -26,7 +26,8 @@ func main() {
 	}
 
 	// execute the root command and log errors (if any)
-	if err = rootCmd.Execute(); err != nil {
+	err = rootCmd.Execute()
+	if err != nil {
 		logging.LogFatal("UNABLE TO RUN THE COMMAND", zap.Error(err))
 	}
 }

go.mod

@@ -2,11 +2,11 @@ module github.com/tecnickcom/rndpwd
 
 go 1.24
 
-toolchain go1.24.4
+toolchain go1.24.5
 
 require (
-	github.com/Vonage/gosrvlib v1.102.2
-	github.com/go-playground/validator/v10 v10.26.0
+	github.com/Vonage/gosrvlib v1.102.6
+	github.com/go-playground/validator/v10 v10.27.0
 	github.com/golang/mock v1.6.0
 	github.com/jstemmer/go-junit-report/v2 v2.1.0
 	github.com/prometheus/client_golang v1.22.0
@@ -77,7 +77,7 @@ require (
 	github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 // indirect
 	github.com/prometheus/client_model v0.6.2 // indirect
 	github.com/prometheus/common v0.65.0 // indirect
-	github.com/prometheus/procfs v0.16.1 // indirect
+	github.com/prometheus/procfs v0.17.0 // indirect
 	github.com/sagikazarmark/crypt v0.29.0 // indirect
 	github.com/sagikazarmark/locafero v0.9.0 // indirect
 	github.com/sourcegraph/conc v0.3.0 // indirect
@@ -102,15 +102,15 @@ require (
 	golang.org/x/mod v0.25.0 // indirect
 	golang.org/x/net v0.41.0 // indirect
 	golang.org/x/oauth2 v0.30.0 // indirect
-	golang.org/x/sync v0.15.0 // indirect
-	golang.org/x/sys v0.33.0 // indirect
-	golang.org/x/text v0.26.0 // indirect
+	golang.org/x/sync v0.16.0 // indirect
+	golang.org/x/sys v0.34.0 // indirect
+	golang.org/x/text v0.27.0 // indirect
 	golang.org/x/time v0.12.0 // indirect
 	golang.org/x/tools v0.34.0 // indirect
-	google.golang.org/api v0.239.0 // indirect
-	google.golang.org/genproto v0.0.0-20250603155806-513f23925822 // indirect
-	google.golang.org/genproto/googleapis/api v0.0.0-20250603155806-513f23925822 // indirect
-	google.golang.org/genproto/googleapis/rpc v0.0.0-20250603155806-513f23925822 // indirect
+	google.golang.org/api v0.241.0 // indirect
+	google.golang.org/genproto v0.0.0-20250707201910-8d1bb00bc6a7 // indirect
+	google.golang.org/genproto/googleapis/api v0.0.0-20250707201910-8d1bb00bc6a7 // indirect
+	google.golang.org/genproto/googleapis/rpc v0.0.0-20250707201910-8d1bb00bc6a7 // indirect
 	google.golang.org/grpc v1.73.0 // indirect
 	google.golang.org/protobuf v1.36.6 // indirect
 	gopkg.in/yaml.v3 v3.0.1 // indirect

go.sum

@@ -36,7 +36,9 @@ func New(version, release string, bootstrapFn bootstrapFunc) (*cobra.Command, er
 	rootCmd.RunE = func(_ *cobra.Command, _ []string) error {
 		// Read CLI configuration
 		cfg := &appConfig{}
-		if err := config.Load(AppName, argConfigDir, appEnvPrefix, cfg); err != nil {
+
+		err := config.Load(AppName, argConfigDir, appEnvPrefix, cfg)
+		if err != nil {
 			return fmt.Errorf("failed loading config: %w", err)
 		}
 
@@ -90,7 +92,8 @@ func New(version, release string, bootstrapFn bootstrapFunc) (*cobra.Command, er
 
 	rootCmd.AddCommand(versionCmd)
 
-	if err := rootCmd.ParseFlags(os.Args); err != nil {
+	err := rootCmd.ParseFlags(os.Args)
+	if err != nil {
 		return nil, fmt.Errorf("failed parsing comman-line arguments: %w", err)
 	}
 

internal/cli/cli.go

@@ -104,7 +104,6 @@ func TestNew(t *testing.T) {
 			var out string
 
 			cmd, err := New("0.0.0-test", "0", bootstrapFunc)
-
 			if err == nil {
 				require.NotNil(t, cmd)
 

internal/cli/cli_test.go

@@ -57,10 +57,11 @@ type randomConfig struct {
 // appConfig contains the full application configuration.
 type appConfig struct {
 	config.BaseConfig `mapstructure:",squash" validate:"required"`
-	Enabled           bool         `mapstructure:"enabled"`
-	Servers           cfgServers   `mapstructure:"servers" validate:"required"`
-	Clients           cfgClients   `mapstructure:"clients" validate:"required"`
-	Random            randomConfig `mapstructure:"random"  validate:"required"`
+
+	Enabled bool         `mapstructure:"enabled"`
+	Servers cfgServers   `mapstructure:"servers" validate:"required"`
+	Clients cfgClients   `mapstructure:"clients" validate:"required"`
+	Random  randomConfig `mapstructure:"random"  validate:"required"`
 }
 
 // SetDefaults sets the default configuration values in Viper.

internal/cli/config.go

@@ -214,7 +214,9 @@ func Test_appConfig_Validate(t *testing.T) {
 			t.Parallel()
 
 			cfg := tt.fcfg(getValidTestConfig())
-			if err := cfg.Validate(); (err != nil) != tt.wantErr {
+
+			err := cfg.Validate()
+			if (err != nil) != tt.wantErr {
 				t.Errorf("Validate() error = %v, wantErr %v", err, tt.wantErr)
 			}
 		})