Release v0.49.0

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "robyn"
-version = "0.48.0"
+version = "0.49.0"
 authors = ["Sanskar Jethi <[email protected]>"]
 edition = "2021"
 description = "A web server that is fast!"

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "maturin"
 
 [project]
 name = "robyn"
-version = "0.48.0"
+version = "0.49.0"
 description = "A High-Performance, Community-Driven, and Innovator Friendly Web Framework with a Rust runtime."
 authors = [{ name = "Sanskar Jethi", email = "[email protected]" }]
 repository = "https://github.com/sparckles/robyn"
@@ -43,7 +43,7 @@ Changelog = "https://github.com/sparckles/robyn/blob/main/CHANGELOG.md"
 
 [tool.poetry]
 name = "robyn"
-version = "0.48.0"
+version = "0.49.0"
 description = "A High-Performance, Community-Driven, and Innovator Friendly Web Framework with a Rust runtime."
 authors = ["Sanskar Jethi <[email protected]>"]
 

src/executors/mod.rs

@@ -1,3 +1,4 @@
+#[deny(clippy::if_same_then_else)]
 /// This is the module that has all the executor functions
 /// i.e. the functions that have the responsibility of parsing and executing functions.
 pub mod web_socket_executors;
@@ -31,21 +32,24 @@ where
     match function.number_of_params {
         0 => handler.call0(),
         1 => {
-            if function.args.as_ref(py).get_item("request").is_some() {
-                handler.call1((function_args,))
-            } else if function.args.as_ref(py).get_item("response").is_some() {
-                // this is for middlewares
+            if function.args.as_ref(py).get_item("request").is_some()
+                || function.args.as_ref(py).get_item("response").is_some()
+            {
+                // If 'request' is present, call handler with 'function_args'
                 handler.call1((function_args,))
             } else {
+                // If neither 'request' nor 'response' is present
                 handler.call((), Some(kwargs))
             }
         }
         2 => {
             if function.args.as_ref(py).get_item("request").is_some()
                 || function.args.as_ref(py).get_item("response").is_some()
             {
+                // If either 'request' or 'response' is present, call handler with 'function_args' and 'kwargs'
                 handler.call((function_args,), Some(kwargs))
             } else {
+                // If neither 'request' nor 'response' is present
                 handler.call((), Some(kwargs))
             }
         }

src/server.rs

@@ -388,7 +388,7 @@ async fn index(
         request.path_params = route_params;
     }
     for before_middleware in before_middlewares {
-        request = match execute_middleware_function(&mut request, &before_middleware).await {
+        request = match execute_middleware_function(&request, &before_middleware).await {
             Ok(MiddlewareReturn::Request(r)) => r,
             Ok(MiddlewareReturn::Response(r)) => {
                 // If a before middleware returns a response, we abort the request and return the response
@@ -449,7 +449,7 @@ async fn index(
         after_middlewares.push(function);
     }
     for after_middleware in after_middlewares {
-        response = match execute_middleware_function(&mut response, &after_middleware).await {
+        response = match execute_middleware_function(&response, &after_middleware).await {
             Ok(MiddlewareReturn::Request(_)) => {
                 error!("After middleware returned a request");
                 return Response::internal_server_error(Some(&response.headers));