fix: auth route conflicts

robyn/__init__.py

@@ -118,7 +118,7 @@ def add_route(
         injected_dependencies = self.dependencies.get_dependency_map(self)
 
         if auth_required:
-            self.middleware_router.add_auth_middleware(endpoint)(handler)
+            self.middleware_router.add_auth_middleware(endpoint, route_type)(handler)
 
         if isinstance(route_type, str):
             http_methods = {

robyn/router.py

@@ -286,13 +286,19 @@ def add_route(
         self.route_middlewares.append(RouteMiddleware(middleware_type, endpoint, function))
         return handler
 
-    def add_auth_middleware(self, endpoint: str):
+    def add_auth_middleware(self, endpoint: str, method: HttpMethod):
         """
         This method adds an authentication middleware to the specified endpoint.
-        """
 
+        Args:
+            endpoint (str): The endpoint path
+            method (HttpMethod): The HTTP method for this route
+        """
         injected_dependencies = {}
 
+        # Create a unique route key that includes both method and path
+        route_key = f"{method}:{endpoint}"
+
         def decorator(handler):
             @wraps(handler)
             def inner_handler(request: Request, *args):
@@ -302,12 +308,11 @@ def inner_handler(request: Request, *args):
                 if identity is None:
                     return self.authentication_handler.unauthorized_response
                 request.identity = identity
-
                 return request
 
             self.add_route(
                 MiddlewareType.BEFORE_REQUEST,
-                endpoint,
+                route_key,
                 inner_handler,
                 injected_dependencies,
             )

src/routers/middleware_router.rs

@@ -38,14 +38,10 @@ impl Router<(FunctionInfo, HashMap<String, String>), MiddlewareType> for Middlew
     ) -> Option<(FunctionInfo, HashMap<String, String>)> {
         let table = self.routes.get(route_method)?;
 
-        let table_lock = table.read().ok()?;
-        let res = table_lock.at(route).ok()?;
-        let mut route_params = HashMap::new();
-        for (key, value) in res.params.iter() {
-            route_params.insert(key.to_string(), value.to_string());
+        match table.read().at(route) {
+            Ok(m) => Some((m.value.clone(), m.params.into_iter().collect())),
+            Err(_) => None,
         }
-
-        Some((res.value.to_owned(), route_params))
     }
 }