fix: refactor linting and session handling in Go codebase

- Update golangci-lint-action from v6 to v7 in GitHub workflow
- Change golangci-lint version to v2.0 and remove timeout argument
- Fix assignment of session options in gorm.go
- Reorder return values in GetRedisStore function and update return statements in redis.go

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>

Author: appleboy

.github/workflows/lint.yml

@@ -26,7 +26,7 @@ jobs:
           ref: ${{ github.ref }}
 
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v6
+        uses: golangci/golangci-lint-action@v7
         with:
-          version: latest
-          args: --verbose --timeout 10m
+          version: v2.0
+          args: --verbose

gorm/gorm.go

@@ -26,5 +26,5 @@ type store struct {
 }
 
 func (s *store) Options(options sessions.Options) {
-	s.Store.SessionOpts = options.ToGorillaOptions()
+	s.SessionOpts = options.ToGorillaOptions()
 }

redis/redis.go

@@ -88,15 +88,15 @@ type store struct {
 // Returns:
 //   - err: An error if the provided Store is not of the expected type.
 //   - rediStore: The retrieved Redis store, or nil if there was an error.
-func GetRedisStore(s Store) (err error, rediStore *redistore.RediStore) {
+func GetRedisStore(s Store) (rediStore *redistore.RediStore, err error) {
 	realStore, ok := s.(*store)
 	if !ok {
 		err = errors.New("unable to get the redis store: Store isn't *store")
-		return
+		return nil, err
 	}
 
 	rediStore = realStore.RediStore
-	return
+	return rediStore, nil
 }
 
 // SetKeyPrefix sets a key prefix for the given Redis store.
@@ -110,7 +110,7 @@ func GetRedisStore(s Store) (err error, rediStore *redistore.RediStore) {
 // Returns:
 //   - error: An error if there is an issue retrieving the Redis store, otherwise nil.
 func SetKeyPrefix(s Store, prefix string) error {
-	err, rediStore := GetRedisStore(s)
+	rediStore, err := GetRedisStore(s)
 	if err != nil {
 		return err
 	}