Testing context variable setting

grycap · Jan 26, 2024 · 0e18eb7 · 0e18eb7
1 parent 7dfbe5d
commit 0e18eb7
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/pkg/handlers/create.go b/pkg/handlers/create.go
@@ -52,8 +52,8 @@ func MakeCreateHandler(cfg *types.Config, back types.ServerlessBackend) gin.Hand
 	return func(c *gin.Context) {
 		var service types.Service
 
-		mcUntyped, mcExists := c.Get("mc")
 		uidOrigin, uidExists := c.Get("uid_origin")
+		mcUntyped, mcExists := c.Get("mc")
 
 		if !mcExists {
 			c.String(http.StatusInternalServerError, fmt.Sprintln("Missing multitenancy config"))
@@ -62,7 +62,7 @@ func MakeCreateHandler(cfg *types.Config, back types.ServerlessBackend) gin.Hand
 			c.String(http.StatusInternalServerError, fmt.Sprintln("Missing EGI user uid"))
 		}
 
-		mc, mcParsed := mcUntyped.(*auth.MultitenancyConfig)
+		mc, mcParsed := mcUntyped.(auth.MultitenancyConfig)
 		uid, uidParsed := uidOrigin.(string)
 
 		if !mcParsed {

diff --git a/pkg/utils/auth/oidc.go b/pkg/utils/auth/oidc.go
@@ -123,8 +123,8 @@ func getOIDCMiddleware(kubeClientset *kubernetes.Clientset, minIOAdminClient *ut
 			}
 		}
 		oidcLogger.Printf("User %s already exists", uid)
-		c.Set("multitenancyConfig", &mc)
-		c.Set("uidOrigin", &uid)
+		c.Set("uidOrigin", uid)
+		c.Set("multitenancyConfig", mc)
 		c.Next()
 	}
 }