From bc461d9baa13911343a601c2dbe1791cfa01cc4d Mon Sep 17 00:00:00 2001
From: BlackDex <black.dex@gmail.com>
Date: Thu, 31 Jan 2019 17:58:03 +0100
Subject: [PATCH] Some small changes on the iter of the cookies

---
 src/api/icons.rs | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/api/icons.rs b/src/api/icons.rs
index 8ae3c83f..4fda717b 100644
--- a/src/api/icons.rs
+++ b/src/api/icons.rs
@@ -167,10 +167,10 @@ fn get_icon_url(domain: &str) -> Result<(String, String), Error> {
         // Extract the URL from the respose in case redirects occured (like @ gitlab.com)
         let url = content.url().clone();
         let raw_cookies = content.headers().get_all("set-cookie");
-        raw_cookies.iter().for_each(|raw_cookie| {
+        cookie_str = raw_cookies.iter().map(|raw_cookie| {
             let cookie = Cookie::parse(raw_cookie.to_str().unwrap_or_default()).unwrap();
-            cookie_str.push_str(&format!("{}={}; ", cookie.name(), cookie.value()));
-        });
+            format!("{}={}; ", cookie.name(), cookie.value())
+        }).collect::<String>();
 
         // Add the default favicon.ico to the list with the domain the content responded from.
         iconlist.push(IconList { priority: 35, href: url.join("/favicon.ico").unwrap().into_string() });