From 60317be1149776ddbae57ad41ca10d7af8cea382 Mon Sep 17 00:00:00 2001 From: Michal Baumgartner Date: Sun, 19 Apr 2020 22:44:41 +0200 Subject: [PATCH] Let Reqwest `text()` convert the Response to a String --- src/req/regional_requester.rs | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/src/req/regional_requester.rs b/src/req/regional_requester.rs index d47231b..6e61e94 100644 --- a/src/req/regional_requester.rs +++ b/src/req/regional_requester.rs @@ -113,15 +113,12 @@ impl RegionalRequester { { log::debug!("Response {} (retried {} times), returning.", status, retries); - // Extract the response body from bytes into a String, - // accounting for potentially non-utf-8 characters. - let content = response.bytes().await; + // Extract the response body as a String + let content = response.text().await; break match content { - Ok(bytes) => { - let body = String::from_utf8_lossy(&bytes).into_owned(); - - Err(RiotApiError::new(err, retries, Some(headers), Some(status), Some(body))) + Ok(str) => { + Err(RiotApiError::new(err, retries, Some(headers), Some(status), Some(str))) } Err(_inner_err) => { // Throw the inner error away and ignore response body parsing