Merge pull request #4 from Isidroalex/change_mw

spatecon · web-flow · commit c79cae23f8d2 · 2023-09-01T15:38:30.000+02:00
Minor changes
diff --git a/internal/app/mw/mw.go b/internal/app/mw/mw.go
@@ -12,8 +12,9 @@ const roleAdmin = "admin"
 func RoleCheck(next echo.HandlerFunc) echo.HandlerFunc {
 	return func(ctx echo.Context) error {
 		val := ctx.Request().Header.Get("User-Role")
+		val = strings.ToLower(val)
 
-		if strings.EqualFold(val, roleAdmin) {
+		if strings.Contains(val, roleAdmin) {
 			log.Println("red button user detected")
 		}
 
diff --git a/internal/pkg/app/app.go b/internal/pkg/app/app.go
@@ -2,7 +2,6 @@ package app
 
 import (
 	"fmt"
-	"log"
 
 	"github.com/labstack/echo/v4"
 

Original file line number	Diff line number	Diff line change
`@@ -12,8 +12,9 @@ const roleAdmin = "admin"`
`12`	`12`	`func RoleCheck(next echo.HandlerFunc) echo.HandlerFunc {`
`13`	`13`	`return func(ctx echo.Context) error {`
`14`	`14`	`val := ctx.Request().Header.Get("User-Role")`
	`15`	`+ val = strings.ToLower(val)`
`15`	`16`
`16`		`- if strings.EqualFold(val, roleAdmin) {`
	`17`	`+ if strings.Contains(val, roleAdmin) {`
`17`	`18`	`log.Println("red button user detected")`
`18`	`19`	`}`
`19`	`20`