Reduce the number of lock and release on origins

src/cache.rs

@@ -42,13 +42,12 @@ impl OriginCacheInner {
     }
 
     pub fn refresh(&self, new_origins: Vec<Origin>) -> Result<(), AppError> {
-        // Create a new HashMap to store the updated origin data
-        let mut map = HashMap::new();
-
         // Iterate over the fetched origins and insert them into the map
-        for origin in new_origins {
-            map.insert(origin.domain.clone(), origin);
-        }
+
+        //*self.origins.write() = map;
+        let map = new_origins.into_iter()
+            .map(|origin| (origin.domain.clone(), origin))
+            .collect();
 
         // Update the cache by acquiring a write lock and replacing the HashMap
         *self.origins.write() = map;
@@ -60,7 +59,11 @@ impl OriginCacheInner {
         let origins = self.origins.read();
 
         // Look up domain in the cache and clone if found
-        let result = origins.get(domain).cloned();
+        let result = {
+            let origins = self.origins.read();
+
+            origins.get(domain).cloned()
+        };
 
         // Mostly for development, but also useful if you want to see how often the cache is hit
         if result.is_some() {