Skip to content

Commit 9213e8a

Browse files
committed
Revert "Log missing and uploaded files"
This reverts commit 6dc4ec8.
1 parent 6dc4ec8 commit 9213e8a

2 files changed

Lines changed: 3 additions & 14 deletions

File tree

src/api/build.rs

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ 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;
65

76
use chrono::Utc;
87
use futures::StreamExt;
@@ -562,21 +561,11 @@ pub async fn upload(
562561

563562
let mut multipart = multipart;
564563
let mut sizes = Vec::new();
565-
let mut saved_files = Vec::new();
566564
while let Some(field) = multipart.next().await {
567565
let field = field.map_err(|e| ApiError::InternalServerError(e.to_string()))?;
568-
let (subpath, size) = save_file(field, &uploadstate).await?;
569-
saved_files.push(subpath);
570-
sizes.push(size);
566+
sizes.push(save_file(field, &uploadstate).await?);
571567
}
572568

573-
info!(
574-
"Build {}: upload: saved {} files: {:?}",
575-
params.id,
576-
sizes.len(),
577-
saved_files,
578-
);
579-
580569
Ok(HttpResponse::Ok().json(sizes))
581570
}
582571

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<(path::PathBuf, i64), ApiError> {
169+
) -> Result<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((repo_subpath, size))
197+
Ok(size)
198198
}
199199

200200
#[cfg(test)]

0 commit comments

Comments
 (0)