use Option<&T> instead of &Option<T> for maybe_get_forwarded

loco-rs · Dec 2, 2024 · a3e031f · a3e031f
1 parent 83dae54
commit a3e031f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/controller/middleware/remote_ip.rs b/src/controller/middleware/remote_ip.rs
@@ -341,15 +341,15 @@ mod tests {
         assert_debug_snapshot!(res);
         let res = maybe_get_forwarded(
             &xff("51.50.51.50,192.1.1.1"),
-            Some(&vec![IpNetwork::from_str("192.1.1.1/8").unwrap()]),
+            Some(vec![IpNetwork::from_str("192.1.1.1/8").unwrap()]).as_ref(),
         );
         assert_debug_snapshot!(res);
 
         // we replaced the proxy list, which is why 192.168.1.1 should appear as a valid
         // remote IP and not skipped
         let res = maybe_get_forwarded(
             &xff("51.50.51.50,192.168.1.1"),
-            Some(&vec![IpNetwork::from_str("192.1.1.1/16").unwrap()]),
+            Some(vec![IpNetwork::from_str("192.1.1.1/16").unwrap()]).as_ref(),
         );
         assert_debug_snapshot!(res);
     }