double slash in URL no longer matches route handlers
weissi opened this issue · comments
Johannes Weiss commented
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 //
Johannes Weiss commented
CC @gwynne
Gwynne Raskind commented
Yep, already 3/4 done with the fix for this and several related issues I caused with that change 😅
Gwynne Raskind commented
Duping to #3138, the central tracking for the underlying issue.