Merge pull request #6 from habibi-dev/develop

✨ refactor(ip): Simplify IP address extraction logic using chained `&&`

Author: habibi-dev

src/services/ip/mod.rs

@@ -10,20 +10,19 @@ pub fn routers_list() -> Vec<(&'static str, MethodRouter)> {
 }
 
 pub fn get_client_ip(headers: &HeaderMap) -> Option<IpAddr> {
-    // 1) X-Forwarded-For → first IP
-    if let Some(val) = headers.get("x-forwarded-for").and_then(|v| v.to_str().ok()) {
-        if let Some(first) = val.split(',').next() {
-            if let Ok(ip) = first.trim().parse::<IpAddr>() {
-                return Some(ip);
-            }
-        }
+    // X-Forwarded-For: take the first IP
+    if let Some(val) = headers.get("x-forwarded-for").and_then(|v| v.to_str().ok())
+        && let Some(first) = val.split(',').next()
+        && let Ok(ip) = first.trim().parse::<IpAddr>()
+    {
+        return Some(ip);
     }
 
-    // 2) X-Real-IP
-    if let Some(val) = headers.get("x-real-ip").and_then(|v| v.to_str().ok()) {
-        if let Ok(ip) = val.trim().parse::<IpAddr>() {
-            return Some(ip);
-        }
+    // X-Real-IP
+    if let Some(val) = headers.get("x-real-ip").and_then(|v| v.to_str().ok())
+        && let Ok(ip) = val.trim().parse::<IpAddr>()
+    {
+        return Some(ip);
     }
 
     None