Handle unsuccessful responses before decoding

Instead of assuming every request is successful, check that the class of
the status is successful (2xx), and if not, return a
`ClientError::InvalidRequest`, which is a renamed
`ClientError::UnexpectedStatusCode`.

1 files changed, 10 insertions, 4 deletions

diff --git a/src/client/rest/mod.rs b/src/client/rest/mod.rs
index b905921..253f0f9 100644
--- a/src/client/rest/mod.rs
+++ b/src/client/rest/mod.rs
@@ -33,7 +33,7 @@ use hyper::client::{
     Request,
 };
 use hyper::method::Method;
-use hyper::status::StatusCode;
+use hyper::status::{StatusClass, StatusCode};
 use hyper::{Error as HyperError, Result as HyperResult, Url, header};
 use multipart::client::Multipart;
 use self::ratelimiting::Route;
@@ -1584,9 +1584,15 @@ pub fn unpin_message(channel_id: u64, message_id: u64) -> Result<()> {
 
 fn request<'a, F>(route: Route, f: F) -> Result<HyperResponse>
     where F: Fn() -> RequestBuilder<'a> {
-    ratelimiting::perform(route, || f()
+    let response = ratelimiting::perform(route, || f()
         .header(header::Authorization(TOKEN.lock().unwrap().clone()))
-        .header(header::ContentType::json()))
+        .header(header::ContentType::json()))?;
+
+    if response.status.class() == StatusClass::Success {
+        Ok(response)
+    } else {
+        Err(Error::Client(ClientError::InvalidRequest(response.status)))
+    }
 }
 
 #[doc(hidden)]
@@ -1626,7 +1632,7 @@ fn verify(expected_status_code: u16, mut response: HyperResponse) -> Result<()>
 
     debug!("Content: {}", s);
 
-    Err(Error::Client(ClientError::UnexpectedStatusCode(response.status)))
+    Err(Error::Client(ClientError::InvalidRequest(response.status)))
 }
 
 /// Representation of the method of a query to send for the [`get_guilds`]