Performance Improvements (#147)

* optimized int to string casting; reduced memory allocations for 405; reduced memory allocations on routing

* args extraction performance improvements

* added unit tests for path args

* added query args caching

Author: RomanEmreis

CHANGELOG.md

@@ -13,6 +13,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 - Removed unnecessary boxing for `HttpBody::full`, `HttpBody::json` and `HttpBody::form`
 - Added additional methods: `HttpBody::text`, `HttpBody::from_static`, `HttpBody::from_static_text`, `HttpBody::from_str` and `HttpBody::from_slice` for better control.
 - Improved performance for all producing response macros and `IntoResponse` implementations.
+- Improved overall extractors' performance + added additional validation for the cases when mixed Path with NamedPath and positional arguments.
+- Added cached query-like path string handling for PathArgs, including into_parts/from_parts, and preserved the cache when cloning to reduce recomputation in extractor/middleware flows.
+- Switched route/allow header internals to Arc<str> and cached Allow values on route nodes to avoid recomputing 405 headers, plus simplified HSTS formatting to avoid intermediate allocation.
+- Added cached query-arg parsing in HttpRequest/HttpRequestMut to reduce repeated query parsing across extractors.
 
 ## 0.8.0
 

volga-dev-cert/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "volga-dev-cert"
-version = "0.8.0"
+version = "0.8.1"
 readme = "README.md"
 description = "A Rust library for generating self-signed TLS certificates for local development."
 edition.workspace = true

volga-di/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "volga-di"
-version = "0.8.0"
+version = "0.8.1"
 readme = "README.md"
 description = "Dependency Injection tools for Volga Web Framework"
 edition.workspace = true

volga-macros/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "volga-macros"
-version = "0.8.0"
+version = "0.8.1"
 readme = "README.md"
 description = "Macros for Volga Web Framework"
 edition.workspace = true

volga-rate-limiter/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "volga-rate-limiter"
-version = "0.8.0"
+version = "0.8.1"
 readme = "README.md"
 description = "Macros for Volga Web Framework"
 edition.workspace = true

volga/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "volga"
-version = "0.8.0"
+version = "0.8.1"
 readme = "../README.md"
 description = "Easy & Fast Web Framework for Rust"
 edition.workspace = true
@@ -52,10 +52,10 @@ twox-hash = { version = "2.1.2", optional = true }
 uuid = { version = "1.19.0", features = ["v4"], optional = true }
 
 # volga
-volga-dev-cert = { path = "../volga-dev-cert", version = "0.8.0", optional = true }
-volga-di = { path = "../volga-di", version = "0.8.0", optional = true }
-volga-macros = { path = "../volga-macros", version = "0.8.0", optional = true }
-volga-rate-limiter = { path = "../volga-rate-limiter", version = "0.8.0", optional = true }
+volga-dev-cert = { path = "../volga-dev-cert", version = "0.8.1", optional = true }
+volga-di = { path = "../volga-di", version = "0.8.1", optional = true }
+volga-macros = { path = "../volga-macros", version = "0.8.1", optional = true }
+volga-rate-limiter = { path = "../volga-rate-limiter", version = "0.8.1", optional = true }
 
 [dev-dependencies]
 base64 = { version = "0.22.1" }

volga/src/app/scope.rs

@@ -177,7 +177,7 @@ async fn handle_impl(
     ) {
         FindResult::RouteNotFound => pipeline.fallback(request).await,
         FindResult::MethodNotFound(allowed) => status!(405; [
-            (ALLOW, allowed)
+            (ALLOW, allowed.as_ref())
         ]),
         FindResult::Ok(endpoint) => {
             #[cfg(feature = "middleware")]

volga/src/http/body/into_body.rs

@@ -84,6 +84,17 @@ impl<T: Serialize> TryFrom<Form<T>> for HttpBody {
     }
 }
 
+macro_rules! impl_int_into_body {
+    { $($type:ident),* $(,)? } => {
+        $(impl From<$type> for HttpBody {
+            #[inline]
+            fn from(s: $type) -> Self {
+                Self::text_ref(itoa::Buffer::new().format(s))
+            }
+        })*
+    };
+}
+
 macro_rules! impl_into_body {
     { $($type:ident),* $(,)? } => {
         $(impl From<$type> for HttpBody {
@@ -96,13 +107,17 @@ macro_rules! impl_into_body {
 }
 
 impl_into_body! {
-    bool, char,
+    bool, 
+    char, 
+    f32, 
+    f64
+}
+
+impl_int_into_body! {
     i8, u8,
     i16, u16,
     i32, u32,
-    f32,
     i64, u64,
-    f64,
     i128, u128,
     isize, usize
 }

volga/src/http/cors.rs

@@ -393,7 +393,9 @@ impl CorsConfig {
     pub fn max_age(&self) -> Option<HeaderValue> {
         match &self.max_age { 
             None => None,
-            Some(max_age) => HeaderValue::from_str(&max_age.as_secs().to_string()).ok()
+            Some(max_age) => HeaderValue::from_str(
+                itoa::Buffer::new().format(max_age.as_secs())
+            ).ok()
         }
     }
 

volga/src/http/endpoints.rs

@@ -1,8 +1,9 @@
 //! Endpoints mapping utilities
 
+use std::sync::Arc;
 use hyper::{Method, Uri};
 use super::endpoints::{
-    route::{RouteNode, RoutePipeline, PathArgs, make_allowed_str},
+    route::{RouteNode, RoutePipeline, PathArgs},
     handlers::RouteHandler
 };
 
@@ -27,7 +28,7 @@ pub(crate) struct Endpoints {
 /// Specifies statuses that could be returned after route matching
 pub(crate) enum FindResult {
     RouteNotFound,
-    MethodNotFound(String),
+    MethodNotFound(Arc<str>),
     Ok(Endpoint)
 }
 
@@ -108,7 +109,7 @@ impl Endpoints {
                 return handlers
                     .binary_search_by(|h| h.cmp(&target_method))
                     .map_or_else(
-                        |_| FindResult::MethodNotFound(make_allowed_str(handlers)),
+                        |_| FindResult::MethodNotFound(route_params.route.allowed_methods()),
                         |i| {
                             let handler = &handlers[i];
                             FindResult::Ok(
@@ -126,7 +127,7 @@ impl Endpoints {
         handlers
             .binary_search_by(|h| h.cmp(method))
             .map_or_else(
-                |_| FindResult::MethodNotFound(make_allowed_str(handlers)),
+                |_| FindResult::MethodNotFound(route_params.route.allowed_methods()),
                 |i| {
                     let handler = &handlers[i];
                     FindResult::Ok(
@@ -265,7 +266,7 @@ mod tests {
         );
 
         match post_handler {
-            FindResult::MethodNotFound(allow) => assert_eq!(allow, "GET"),
+            FindResult::MethodNotFound(allow) => assert_eq!(allow.as_ref(), "GET"),
             _ => panic!("`post_handler` must be is the `MethodNotFound` state")
         }
     }