refactor

src/archive.rs

@@ -202,7 +202,7 @@ impl WebsiteArchive {
 
         let path = parsed_url.path();
 
-        let mut folder_name = self.dir.join(&domain);
+        let mut folder_name = self.dir.join(domain);
 
         if !std::fs::exists(&folder_name).unwrap() {
             download_fav_for(domain).await;
@@ -217,11 +217,11 @@ impl WebsiteArchive {
         std::fs::create_dir_all(&folder_name).unwrap();
 
         let timestamp = chrono::Utc::now().format("%Y-%m-%d").to_string();
-        let filename = folder_name.join(&format!("index_{timestamp}.html"));
+        let filename = folder_name.join(format!("index_{timestamp}.html"));
 
         log::info!("Archiving {url} to {}", filename.to_str().unwrap());
 
-        run_command(&vec![
+        run_command(&[
             "monolith",
             "-I",
             "-o",
@@ -239,7 +239,7 @@ impl WebsiteArchive {
 fn run_command(cmd: &[&str]) {
     let mut cmd_setup = std::process::Command::new(cmd[0]);
     let cmd_setup = cmd_setup
-        .args(cmd.into_iter().skip(1).collect::<Vec<_>>())
+        .args(cmd.iter().skip(1).collect::<Vec<_>>())
         .stdout(std::process::Stdio::inherit())
         .stderr(std::process::Stdio::inherit());
 

src/blacklist.rs

@@ -14,5 +14,5 @@ pub fn check_blacklist(domain: &str) -> bool {
         }
     }
 
-    return false;
+    false
 }

src/favicon.rs

@@ -16,7 +16,7 @@ pub async fn download_favicon(domain: &str) -> Option<Vec<u8>> {
 }
 
 pub async fn download_fav_for(site: &str) {
-    if let Some(fav) = download_favicon(&site).await {
+    if let Some(fav) = download_favicon(site).await {
         std::fs::write(std::path::Path::new("./favicon").join(site), fav).unwrap();
         log::info!("Writting favicon for {site}");
     }

src/pages/mod.rs

@@ -114,11 +114,7 @@ pub async fn render_website(
     let document = arc.get_domain(domain).path(path.to_str().unwrap());
 
     let content = document
-        .render_local(if time.is_some() {
-            Some(time.unwrap().to_string())
-        } else {
-            None
-        })
+        .render_local(time.map(|time| time.to_string()))
         .await;
 
     if let Some(content) = content {