Skip to content

Commit b30c0ab

Browse files
committed
Reduce verbosity of logs
Signed-off-by: Denys Fedoryshchenko <denys.f@collabora.com>
1 parent 2c8fdf7 commit b30c0ab

2 files changed

Lines changed: 36 additions & 13 deletions

File tree

src/azure.rs

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ fn save_headers_to_file(filename: String, headers: HeaderMap) {
171171
/// Get file from Azure blob storage
172172
async fn get_file_from_blob(filename: String) -> ReceivedFile {
173173
let azure_cfg = Arc::new(get_azure_credentials("azure"));
174-
println!("get_file_from_blob {}", filename);
174+
//println!("get_file_from_blob {}", filename);
175175
let storage_account = azure_cfg.account.as_str();
176176
let storage_key = azure_cfg.key.clone();
177177
let storage_container = azure_cfg.container.as_str();
@@ -192,7 +192,7 @@ async fn get_file_from_blob(filename: String) -> ReceivedFile {
192192

193193
match blob_url_res {
194194
Ok(url) => {
195-
println!("Blob URL: {}", url);
195+
//println!("Blob URL: {}", url);
196196
blob_url = url.to_string();
197197
}
198198
Err(e) => {
@@ -208,11 +208,11 @@ async fn get_file_from_blob(filename: String) -> ReceivedFile {
208208
let cache_filename_headers = format!("cache/{}.headers", digest.to_hex_lowercase());
209209
// check if cache file exists
210210
if std::path::Path::new(&cache_filename).exists() {
211-
println!("Cache file {} exists", cache_filename);
211+
//println!("Cache file {} exists", cache_filename);
212212
// is cached file non-zero length?
213213
let metadata = std::fs::metadata(&cache_filename).unwrap();
214214
if metadata.len() > 0 {
215-
println!("Cache file {} is non-zero length", cache_filename);
215+
//println!("Cache file {} is non-zero length", cache_filename);
216216
received_file.cached_filename = cache_filename;
217217
received_file.headers = get_headers_from_file(cache_filename_headers);
218218
received_file.valid = true;
@@ -240,10 +240,12 @@ async fn get_file_from_blob(filename: String) -> ReceivedFile {
240240
}
241241
}
242242
}
243+
/*
243244
println!(
244245
"Downloading blob to cache file {} from {}",
245246
cache_filename, blob_url
246247
);
248+
*/
247249
let client = Client::new();
248250
let response = client.get(blob_url).send().await;
249251
match response {

src/main.rs

Lines changed: 30 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -135,10 +135,7 @@ async fn main() {
135135
.route("/v1/file", post(ax_post_file))
136136
.route("/upload", post(ax_post_file))
137137
.route("/*filepath", get(ax_get_file))
138-
.layer(
139-
ServiceBuilder::new()
140-
.layer(DefaultBodyLimit::max(1024 * 1024 * 1024 * 4)),
141-
);
138+
.layer(ServiceBuilder::new().layer(DefaultBodyLimit::max(1024 * 1024 * 1024 * 4)));
142139

143140
/*
144141
.layer(SecureClientIpSource::ConnectInfo.into_extension())
@@ -292,16 +289,18 @@ async fn ax_get_file(
292289
) -> impl IntoResponse {
293290
let timestamp = std::time::SystemTime::now();
294291
let human_time = chrono::DateTime::<chrono::Utc>::from(timestamp);
295-
// or none
296292
let user_agent = rxheaders.get("User-Agent");
297293
let user_agent_str = match user_agent {
298294
Some(user_agent) => user_agent.to_str().unwrap(),
299295
None => "",
300296
};
301-
// remote_addr human_time method filepath
302-
println!("{:?} {} {} {} {}", remote_addr, human_time, method, filepath, user_agent_str);
297+
303298
let received_file = driver_get_file(filepath.clone());
304299
if !received_file.valid {
300+
println!(
301+
"{:?} 404 0 {} {} {} {}",
302+
remote_addr, human_time, method, filepath, user_agent_str
303+
);
305304
return (StatusCode::NOT_FOUND, format!("Not Found: {}", filepath)).into_response();
306305
}
307306
let cached_file = received_file.cached_filename;
@@ -325,7 +324,11 @@ async fn ax_get_file(
325324

326325
/* Usually HEAD is used to check if the file exists and range is supported */
327326
if method == axum::http::Method::HEAD {
328-
println!("HEAD request, returning headers only");
327+
//println!("HEAD request, returning headers only");
328+
println!(
329+
"{:?} 200 0 {} {} {} {}",
330+
remote_addr, human_time, method, filepath, user_agent_str
331+
);
329332
return (headers, Body::empty()).into_response();
330333
}
331334
match tokio::fs::File::open(&cached_file).await {
@@ -370,14 +373,32 @@ async fn ax_get_file(
370373
let stream = ReaderStream::new(file);
371374
let axbody = Body::from_stream(stream);
372375

373-
println!("Headers: {:?}", headers);
376+
//println!("Headers: {:?}", headers);
374377
if start != 0 {
378+
let body_size = end - start;
379+
println!(
380+
"{:?} 206 {} {} {} {} {}",
381+
remote_addr, body_size, human_time, method, filepath, user_agent_str
382+
);
375383
return (StatusCode::PARTIAL_CONTENT, headers, axbody).into_response();
376384
}
385+
println!(
386+
"{:?} 200 {} {} {} {} {}",
387+
remote_addr,
388+
metadata.len(),
389+
human_time,
390+
method,
391+
filepath,
392+
user_agent_str
393+
);
377394
return (StatusCode::OK, headers, axbody).into_response();
378395
}
379396
Err(_) => {
380397
println!("Error opening file in ax_get_file");
398+
println!(
399+
"{:?} 404 0 {} {} {} {}",
400+
remote_addr, human_time, method, filepath, user_agent_str
401+
);
381402
return (StatusCode::NOT_FOUND, headers, Body::empty()).into_response();
382403
}
383404
};

0 commit comments

Comments
 (0)