Merge pull request #135 from opengovern/feat-status-codes

feat: change status of role check
opengovern · Oct 28, 2024 · 3f20bde · 3f20bde
2 parents 668e969 + 53f529c
commit 3f20bde
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/httpserver/auth.go b/pkg/httpserver/auth.go
@@ -29,7 +29,7 @@ func RequireMinRole(ctx echo.Context, minRole api.Role) error {
 	if !hasAccess(GetUserRole(ctx), minRole) {
 		userRole := GetUserRole(ctx)
 		fmt.Println("required role = ", minRole, " user role = ", userRole)
-		return echo.NewHTTPError(http.StatusForbidden, "missing required permission")
+		return echo.NewHTTPError(http.StatusNotAcceptable, "missing required permission")
 	}
 
 	return nil
@@ -69,7 +69,7 @@ func CheckAccessToConnectionID(ctx echo.Context, connectionID string) error {
 			return nil
 		}
 	}
-	return echo.NewHTTPError(http.StatusForbidden, "Invalid connection ID")
+	return echo.NewHTTPError(http.StatusNotAcceptable, "Invalid connection ID")
 }
 
 func ResolveConnectionIDs(ctx echo.Context, connectionIDs []string) ([]string, error) {