fix: do not create api client on every lookup (#230)

phnx-im · Dec 6, 2024 · 56df9ef · 56df9ef
1 parent 0c27503
commit 56df9ef
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/coreclient/src/clients/api_clients.rs b/coreclient/src/clients/api_clients.rs
@@ -2,6 +2,7 @@
 //
 // SPDX-License-Identifier: AGPL-3.0-or-later
 
+use std::collections::hash_map::Entry;
 use std::{collections::HashMap, sync::Mutex};
 
 use phnxtypes::identifiers::Fqdn;
@@ -40,10 +41,14 @@ impl ApiClients {
             .clients
             .lock()
             .map_err(|_| ApiClientsError::MutexPoisonError)?;
-        let client = clients
-            .entry(lookup_domain.clone())
-            .or_insert(ApiClient::initialize(lookup_domain)?);
-        Ok(client.clone())
+        match clients.entry(lookup_domain.clone()) {
+            Entry::Occupied(entry) => Ok(entry.get().clone()),
+            Entry::Vacant(entry) => {
+                let client = ApiClient::initialize(lookup_domain.clone())?;
+                entry.insert(client.clone());
+                Ok(client)
+            }
+        }
     }
 
     pub(super) fn default_client(&self) -> Result<ApiClient, ApiClientsError> {