don't compress json twice

Author: syphar

src/docbuilder/rustwide_builder.rs

@@ -923,7 +923,8 @@ impl RustwideBuilder {
             let _span = info_span!("store_json", %format_version).entered();
             let path = rustdoc_json_path(name, version, target, format_version);
 
-            self.storage.store_one(&path, compressed_json.clone())?;
+            self.storage
+                .store_one_uncompressed(&path, compressed_json.clone())?;
             self.storage.set_public_access(&path, true)?;
         }
 

src/storage/mod.rs

@@ -569,6 +569,30 @@ impl AsyncStorage {
         Ok(alg)
     }
 
+    // Store file without compressing, assumes it's already compressed
+    #[instrument(skip(self, content))]
+    pub(crate) async fn store_one_uncompressed(
+        &self,
+        path: impl Into<String> + std::fmt::Debug,
+        content: impl Into<Vec<u8>>,
+    ) -> Result<()> {
+        let path = path.into();
+        let content = content.into();
+        let mime = detect_mime(&path).to_owned();
+
+        self.store_inner(vec![Blob {
+            path,
+            mime,
+            content,
+            compression: None,
+            // this field is ignored by the backend
+            date_updated: Utc::now(),
+        }])
+        .await?;
+
+        Ok(())
+    }
+
     async fn store_inner(&self, batch: Vec<Blob>) -> Result<()> {
         match &self.backend {
             StorageBackend::Database(db) => db.store_batch(batch).await,
@@ -777,6 +801,16 @@ impl Storage {
         self.runtime.block_on(self.inner.store_one(path, content))
     }
 
+    #[instrument(skip(self, content))]
+    pub(crate) async fn store_one_uncompressed(
+        &self,
+        path: impl Into<String> + std::fmt::Debug,
+        content: impl Into<Vec<u8>>,
+    ) -> Result<()> {
+        self.runtime
+            .block_on(self.inner.store_one_uncompressed(path, content))
+    }
+
     /// sync wrapper for the list_prefix function
     /// purely for testing purposes since it collects all files into a Vec.
     #[cfg(test)]