refactor: replace bare .unwrap() with documented .expect() in hot paths

Author: Algorithm5838

src/client.rs

@@ -73,8 +73,8 @@ pub fn build_client() -> WreqClient {
 	let emulation_os = [EmulationOS::Android, EmulationOS::Windows, EmulationOS::Linux];
 
 	let emulation = EmulationOption::builder()
-		.emulation(*fastrand::choice(emulations.iter()).unwrap())
-		.emulation_os(*fastrand::choice(emulation_os.iter()).unwrap())
+		.emulation(*fastrand::choice(emulations.iter()).expect("emulations is a non-empty array literal"))
+		.emulation_os(*fastrand::choice(emulation_os.iter()).expect("emulation_os is a non-empty array literal"))
 		.build()
 		.emulation();
 

src/main.rs

@@ -185,8 +185,8 @@ async fn main() {
 		}
 	}
 
-	let address = matches.get_one::<String>("address").unwrap();
-	let port = matches.get_one::<String>("port").unwrap();
+	let address = matches.get_one::<String>("address").expect("'address' has a default_value and is always present");
+	let port = matches.get_one::<String>("port").expect("'port' has a default_value and is always present");
 	let hsts = matches.get_one("hsts").map(|m: &String| m.as_str());
 
 	let ipv4_only = std::env::var("IPV4_ONLY").is_ok() || matches.get_flag("ipv4-only");

src/server.rs

@@ -604,8 +604,8 @@ async fn compress_response(req_headers: &HeaderMap<header::HeaderValue>, res: &m
 	let body_bytes: Vec<u8> = {
 		// Swap body with empty, collect old body
 		let old_body = std::mem::replace(res.body_mut(), empty());
-		// Body error type is Infallible, so unwrap is safe
-		old_body.collect().await.unwrap().to_bytes().to_vec()
+		// Body error type is Infallible, so collect cannot fail
+		old_body.collect().await.expect("Body<Infallible> collect cannot fail").to_bytes().to_vec()
 	};
 
 	// Don't bother compressing tiny responses