fix(route53,cloudfront): route trailing-slash collection paths to List (#1796)

Author: vieiralucas

crates/fakecloud-cloudfront/src/router.rs

@@ -52,11 +52,11 @@ impl Route {
 pub fn route(method: &Method, path: &str, raw_query: &str) -> Option<Route> {
     let path = path.strip_prefix("/2020-05-31").unwrap_or(path);
     let path = path.trim_start_matches('/');
-    let segs: Vec<&str> = if path.is_empty() {
-        Vec::new()
-    } else {
-        path.split('/').collect()
-    };
+    // Filter out empty segments so a trailing slash (`/distribution/`, which
+    // botocore/AWS CLI < 1.40 and curl emit for a collection root) routes to the
+    // List op, not GetDistribution(id="") -> NoSuchDistribution (the #1645
+    // shape; bug-audit 2026-06-20, 1.1).
+    let segs: Vec<&str> = path.split('/').filter(|s| !s.is_empty()).collect();
     let q = QueryFlags::parse(raw_query);
     match (method, segs.as_slice()) {
         // ─── Distributions ──────────────────────────────────────────
@@ -576,6 +576,14 @@ mod tests {
         assert_eq!(r.id.as_deref(), Some("EDFDVBD632BHDS5"));
     }
 
+    #[test]
+    fn trailing_slash_collection_routes_to_list() {
+        // A trailing slash on the collection root must list, not
+        // GetDistribution(id="") -> NoSuchDistribution (1.1).
+        let r = route(&Method::GET, "/2020-05-31/distribution/", "").unwrap();
+        assert_eq!(r.action, "ListDistributions");
+    }
+
     #[test]
     fn create_invalidation() {
         let r = route(

crates/fakecloud-route53/src/router.rs

@@ -44,11 +44,11 @@ pub fn route(method: &Method, path: &str, _raw_query: &str) -> Option<Route> {
         return None;
     }
     let path = path.trim_start_matches('/');
-    let segs: Vec<&str> = if path.is_empty() {
-        Vec::new()
-    } else {
-        path.split('/').collect()
-    };
+    // Filter out empty segments so a trailing slash (`/hostedzone/`, which
+    // botocore/AWS CLI < 1.40 and curl emit for a collection root) routes to the
+    // List op, not GetHostedZone(id="") -> NoSuchHostedZone (the #1645 shape;
+    // bug-audit 2026-06-20, 1.1).
+    let segs: Vec<&str> = path.split('/').filter(|s| !s.is_empty()).collect();
 
     match (method, segs.as_slice()) {
         // ─── Hosted Zones ────────────────────────────────────────────
@@ -265,4 +265,19 @@ mod tests {
             Some(Route::with_id("ChangeResourceRecordSets", "Z123"))
         );
     }
+
+    #[test]
+    fn trailing_slash_collection_routes_to_list() {
+        // botocore/AWS CLI < 1.40 and curl append `/` to a bare collection URI;
+        // it must route to the List op, not GetHostedZone(id="") (1.1).
+        assert_eq!(
+            route(&Method::GET, "/2013-04-01/hostedzone/", ""),
+            Some(Route::just("ListHostedZones"))
+        );
+        // Without a trailing slash still lists.
+        assert_eq!(
+            route(&Method::GET, "/2013-04-01/hostedzone", ""),
+            Some(Route::just("ListHostedZones"))
+        );
+    }
 }