feat: optimize robyn

Author: sansyrox

Cargo.toml

@@ -44,9 +44,13 @@ io-uring = ["actix-web/experimental-io-uring"]
 
 [profile.release]
 codegen-units = 1
-lto = true
+lto = "fat"
 panic = "abort"
 strip = true
+opt-level = 3
+
+[profile.release.build-override]
+opt-level = 3
 
 [package.metadata.maturin]
 name = "robyn"

src/executors/mod.rs

@@ -30,7 +30,7 @@ where
     let function_args: PyObject = function_args
         .clone()
         .into_pyobject(py)
-        .unwrap()
+        .map_err(|e| PyErr::new::<pyo3::exceptions::PyRuntimeError, _>(format!("Failed to convert args: {:?}", e)))?
         .into_any()
         .unbind();
     debug!("Function args: {:?}", function_args);

src/server.rs

@@ -545,13 +545,11 @@ async fn index(
         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));
+                return Response::internal_server_error(None);
             }
             Ok(MiddlewareReturn::Response(r)) => {
-                let response = r;
-
-                debug!("Response returned: {:?}", response);
-                response
+                debug!("Response returned: {:?}", r);
+                r
             }
             Err(e) => {
                 error!(

src/types/request.rs

@@ -41,9 +41,9 @@ impl<'py> IntoPyObject<'py> for Request {
         let body = if self.body.is_empty() {
             PyString::new(py, "").into_any()
         } else {
-            match String::from_utf8(self.body.clone()) {
-                Ok(s) => s.into_pyobject(py)?.into_any(),
-                Err(_) => self.body.into_pyobject(py)?.into_any(),
+            match std::str::from_utf8(&self.body) {
+                Ok(s) => PyString::new(py, s).into_any(),
+                Err(_) => PyBytes::new(py, &self.body).into_any(),
             }
         };
 

src/types/response.rs

@@ -28,7 +28,7 @@ impl Responder for Response {
 
     fn respond_to(self, _req: &HttpRequest) -> HttpResponse<Self::Body> {
         let mut response_builder =
-            HttpResponseBuilder::new(StatusCode::from_u16(self.status_code).unwrap());
+            HttpResponseBuilder::new(StatusCode::from_u16(self.status_code).unwrap_or(StatusCode::INTERNAL_SERVER_ERROR));
         apply_hashmap_headers(&mut response_builder, &self.headers);
         response_builder.body(self.description)
     }