regression

rust-lang#2355

Author: syphar

src/web/crate_details.rs

@@ -584,8 +584,9 @@ pub(crate) async fn get_all_releases(
     Path(params): Path<RustdocHtmlParams>,
     mut conn: DbConnection,
 ) -> AxumResult<AxumResponse> {
-    let req_path: String = params.path.clone().unwrap_or_default();
-    let req_path: Vec<&str> = req_path.split('/').collect();
+    // NOTE: we're getting RustDocHtmlParams here, where both target and path are optional.
+    // Due to how this handler is used in the `releases_list` macro, we either have
+    // both values (when used in the topbar), or none (when used in the crate-details handler).
 
     let matched_release = match_version(&mut conn, &params.name, &params.version)
         .await?
@@ -599,37 +600,8 @@ pub(crate) async fn get_all_releases(
         return Err(AxumNope::CrateNotFound);
     }
 
-    let doc_targets = sqlx::query_scalar!(
-        "SELECT
-            releases.doc_targets
-         FROM releases
-         WHERE releases.id = $1;",
-        matched_release.id().0,
-    )
-    .fetch_optional(&mut *conn)
-    .await?
-    .ok_or(AxumNope::CrateNotFound)?
-    .map(MetaData::parse_doc_targets)
-    .ok_or_else(|| anyhow!("empty doc targets for successful release"))?;
-
-    let inner;
-    let (target, inner_path) = {
-        let mut inner_path = req_path.clone();
-
-        let target = if inner_path.len() > 1
-            && doc_targets
-                .iter()
-                .any(|s| Some(s) == params.target.as_ref())
-        {
-            inner_path.remove(0);
-            params.target.as_ref().unwrap()
-        } else {
-            ""
-        };
-
-        inner = inner_path.join("/");
-        (target, inner.trim_end_matches('/'))
-    };
+    let inner_path = params.path.unwrap_or_default();
+    let inner_path = inner_path.trim_end_matches('/');
 
     let target_name = matched_release
         .target_name()
@@ -641,19 +613,23 @@ pub(crate) async fn get_all_releases(
         format!("{target_name}/{inner_path}")
     };
 
-    let target = if target.is_empty() {
-        String::new()
-    } else {
-        format!("{target}/")
+    // NOTE: we don't check if the target exists here.
+    // If the target doesn't exist, the target-redirect will think
+    // it's part of the `inner_path`, don't find the file in storage,
+    // and redirect to a search.
+    let target = match params.target {
+        Some(t) if t.is_empty() => t,
+        Some(t) => format!("{t}/"),
+        None => String::new(),
     };
 
-    let res = ReleaseList {
+    Ok(ReleaseList {
         releases: matched_release.all_releases,
         target,
         inner_path,
         crate_name: params.name,
-    };
-    Ok(res.into_response())
+    }
+    .into_response())
 }
 
 #[derive(Debug, Clone, PartialEq)]