Simplify AllowOrigin::list

tower-rs · Jan 27, 2024 · 2d059c6 · 2d059c6
1 parent 6f992ce
commit 2d059c6
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/tower-http/src/cors/allow_origin.rs b/tower-http/src/cors/allow_origin.rs
@@ -51,11 +51,14 @@ impl AllowOrigin {
         I: IntoIterator<Item = HeaderValue>,
     {
         let origins = origins.into_iter().collect::<Vec<_>>();
-        if origins.iter().any(|o| o == WILDCARD) {
-            panic!("Wildcard origin (`*`) cannot be passed to `AllowOrigin::list`. Use `AllowOrigin::any()` instead");
-        } else {
-            Self(OriginInner::List(origins))
+        if origins.contains(&WILDCARD) {
+            panic!(
+                "Wildcard origin (`*`) cannot be passed to `AllowOrigin::list`. \
+                 Use `AllowOrigin::any()` instead"
+            );
         }
+
+        Self(OriginInner::List(origins))
     }
 
     /// Set the allowed origins from a predicate