only check file ids when necessary

BillysBigCompany · Aug 18, 2024 · c3586c2 · c3586c2
1 parent e4d9d3a
commit c3586c2
Showing 1 changed file with 11 additions and 13 deletions.
diff --git a/src/auth.rs b/src/auth.rs
@@ -57,19 +57,17 @@ pub fn authorize(
         ))
         .unwrap();
 
-    if !file_ids.is_empty() {
-        let auth_file_ids = set(file_ids
-            .iter()
-            .map(|id| string(id))
-            .collect::<BTreeSet<_>>());
-
-        authorizer
-            .add_fact(biscuit_auth::builder::fact(
-                "file_ids",
-                &[auth_file_ids.clone()],
-            ))
-            .unwrap();
-    }
+    let auth_file_ids = set(file_ids
+        .iter()
+        .map(|id| string(id))
+        .collect::<BTreeSet<_>>());
+
+    authorizer
+        .add_fact(biscuit_auth::builder::fact(
+            "file_ids",
+            &[auth_file_ids.clone()],
+        ))
+        .unwrap();
 
     authorizer.authorize().unwrap();