Skip to content

Commit 6dc4ec8

Browse files
committed
Log missing and uploaded files
1 parent c9af553 commit 6dc4ec8

2 files changed

Lines changed: 14 additions & 3 deletions

File tree

src/api/build.rs

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ use actix::prelude::*;
22
use actix_multipart::Multipart;
33
use actix_web::web::{self, Data, Json, Path, Query};
44
use actix_web::{HttpRequest, HttpResponse, ResponseError, Result};
5+
use log::info;
56

67
use chrono::Utc;
78
use futures::StreamExt;
@@ -561,11 +562,21 @@ pub async fn upload(
561562

562563
let mut multipart = multipart;
563564
let mut sizes = Vec::new();
565+
let mut saved_files = Vec::new();
564566
while let Some(field) = multipart.next().await {
565567
let field = field.map_err(|e| ApiError::InternalServerError(e.to_string()))?;
566-
sizes.push(save_file(field, &uploadstate).await?);
568+
let (subpath, size) = save_file(field, &uploadstate).await?;
569+
saved_files.push(subpath);
570+
sizes.push(size);
567571
}
568572

573+
info!(
574+
"Build {}: upload: saved {} files: {:?}",
575+
params.id,
576+
sizes.len(),
577+
saved_files,
578+
);
579+
569580
Ok(HttpResponse::Ok().json(sizes))
570581
}
571582

src/api/utils.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ pub fn start_save(
166166
pub async fn save_file(
167167
mut field: actix_multipart::Field,
168168
state: &Arc<UploadState>,
169-
) -> Result<i64, ApiError> {
169+
) -> Result<(path::PathBuf, i64), ApiError> {
170170
let repo_subpath = get_upload_subpath(&field, state)?;
171171

172172
let (mut named_file, object_file) = start_save(&repo_subpath, state)
@@ -194,7 +194,7 @@ pub async fn save_file(
194194
warn!("Can't get permissions on uploaded file");
195195
};
196196

197-
Ok(size)
197+
Ok((repo_subpath, size))
198198
}
199199

200200
#[cfg(test)]

0 commit comments

Comments
 (0)