refactor: add last source error to failed retry of upload / download

It was missing for aggregator file upload and client database v2
download.

Author: Alenar

mithril-aggregator/src/file_uploaders/interface.rs

@@ -53,11 +53,11 @@ pub trait FileUploader: Sync + Send {
             nb_attempts += 1;
             match self.upload_without_retry(filepath).await {
                 Ok(result) => return Ok(result),
-                Err(_) if nb_attempts >= retry_policy.attempts => {
-                    return Err(anyhow::anyhow!(
-                        "Upload failed after {} attempts",
-                        nb_attempts
-                    ));
+                Err(e) if nb_attempts >= retry_policy.attempts => {
+                    return Err(anyhow::anyhow!(e).context(format!(
+                        "Upload failed after {nb_attempts} attempts. Uploaded file path: {}",
+                        filepath.display()
+                    )));
                 }
                 _ => tokio::time::sleep(retry_policy.delay_between_attempts).await,
             }

mithril-client/src/file_downloader/retry.rs

@@ -81,12 +81,10 @@ impl FileDownloader for RetryDownloader {
                 .await
             {
                 Ok(result) => return Ok(result),
-                Err(_) if nb_attempts >= retry_policy.attempts => {
-                    return Err(anyhow::anyhow!(
-                        "Download of location {:?} failed after {} attempts",
-                        location,
-                        nb_attempts
-                    ));
+                Err(e) if nb_attempts >= retry_policy.attempts => {
+                    return Err(anyhow::anyhow!(e).context(format!(
+                        "Download of location {location:?} failed after {nb_attempts} attempts",
+                    )));
                 }
                 _ => tokio::time::sleep(retry_policy.delay_between_attempts).await,
             }