vapor/vapor

double slash in URL no longer matches route handlers

weissi opened this issue · 3 comments

Consider this route

app.get("foo", "bar", "buz") { request -> String in
    return try request.query.get(at: "v")
}

Before 6db3d91 everything worked as expected:

$ curl http://127.1:8080/foo/bar/buz?v=M%26M
M&M

But after that (including 4.91.1), this will fail with

$ curl http://127.1:8080//foo/bar/buz?v=M%26M
{"error":true,"reason":"Not Found"}

because it seemingly no longer matches the route on //

Yep, already 3/4 done with the fix for this and several related issues I caused with that change 😅

Duping to #3138, the central tracking for the underlying issue.