improve account lookup

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## v0.2.4
+- make account lookup try harder
+
 ## v0.2.3
 - `/add user` will now try to follow even if already a member
 

Cargo.lock

@@ -328,7 +328,7 @@ checksum = "e88a8acf291dafb59c2d96e8f59828f3838bb1a70398823ade51a84de6a6deed"
 
 [[package]]
 name = "fedigroups"
-version = "0.2.3"
+version = "0.2.4"
 dependencies = [
  "anyhow",
  "clap",

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "fedigroups"
-version = "0.2.3"
+version = "0.2.4"
 authors = ["Ondřej Hruška <ondra@ondrovo.com>"]
 edition = "2018"
 publish = false

src/group_handler/handle_mention.rs

@@ -46,24 +46,21 @@ impl<'a> ProcessMention<'a> {
                 Err(e.into())
             }
             Ok(Ok(res)) => {
-                debug!("Result: {:#?}", res);
-
-                if let Some(item) = res.accounts.into_iter().next() {
+                for item in res.accounts {
                     let acct_normalized = normalize_acct(&item.acct, &self.group_acct)?;
                     if acct_normalized == acct {
                         debug!("Search done, account found: {}", item.acct);
-                        Ok(Some(item.id))
+                        return Ok(Some(item.id))
                     } else {
-                        warn!("Search done but found wrong account: {}", item.acct);
-                        Ok(None)
+                        warn!("Found wrong account: {}", item.acct);
                     }
-                } else {
+                }
+
                 debug!("Search done, nothing found");
                 Ok(None)
             }
         }
     }
-    }
 
     fn append_admin_list_to_reply(&mut self) {
         let mut admins = self.config.get_admins().collect::<Vec<_>>();