forked/reqwest-impersonate
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

log invalid location header value on error

Browse Source
This commit is contained in:
Sean McArthur
2018-09-21 14:22:41 -07:00
parent 5bf94e7bdd
commit cd7dbe226c
1 changed files with 11 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/async_impl/client.rs
View File

@@ -476,9 +476,17 @@ impl Future for PendingRequest {
if should_redirect { if should_redirect {
let loc = res.headers() let loc = res.headers()
.get(LOCATION) .get(LOCATION)
.and_then(|val| val.to_str().ok()) .and_then(|val| {
.map(|loc| self.url.join(loc)); let loc = (|| -> Option<Url> {
if let Some(Ok(loc)) = loc { self.url.join(val.to_str().ok()?).ok()
})();
if loc.is_none() {
debug!("Location header had invalid URI: {:?}", val);
}
loc
});
if let Some(loc) = loc {
if self.client.referer { if self.client.referer {
if let Some(referer) = make_referer(&loc, &self.url) { if let Some(referer) = make_referer(&loc, &self.url) {
self.headers.insert(REFERER, referer); self.headers.insert(REFERER, referer);
@@ -522,8 +530,6 @@ impl Future for PendingRequest {
return Err(::error::too_many_redirects(self.url.clone())); return Err(::error::too_many_redirects(self.url.clone()));
} }
} }
} else if let Some(Err(e)) = loc {
debug!("Location header had invalid URI: {:?}", e);
} }
} }
let res = Response::new(res, self.url.clone(), self.client.gzip); let res = Response::new(res, self.url.clone(), self.client.gzip);