use a proper function for this instead of checking 200

Signed-off-by: girlbossceo <june@girlboss.ceo>

src/api/appservice_server.rs

@@ -79,7 +79,7 @@ where
         Vec::new().into()
     }); // TODO: handle timeout
 
-    if status != 200 {
+    if !status.is_success() {
         warn!(
             "Appservice returned bad response {} {}\n{}\n{:?}",
             destination,

src/api/server_server.rs

@@ -267,7 +267,7 @@ where
             }); // TODO: handle timeout
             debug!("Got response bytes from {destination}");
 
-            if status != 200 {
+            if !status.is_success() {
                 warn!(
                     "{} {}: {}",
                     url,
@@ -283,7 +283,7 @@ where
                 .body(body)
                 .expect("reqwest body is valid http body");
 
-            if status == 200 {
+            if status.is_success() {
                 debug!("Parsing response bytes from {destination}");
                 let response = T::IncomingResponse::try_from_http_response(http_response);
                 if response.is_ok() && write_destination_to_cache {

src/service/pusher/mod.rs

@@ -97,7 +97,7 @@ impl Service {
                     Vec::new().into()
                 }); // TODO: handle timeout
 
-                if status != 200 {
+                if !status.is_success() {
                     info!(
                         "Push gateway returned bad response {} {}\n{}\n{:?}",
                         destination,