diff --git a/src/archive.rs b/src/archive.rs index aa29525..838b7c4 100644 --- a/src/archive.rs +++ b/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::>()) + .args(cmd.iter().skip(1).collect::>()) .stdout(std::process::Stdio::inherit()) .stderr(std::process::Stdio::inherit()); diff --git a/src/blacklist.rs b/src/blacklist.rs index 9a31b69..67d480b 100644 --- a/src/blacklist.rs +++ b/src/blacklist.rs @@ -14,5 +14,5 @@ pub fn check_blacklist(domain: &str) -> bool { } } - return false; + false } diff --git a/src/favicon.rs b/src/favicon.rs index 01d48ed..1f98b9d 100644 --- a/src/favicon.rs +++ b/src/favicon.rs @@ -16,7 +16,7 @@ pub async fn download_favicon(domain: &str) -> Option> { } 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}"); } diff --git a/src/pages/mod.rs b/src/pages/mod.rs index e0ff696..8ca3ab0 100644 --- a/src/pages/mod.rs +++ b/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 {