Inconsistent `request.uri()` Behaviour in Dynamic Catch-All Routes #126

elcharitas · 2023-10-07T03:24:14Z

Problem Description:
I am currently implementing a catch-all route in my project using this runtime. Within the handler for this catch-all route, I need to access the request URI using request.uri().

Issue Details:

On the initial page load, everything appears to work as expected. The request.uri() correctly returns the expected path, such as "/hello."
However, upon refreshing the page or navigating to a different part of the application, I encounter unexpected behavior. The request.uri() function now returns "/api/[...all]" instead of the expected path as shown in the snippet below:

// api/[...all].rs
use vercel_runtime::{run, Body, Error, Request, Response};

#[tokio::main]
async fn main() -> Result<(), Error> {
    run(handler).await
}

pub async fn handler(req: Request) -> Result<Response<Body>, Error> {
    println!("request.uri, {}", request.uri()); // prints /hello then /api/[...all]
}

This behavior is unexpected and inconsistent, as I would expect request.uri() to consistently return the path of the current request. I have reviewed my vercel configuration and am confident that it's not causing this issue. Here's my vercel.json JIC.

{
  "functions": {
    "api/**/*.rs": {
      "runtime": "vercel-rust@4.0.6"
    }
  },
  "rewrites": [
    {
      "source": "/(.*)",
      "destination": "/api/$1"
    }
  ],
  "redirects": [
    {
      "source": "/",
      "destination": "/home",
      "statusCode": 301
    }
  ]
}

Steps to Reproduce:

Create a catch-all route using the exact snippet above.
Observe the value returned by request.uri().
Refresh the page or navigate to a different route.
Again, observe the value returned by request.uri(), which may now be unexpected.

Expected Behavior:
request.uri() should consistently return the correct path for the current request, regardless of how many times the page is refreshed or navigated.

Environment:

Rust version: 1.72
Vercel-Rust: 4.0.6

I appreciate any insights or suggestions on resolving this issue. Thank you!

The text was updated successfully, but these errors were encountered:

elcharitas · 2023-10-09T16:09:31Z

[UPDATE]

It seems the issue stems from lack of support for both rewrites and redirects with functions. Why? I have another sample where I used this simple vercel config and which works perfectly well:

{
  "functions": {
    "api/**/*.rs": {
      "runtime": "vercel-rust@4.0.6"
    }
  },
  "rewrites": [
    {
      "source": "/(.*)",
      "destination": "/api/$1"
    }
  ]
}

This means this issue relates to #121 except, I'm not making use of the bundling api

elcharitas · 2023-11-24T22:33:21Z

Further investigation shows that the issue described above has nothing to do with vercel.json or the rewrites config. It turns out the vercel rust runtime router is the best candidate to be held responsible for this bug as the issue totally disappears when an optional catch all route is created.

For anyone who may not be aware, there are two possible catch all routes file names which this runtime supports:

Dynamic catch all routes - [...all].rs
Optional catch all routes - [[...all]].rs

Until this issue is fixed (I may open a PR if I have the time), if you're experiencing this same issue, you can switch to making use of an optional catch all route which is more friendly IMO 😅

elcharitas changed the title ~~Inconsistent request.uri() Behaviour in Catch-All Routes~~ Inconsistent request.uri() Behaviour in Dynamic Catch-All Routes Nov 24, 2023

elcharitas mentioned this issue Nov 26, 2023

fix: use optional catch all routes in vercel app ngyn-rs/ngyn#54

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Inconsistent `request.uri()` Behaviour in Dynamic Catch-All Routes #126

Inconsistent `request.uri()` Behaviour in Dynamic Catch-All Routes #126

elcharitas commented Oct 7, 2023

elcharitas commented Oct 9, 2023

elcharitas commented Nov 24, 2023 •

edited

Inconsistent request.uri() Behaviour in Dynamic Catch-All Routes #126

Inconsistent request.uri() Behaviour in Dynamic Catch-All Routes #126

Comments

elcharitas commented Oct 7, 2023

elcharitas commented Oct 9, 2023

elcharitas commented Nov 24, 2023 • edited

Inconsistent `request.uri()` Behaviour in Dynamic Catch-All Routes #126

Inconsistent `request.uri()` Behaviour in Dynamic Catch-All Routes #126

elcharitas commented Nov 24, 2023 •

edited