From edd6ae6886f889cfc3ab96febb511f8d39c742be Mon Sep 17 00:00:00 2001
From: Tobias Bieniek <tobias@bieniek.cloud>
Date: Thu, 28 Sep 2023 08:57:19 +0200
Subject: [PATCH 1/2] publish: Use `homepage`, `documentation` and `repository`
 fields from embedded `Cargo.toml` file

---
 src/controllers/krate/publish.rs | 20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

diff --git a/src/controllers/krate/publish.rs b/src/controllers/krate/publish.rs
index e3f7cc20b59..b46ffa6ae18 100644
--- a/src/controllers/krate/publish.rs
+++ b/src/controllers/krate/publish.rs
@@ -122,6 +122,9 @@ pub async fn publish(app: AppState, req: BytesRequest) -> AppResult<Json<GoodCra
         let description = package.description.map(|it| it.as_local().unwrap());
         let mut license = package.license.map(|it| it.as_local().unwrap());
         let license_file = package.license_file.map(|it| it.as_local().unwrap());
+        let homepage = package.homepage.map(|it| it.as_local().unwrap());
+        let documentation = package.documentation.map(|it| it.as_local().unwrap());
+        let repository = package.repository.map(|it| it.as_local().unwrap());
 
         // Make sure required fields are provided
         fn empty(s: Option<&String>) -> bool {
@@ -150,13 +153,16 @@ pub async fn publish(app: AppState, req: BytesRequest) -> AppResult<Json<GoodCra
             license = Some(String::from("non-standard"));
         }
 
