refactor(bot): use match expression for router

BastiDood · Apr 10, 2023 · 4a10529 · 4a10529
1 parent 7c3752c
commit 4a10529
Showing 1 changed file with 22 additions and 13 deletions.
diff --git a/crates/api/src/lib.rs b/crates/api/src/lib.rs
@@ -24,19 +24,28 @@ impl App {
         let (Parts { uri, method, headers, .. }, body) = req.into_parts();
         let path = uri.path();
 
-        if method == Method::GET && path == "/healthz" {
-            log::info!("Health check pinged");
-            return Ok(Response::new(Body::empty()));
-        }
-
-        if method != Method::POST {
-            log::error!("Unexpected `{method}` request received");
-            return Err(StatusCode::METHOD_NOT_ALLOWED);
-        }
-
-        if path != "/discord" {
-            log::error!("Unexpected `POST {path}` request received");
-            return Err(StatusCode::NOT_FOUND);
+        match method {
+            Method::GET | Method::HEAD => match path {
+                "/health" => {
+                    log::info!("Health check pinged");
+                    return Ok(Response::new(Body::empty()));
+                }
+                _ => {
+                    log::error!("Unexpected `{method} {path}` request received");
+                    return Err(StatusCode::NOT_FOUND);
+                }
+            }
+            Method::POST => match path {
+                "/discord" => (),
+                _ => {
+                    log::error!("Unexpected `POST {path}` request received");
+                    return Err(StatusCode::NOT_FOUND);
+                }
+            }
+            _ => {
+                log::error!("Unexpected `{method} {path}` request received");
+                return Err(StatusCode::METHOD_NOT_ALLOWED);
+            }
         }
 
         // Retrieve security headers