From e1272ebf774397bb211af831ba041fd51d8fd0fb Mon Sep 17 00:00:00 2001 From: Johannes Naylor Date: Sun, 18 Aug 2024 12:51:45 +0200 Subject: [PATCH] lint --- src/main.rs | 2 +- src/output.rs | 4 ++-- src/sherlock.rs | 12 +++++------- 3 files changed, 8 insertions(+), 10 deletions(-) diff --git a/src/main.rs b/src/main.rs index d093a86..7416340 100644 --- a/src/main.rs +++ b/src/main.rs @@ -108,7 +108,7 @@ async fn main() -> Result<()> { true => targets, false => targets .into_iter() - .filter(|(site, _)| cli.site_list.contains(&site)) + .filter(|(site, _)| cli.site_list.contains(site)) .collect(), }; diff --git a/src/output.rs b/src/output.rs index e3346e6..3c657f9 100644 --- a/src/output.rs +++ b/src/output.rs @@ -41,7 +41,7 @@ pub fn save_results( if let Some(output_folder) = output_folder { // make sure the output folder exists - std::fs::create_dir_all(&output_folder)?; + std::fs::create_dir_all(output_folder)?; } let output_file = match (output_file, output_folder) { @@ -60,7 +60,7 @@ pub fn save_results( writeln!(file, "Total Websites Username Detected On: {}", total_hits)?; if csv { - write_csv(&username, &results, output_folder, print_all, print_found)?; + write_csv(username, &results, output_folder, print_all, print_found)?; } if xlsx { diff --git a/src/sherlock.rs b/src/sherlock.rs index 5f2a384..361eb6a 100644 --- a/src/sherlock.rs +++ b/src/sherlock.rs @@ -37,7 +37,7 @@ pub struct RequestResult { } pub async fn check_username( - username: &String, + username: &str, site_data: HashMap, timeout: u64, proxy: Option<&String>, @@ -52,11 +52,11 @@ pub async fn check_username( // ping sites for username matches for (site, info) in site_data.into_iter() { add_result_to_channel( - username.clone(), + username.to_owned(), site, info, timeout, - proxy.map(|s| s.clone()), + proxy.cloned(), tx.clone(), )?; } @@ -129,10 +129,8 @@ pub async fn check_username( if error_codes.contains(&status_code) { status = QueryStatus::Available; } - } else { - if &status_code > &399 || &status_code < &200 { - status = QueryStatus::Available; - } + } else if !(200..=399).contains(&status_code) { + status = QueryStatus::Available; } status