reject empty route parameters

src/tree.rs

@@ -562,6 +562,11 @@ impl<T> Node<T> {
  NodeType::Param => {
  // Check for more path segments.
  let i = match path.iter().position(|&c| c == b'/') {
+ // Double `//` implying an empty parameter, no match.
+ Some(0) => {
+ try_backtrack!();
+ return Err(MatchError::NotFound);
+ }
  // Found another segment.
  Some(i) => i,
  // This is the last path segment.

tests/match.rs

@@ -578,6 +578,19 @@ fn escaped() {
  .run()
 }
 
+#[test]
+fn empty_param() {
+ MatchTest {
+ routes: vec!["/y/{foo}", "/x/{foo}/bar", "/foo/bar/{*xx}"],
+ matches: vec![
+ ("/y/", "", Err(())),
+ ("/x//bar", "", Err(())),
+ ("/foo/bar/", "", Err(())),
+ ],
+ }
+ .run();
+}
+
 #[test]
 fn basic() {
  MatchTest {