fix: Use concurrentMap instead of for loop

Infomaniak · Jan 9, 2025 · 9538468 · 9538468
1 parent 683504b
commit 9538468
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/MailCore/API/MailApiFetcher/MailApiFetcher+Common.swift b/MailCore/API/MailApiFetcher/MailApiFetcher+Common.swift
@@ -20,6 +20,7 @@ import Alamofire
 import Foundation
 import InfomaniakCore
 import InfomaniakLogin
+import InfomaniakConcurrency
 
 /// implementing `MailApiCommonFetchable`
 public extension MailApiFetcher {
@@ -86,10 +87,9 @@ public extension MailApiFetcher {
     }
 
     func download(messages: [Message]) async throws -> [URL] {
-        var downloadedMessageURLs: [URL] = []
         let temporaryDirectory = FileManager.default.temporaryDirectory
 
-        for message in messages {
+        return try await messages.concurrentMap { message in
             let directoryURL = temporaryDirectory.appendingPathComponent(message.uid, isDirectory: true)
             let destination: DownloadRequest.Destination = { _, response in
                 (
@@ -98,11 +98,10 @@ public extension MailApiFetcher {
                 )
             }
 
-            let download = authenticatedSession.download(Endpoint.resource(message.downloadResource).url, to: destination)
+            let download = self.authenticatedSession.download(Endpoint.resource(message.downloadResource).url, to: destination)
             let messageUrl = try await download.serializingDownloadedFileURL().value
-            downloadedMessageURLs.append(messageUrl)
+            return messageUrl
         }
-        return downloadedMessageURLs
     }
 
     func quotas(mailbox: Mailbox) async throws -> Quotas {