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));

src/types/headers.rs

@@ -49,7 +49,7 @@ impl Headers {
         debug!("Setting header {} to {}", key, value);
         self.headers
             .entry(key.to_lowercase())
-            .or_insert_with(Vec::new)
+            .or_default()
             .push(value);
     }
 
@@ -105,10 +105,7 @@ impl Headers {
                 self.headers.entry(key).or_insert_with(Vec::new).push(value);
             } else {
                 let value: Vec<String> = new_value.unwrap().iter().map(|x| x.to_string()).collect();
-                self.headers
-                    .entry(key)
-                    .or_insert_with(Vec::new)
-                    .extend(value);
+                self.headers.entry(key).or_default().extend(value);
             }
         }
     }
@@ -167,11 +164,7 @@ impl Headers {
         for (key, value) in req_headers {
             let key = key.to_string().to_lowercase();
             let value = value.to_str().unwrap().to_string();
-            headers
-                .headers
-                .entry(key)
-                .or_insert_with(Vec::new)
-                .push(value);
+            headers.headers.entry(key).or_default().push(value);
         }
 
         headers

src/types/multimap.rs

@@ -21,7 +21,7 @@ impl QueryParams {
 
     pub fn set(&mut self, key: String, value: String) {
         debug!("Setting key: {} to value: {}", key, value);
-        self.queries.entry(key).or_insert_with(Vec::new).push(value);
+        self.queries.entry(key).or_default().push(value);
         debug!("Multimap: {:?}", self.queries);
     }