From a273f6087d203675e5fdbf336cbc291c61b89529 Mon Sep 17 00:00:00 2001 From: William Will <10997562+willwill96@users.noreply.github.com> Date: Tue, 19 Aug 2025 11:21:02 -0600 Subject: [PATCH] linting --- pkg/apis/options/aws_iam.go | 6 +++--- pkg/apis/options/sessions.go | 26 +++++++++++++------------- pkg/sessions/redis/redis_store_test.go | 6 ++---- 3 files changed, 18 insertions(+), 20 deletions(-) diff --git a/pkg/apis/options/aws_iam.go b/pkg/apis/options/aws_iam.go index 54743a28..534ba718 100644 --- a/pkg/apis/options/aws_iam.go +++ b/pkg/apis/options/aws_iam.go @@ -2,9 +2,9 @@ package options type AWSIAMConfig struct { // AWS service redis service being used. "elasticache" or "memorydb" - ServiceName string `json:"serviceName,omitempty` + ServiceName string `json:"serviceName,omitempty"` // AWS Cluster name - ClusterName string `json:"clusterName,omitempty` + ClusterName string `json:"clusterName,omitempty"` // AWS Username - Username string `json:"userName,omitempty` + Username string `json:"userName,omitempty"` } diff --git a/pkg/apis/options/sessions.go b/pkg/apis/options/sessions.go index 80f0050c..d0085ff5 100644 --- a/pkg/apis/options/sessions.go +++ b/pkg/apis/options/sessions.go @@ -22,19 +22,19 @@ type CookieStoreOptions struct { // RedisStoreOptions contains configuration options for the RedisSessionStore. type RedisStoreOptions struct { - ConnectionURL string `flag:"redis-connection-url" cfg:"redis_connection_url"` - Username string `flag:"redis-username" cfg:"redis_username"` - Password string `flag:"redis-password" cfg:"redis_password"` - UseSentinel bool `flag:"redis-use-sentinel" cfg:"redis_use_sentinel"` - SentinelPassword string `flag:"redis-sentinel-password" cfg:"redis_sentinel_password"` - SentinelMasterName string `flag:"redis-sentinel-master-name" cfg:"redis_sentinel_master_name"` - SentinelConnectionURLs []string `flag:"redis-sentinel-connection-urls" cfg:"redis_sentinel_connection_urls"` - UseCluster bool `flag:"redis-use-cluster" cfg:"redis_use_cluster"` - ClusterConnectionURLs []string `flag:"redis-cluster-connection-urls" cfg:"redis_cluster_connection_urls"` - CAPath string `flag:"redis-ca-path" cfg:"redis_ca_path"` - InsecureSkipTLSVerify bool `flag:"redis-insecure-skip-tls-verify" cfg:"redis_insecure_skip_tls_verify"` - IdleTimeout int `flag:"redis-connection-idle-timeout" cfg:"redis_connection_idle_timeout"` - AWSIAMConfig *AWSIAMConfig `cfg:",internal"` + ConnectionURL string `flag:"redis-connection-url" cfg:"redis_connection_url"` + Username string `flag:"redis-username" cfg:"redis_username"` + Password string `flag:"redis-password" cfg:"redis_password"` + UseSentinel bool `flag:"redis-use-sentinel" cfg:"redis_use_sentinel"` + SentinelPassword string `flag:"redis-sentinel-password" cfg:"redis_sentinel_password"` + SentinelMasterName string `flag:"redis-sentinel-master-name" cfg:"redis_sentinel_master_name"` + SentinelConnectionURLs []string `flag:"redis-sentinel-connection-urls" cfg:"redis_sentinel_connection_urls"` + UseCluster bool `flag:"redis-use-cluster" cfg:"redis_use_cluster"` + ClusterConnectionURLs []string `flag:"redis-cluster-connection-urls" cfg:"redis_cluster_connection_urls"` + CAPath string `flag:"redis-ca-path" cfg:"redis_ca_path"` + InsecureSkipTLSVerify bool `flag:"redis-insecure-skip-tls-verify" cfg:"redis_insecure_skip_tls_verify"` + IdleTimeout int `flag:"redis-connection-idle-timeout" cfg:"redis_connection_idle_timeout"` + AWSIAMConfig *AWSIAMConfig `cfg:",internal"` } func sessionOptionsDefaults() SessionOptions { diff --git a/pkg/sessions/redis/redis_store_test.go b/pkg/sessions/redis/redis_store_test.go index a5471633..ca0af0ef 100644 --- a/pkg/sessions/redis/redis_store_test.go +++ b/pkg/sessions/redis/redis_store_test.go @@ -284,8 +284,7 @@ var _ = Describe("Redis SessionStore Tests", func() { }, } - var opt *redis.Options - opt = &redis.Options{} + var opt = &redis.Options{} err := setupAWSIAMAuth(redisOpts, opt) Expect(err).ToNot(HaveOccurred()) @@ -300,8 +299,7 @@ var _ = Describe("Redis SessionStore Tests", func() { AWSIAMConfig: nil, } - var opt *redis.Options - opt = &redis.Options{} + var opt = &redis.Options{} err := setupAWSIAMAuth(redisOpts, opt) Expect(err).ToNot(HaveOccurred())