fix: trailing slash issue (sparckles#819)

* fixes sparckles#818

* fix: trailing slash issue

* use `str#ends_with`

* something like this

* update

* fix: error[E0716]: temporary value dropped while borrowed

* use only one `#clone`, pass the function to the other

* change back to inline assignment itself

* test: add tests

* docs: add comment

* fix(ci): change string to char

* fix(ci): formatting -- run `cargo fmt`

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Sanskar Jethi <29942790+sansyrox@users.noreply.github.com>

Author: 3 people

integration_tests/base_routes.py

@@ -219,6 +219,11 @@ async def hello_world(request):
     return "Hello, world!"
 
 
+@app.get("/trailing")
+def trailing_slash(request):
+    return "Trailing slash test successful!"
+
+
 @app.get("/sync/str")
 def sync_str_get():
     return "sync str get"

integration_tests/test_get_requests.py

@@ -1,4 +1,5 @@
 import pytest
+import requests
 from requests import Response
 
 from integration_tests.helpers.http_methods_helpers import get
@@ -51,3 +52,12 @@ def test_queries(function_type: str, session):
 
     r = get(f"/{function_type}/queries")
     assert r.json() == {}
+
+
+@pytest.mark.benchmark
+def test_trailing_slash(session):
+    r = requests.get("http://localhost:8080/trailing")  # `integration_tests#get` strips the trailing slash, tests always pass!`
+    assert r.text == "Trailing slash test successful!"
+
+    r = requests.get("http://localhost:8080/trailing/")
+    assert r.text == "Trailing slash test successful!"

src/server.rs

@@ -279,6 +279,24 @@ impl Server {
         route: &str,
         function: FunctionInfo,
         is_const: bool,
+    ) {
+        let second_route: String = if route.ends_with('/') {
+            route[0..route.len() - 1].to_string()
+        } else {
+            format!("{}/", route)
+        };
+
+        self._add_route(py, route_type, route, function.clone(), is_const);
+        self._add_route(py, route_type, &second_route, function, is_const);
+    }
+
+    fn _add_route(
+        &self,
+        py: Python,
+        route_type: &HttpMethod,
+        route: &str,
+        function: FunctionInfo,
+        is_const: bool,
     ) {
         debug!("Route added for {:?} {} ", route_type, route);
         let asyncio = py.import("asyncio").unwrap();