+        validate_url(homepage.as_deref(), "homepage")?;
+        validate_url(documentation.as_deref(), "documentation")?;
+        validate_url(repository.as_deref(), "repository")?;
+
         // Create a transaction on the database, if there are no errors,
         // commit the transactions to record a new or updated crate.
         conn.transaction(|conn| {
             let name = metadata.name;
             let vers = &*metadata.vers;
             let links = metadata.links;
-            let repo = metadata.repository;
             let features = metadata
                 .features
                 .into_iter()
@@ -177,17 +183,13 @@ pub async fn publish(app: AppState, req: BytesRequest) -> AppResult<Json<GoodCra
             let persist = NewCrate {
                 name: &name,
                 description: description.as_deref(),
-                homepage: metadata.homepage.as_deref(),
-                documentation: metadata.documentation.as_deref(),
+                homepage: homepage.as_deref(),
+                documentation: documentation.as_deref(),
                 readme: metadata.readme.as_deref(),
-                repository: repo.as_deref(),
+                repository: repository.as_deref(),
                 max_upload_size: None,
             };
 
-            validate_url(persist.homepage, "homepage")?;
-            validate_url(persist.documentation, "documentation")?;
-            validate_url(persist.repository, "repository")?;
-
             if is_reserved_name(persist.name, conn)? {
                 return Err(cargo_err("cannot upload a crate with a reserved name"));
             }
@@ -271,7 +273,7 @@ pub async fn publish(app: AppState, req: BytesRequest) -> AppResult<Json<GoodCra
                         metadata
                             .readme_file
                             .unwrap_or_else(|| String::from("README.md")),
-                        repo,
+                        repository,
                         pkg_path_in_vcs,
                     )
                     .enqueue_with_priority(conn, PRIORITY_RENDER_README)?;

From 31f15486fef26938e40375fdb04e9c1b73c5d5a9 Mon Sep 17 00:00:00 2001
From: Tobias Bieniek <tobias@bieniek.cloud>
Date: Thu, 28 Sep 2023 09:00:09 +0200
Subject: [PATCH 2/2] views/krate_publish: Remove obsolete `homepage`,
 `documentation` and `repository` fields

---
 src/tests/builders/publish.rs                                  | 3 ---
 .../all__krate__publish__categories__too_many_categories.snap  | 2 +-
 .../all__krate__publish__keywords__bad_keywords-2.snap         | 2 +-
 .../all__krate__publish__keywords__bad_keywords-3.snap         | 2 +-
 .../snapshots/all__krate__publish__keywords__bad_keywords.snap | 2 +-
 .../all__krate__publish__keywords__too_many_keywords.snap      | 2 +-
 src/views/krate_publish.rs                                     | 3 ---
 7 files changed, 5 insertions(+), 11 deletions(-)

diff --git a/src/tests/builders/publish.rs b/src/tests/builders/publish.rs
index 6a344a62531..6bcc2dd40df 100644
--- a/src/tests/builders/publish.rs
+++ b/src/tests/builders/publish.rs
@@ -142,8 +142,6 @@ impl PublishBuilder {
             vers: u::EncodableCrateVersion(self.version.clone()),
             features: self.features.clone(),
             deps: self.deps.clone(),
-            homepage: None,
-            documentation: self.doc_url.clone(),
             readme: self.readme,
             readme_file: None,
             keywords: u::EncodableKeywordList(
@@ -160,7 +158,6 @@ impl PublishBuilder {
                     .map(u::EncodableCategory)
                     .collect(),
             ),
-            repository: None,
             links: None,
         };
 
diff --git a/src/tests/krate/publish/snapshots/all__krate__publish__categories__too_many_categories.snap b/src/tests/krate/publish/snapshots/all__krate__publish__categories__too_many_categories.snap
index f1300f6967d..601abf858d9 100644
--- a/src/tests/krate/publish/snapshots/all__krate__publish__categories__too_many_categories.snap
+++ b/src/tests/krate/publish/snapshots/all__krate__publish__categories__too_many_categories.snap
@@ -5,7 +5,7 @@ expression: response.into_json()
 {
   "errors": [
     {
-      "detail": "invalid upload request: invalid length 6, expected at most 5 categories per crate at line 1 column 191"
+      "detail": "invalid upload request: invalid length 6, expected at most 5 categories per crate at line 1 column 154"
     }
   ]
 }
diff --git a/src/tests/krate/publish/snapshots/all__krate__publish__keywords__bad_keywords-2.snap b/src/tests/krate/publish/snapshots/all__krate__publish__keywords__bad_keywords-2.snap
index 82ef682c51d..3a5f4a5cd87 100644
--- a/src/tests/krate/publish/snapshots/all__krate__publish__keywords__bad_keywords-2.snap
+++ b/src/tests/krate/publish/snapshots/all__krate__publish__keywords__bad_keywords-2.snap
@@ -5,7 +5,7 @@ expression: response.into_json()
 {
   "errors": [
     {
-      "detail": "invalid upload request: invalid value: string \"?@?%\", expected a valid keyword specifier at line 1 column 150"
+      "detail": "invalid upload request: invalid value: string \"?@?%\", expected a valid keyword specifier at line 1 column 113"
     }
   ]
 }
diff --git a/src/tests/krate/publish/snapshots/all__krate__publish__keywords__bad_keywords-3.snap b/src/tests/krate/publish/snapshots/all__krate__publish__keywords__bad_keywords-3.snap
index f55b4d0f859..789e18bacf7 100644
--- a/src/tests/krate/publish/snapshots/all__krate__publish__keywords__bad_keywords-3.snap
+++ b/src/tests/krate/publish/snapshots/all__krate__publish__keywords__bad_keywords-3.snap
@@ -5,7 +5,7 @@ expression: response.into_json()
 {
   "errors": [
     {
-      "detail": "invalid upload request: invalid value: string \"áccênts\", expected a valid keyword specifier at line 1 column 155"
+      "detail": "invalid upload request: invalid value: string \"áccênts\", expected a valid keyword specifier at line 1 column 118"
     }
   ]
 }
diff --git a/src/tests/krate/publish/snapshots/all__krate__publish__keywords__bad_keywords.snap b/src/tests/krate/publish/snapshots/all__krate__publish__keywords__bad_keywords.snap
index 287958e9cb0..1d4484bf966 100644
--- a/src/tests/krate/publish/snapshots/all__krate__publish__keywords__bad_keywords.snap
+++ b/src/tests/krate/publish/snapshots/all__krate__publish__keywords__bad_keywords.snap
@@ -5,7 +5,7 @@ expression: response.into_json()
 {
   "errors": [
     {
-      "detail": "invalid upload request: invalid length 29, expected a keyword with less than 20 characters at line 1 column 175"
+      "detail": "invalid upload request: invalid length 29, expected a keyword with less than 20 characters at line 1 column 138"
     }
   ]
 }
diff --git a/src/tests/krate/publish/snapshots/all__krate__publish__keywords__too_many_keywords.snap b/src/tests/krate/publish/snapshots/all__krate__publish__keywords__too_many_keywords.snap
index 45130bd6de1..cb921102eeb 100644
--- a/src/tests/krate/publish/snapshots/all__krate__publish__keywords__too_many_keywords.snap
+++ b/src/tests/krate/publish/snapshots/all__krate__publish__keywords__too_many_keywords.snap
@@ -5,7 +5,7 @@ expression: response.into_json()
 {
   "errors": [
     {
-      "detail": "invalid upload request: invalid length 6, expected at most 5 keywords per crate at line 1 column 175"
+      "detail": "invalid upload request: invalid length 6, expected at most 5 keywords per crate at line 1 column 138"
     }
   ]
 }
diff --git a/src/views/krate_publish.rs b/src/views/krate_publish.rs
index 4c1c94be9eb..7901dbe75e1 100644
--- a/src/views/krate_publish.rs
+++ b/src/views/krate_publish.rs
@@ -18,15 +18,12 @@ pub struct PublishMetadata {
     pub vers: EncodableCrateVersion,
     pub deps: Vec<EncodableCrateDependency>,
     pub features: BTreeMap<EncodableFeatureName, Vec<EncodableFeature>>,
-    pub homepage: Option<String>,
-    pub documentation: Option<String>,
     pub readme: Option<String>,
     pub readme_file: Option<String>,
     #[serde(default)]
     pub keywords: EncodableKeywordList,
     #[serde(default)]
     pub categories: EncodableCategoryList,
-    pub repository: Option<String>,
     #[serde(default)]
     pub links: Option<String>,
 }