berkus
/
actix-web
mirror of https://github.com/fafhrd91/actix-web
1
0
Fork 0
Code Issues Releases Wiki Activity

Should not send body with GET request

This commit is contained in:
Thiago Arrais 2020-11-04 13:33:44 -03:00
parent e7d96288cd
commit d16d1de93f
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
awc/src/connect.rs
View File

@ -109,8 +109,7 @@ where
Box::pin(async move { Box::pin(async move {
let connection = fut.await?; let connection = fut.await?;
// FIXME: whether we'll resend the body depends on the redirect status code let mut reqbody = match body {
let reqbody = match body {
Body::None => Body::None, Body::None => Body::None,
Body::Empty => Body::Empty, Body::Empty => Body::Empty,
Body::Bytes(ref b) => Body::Bytes(b.clone()), Body::Bytes(ref b) => Body::Bytes(b.clone()),
@ -120,7 +119,6 @@ where
}; };
let mut reqhead = RequestHead::default(); let mut reqhead = RequestHead::default();
// FIXME: method depends on redirect code
reqhead.method = head.method.clone(); reqhead.method = head.method.clone();
reqhead.version = head.version.clone(); reqhead.version = head.version.clone();
// FIXME: not all headers should be mirrored on redirect // FIXME: not all headers should be mirrored on redirect
@ -142,6 +140,10 @@ where
if let Ok(location_str) = location_value.to_str(); if let Ok(location_str) = location_value.to_str();
if let Ok(location_uri) = location_str.parse::<Uri>(); if let Ok(location_uri) = location_str.parse::<Uri>();
then { then {
if resphead.status == actix_http::http::StatusCode::SEE_OTHER {
reqhead.method = actix_http::http::Method::GET;
reqbody = Body::None;
}
reqhead.uri = location_uri; reqhead.uri = location_uri;
return deal_with_redirects( return deal_with_redirects(
backend.clone(), backend.clone(),