From 3125db29444a835b8fb1b2b66c2bce7a49e9d19e Mon Sep 17 00:00:00 2001 From: Omid Rad Date: Mon, 12 Jul 2021 14:36:54 +0200 Subject: [PATCH] cargo fmt --- awc/src/middleware/redirect.rs | 53 ++++++++++++++++------------------ 1 file changed, 25 insertions(+), 28 deletions(-) diff --git a/awc/src/middleware/redirect.rs b/awc/src/middleware/redirect.rs index f82e63e49..a6b7c0eb3 100644 --- a/awc/src/middleware/redirect.rs +++ b/awc/src/middleware/redirect.rs @@ -158,7 +158,9 @@ where connector, } => match ready!(fut.poll(cx))? { ConnectResponse::Client(res) => match res.head().status { - status @ (StatusCode::MOVED_PERMANENTLY + status + @ + (StatusCode::MOVED_PERMANENTLY | StatusCode::FOUND | StatusCode::SEE_OTHER | StatusCode::TEMPORARY_REDIRECT @@ -166,9 +168,11 @@ where if *max_redirect_times > 0 => { let is_redirect = match status { - StatusCode::TEMPORARY_REDIRECT | StatusCode::PERMANENT_REDIRECT => true, - _ => false - }; + StatusCode::TEMPORARY_REDIRECT | StatusCode::PERMANENT_REDIRECT => { + true + } + _ => false, + }; let prev_uri = uri.take().unwrap(); @@ -273,7 +277,8 @@ fn build_next_uri(res: &ClientResponse, prev_uri: &Uri) -> Result HttpResponse { - if req.headers() - .get(header::AUTHORIZATION) - .is_none() - { + if req.headers().get(header::AUTHORIZATION).is_none() { HttpResponse::Ok().finish() } else { HttpResponse::InternalServerError().finish() } } - App::new() - .service(web::resource("/").route(web::to(root))) + App::new().service(web::resource("/").route(web::to(root))) }); let srv2_port: u16 = srv2.addr().port(); let srv1 = actix_test::start(move || { async fn root(req: HttpRequest) -> HttpResponse { let port = *req.app_data::().unwrap(); - if req - .headers() - .get(header::AUTHORIZATION) - .is_some() - { + if req.headers().get(header::AUTHORIZATION).is_some() { HttpResponse::Found() - .append_header(("location", format!("http://localhost:{}/", port).as_str())) + .append_header(( + "location", + format!("http://localhost:{}/", port).as_str(), + )) .finish() } else { HttpResponse::InternalServerError().finish() @@ -443,11 +443,7 @@ mod tests { } async fn test1(req: HttpRequest) -> HttpResponse { - if req - .headers() - .get(header::AUTHORIZATION) - .is_some() - { + if req.headers().get(header::AUTHORIZATION).is_some() { HttpResponse::Found() .append_header(("location", "/test2")) .finish() @@ -457,10 +453,7 @@ mod tests { } async fn test2(req: HttpRequest) -> HttpResponse { - if req.headers() - .get(header::AUTHORIZATION) - .is_some() - { + if req.headers().get(header::AUTHORIZATION).is_some() { HttpResponse::Ok().finish() } else { HttpResponse::InternalServerError().finish() @@ -475,12 +468,16 @@ mod tests { }); // send a request to different origins, http://srv1/ then http://srv2/. So it should remove the header - let client = ClientBuilder::new().header(header::AUTHORIZATION, "auth_key_value").finish(); + let client = ClientBuilder::new() + .header(header::AUTHORIZATION, "auth_key_value") + .finish(); let res = client.get(srv1.url("/")).send().await.unwrap(); assert_eq!(res.status().as_u16(), 200); // send a request to same origin, http://srv1/test1 then http://srv1/test2. So it should NOT remove any header - let client = ClientBuilder::new().header(header::AUTHORIZATION, "auth_key_value").finish(); + let client = ClientBuilder::new() + .header(header::AUTHORIZATION, "auth_key_value") + .finish(); let res = client.get(srv1.url("/test1")).send().await.unwrap(); assert_eq!(res.status().as_u16(), 200); }