From ef4aea97ef344bf0f61ba3b50844987b7dac8169 Mon Sep 17 00:00:00 2001
From: toimtoimtoim <desinformatsioon@gmail.com>
Date: Tue, 21 Feb 2023 12:20:30 +0200
Subject: [PATCH] use different variable name so returned function would not
 accidentally be able to use it in future and cause data race

---
 middleware/csrf.go     | 6 +++---
 middleware/jwt.go      | 6 +++---
 middleware/key_auth.go | 6 +++---
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/middleware/csrf.go b/middleware/csrf.go
index 8661c9f8..6899700c 100644
--- a/middleware/csrf.go
+++ b/middleware/csrf.go
@@ -119,9 +119,9 @@ func CSRFWithConfig(config CSRFConfig) echo.MiddlewareFunc {
 		config.CookieSecure = true
 	}
 
-	extractors, err := CreateExtractors(config.TokenLookup)
-	if err != nil {
-		panic(err)
+	extractors, cErr := CreateExtractors(config.TokenLookup)
+	if cErr != nil {
+		panic(cErr)
 	}
 
 	return func(next echo.HandlerFunc) echo.HandlerFunc {
diff --git a/middleware/jwt.go b/middleware/jwt.go
index bd628264..bc318c97 100644
--- a/middleware/jwt.go
+++ b/middleware/jwt.go
@@ -196,9 +196,9 @@ func JWTWithConfig(config JWTConfig) echo.MiddlewareFunc {
 		config.ParseTokenFunc = config.defaultParseToken
 	}
 
-	extractors, err := createExtractors(config.TokenLookup, config.AuthScheme)
-	if err != nil {
-		panic(err)
+	extractors, cErr := createExtractors(config.TokenLookup, config.AuthScheme)
+	if cErr != nil {
+		panic(cErr)
 	}
 	if len(config.TokenLookupFuncs) > 0 {
 		extractors = append(config.TokenLookupFuncs, extractors...)
diff --git a/middleware/key_auth.go b/middleware/key_auth.go
index e8a6b085..f6fcc5d6 100644
--- a/middleware/key_auth.go
+++ b/middleware/key_auth.go
@@ -108,9 +108,9 @@ func KeyAuthWithConfig(config KeyAuthConfig) echo.MiddlewareFunc {
 		panic("echo: key-auth middleware requires a validator function")
 	}
 
-	extractors, err := createExtractors(config.KeyLookup, config.AuthScheme)
-	if err != nil {
-		panic(err)
+	extractors, cErr := createExtractors(config.KeyLookup, config.AuthScheme)
+	if cErr != nil {
+		panic(cErr)
 	}
 
 	return func(next echo.HandlerFunc) echo.HandlerFunc {