diff options
| author | Rutger Broekhoff | 2024-01-24 18:14:05 +0100 |
|---|---|---|
| committer | Rutger Broekhoff | 2024-01-24 18:14:05 +0100 |
| commit | 09f6e35326d15f9d415b052118228a13fa7421ae (patch) | |
| tree | 70607cc9dbe614a7e56c31abd8cce120b94ee552 /rs/server | |
| parent | 205bb98cb8a61dd2bc2fc154e3e81c6df0d649a2 (diff) | |
| download | gitolfs3-09f6e35326d15f9d415b052118228a13fa7421ae.tar.gz gitolfs3-09f6e35326d15f9d415b052118228a13fa7421ae.zip | |
Log unexpected S3 operation failure
Diffstat (limited to 'rs/server')
| -rw-r--r-- | rs/server/src/main.rs | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/rs/server/src/main.rs b/rs/server/src/main.rs index 68ef3b0..0a37155 100644 --- a/rs/server/src/main.rs +++ b/rs/server/src/main.rs | |||
| @@ -468,7 +468,8 @@ async fn handle_upload_object( | |||
| 468 | } | 468 | } |
| 469 | } | 469 | } |
| 470 | Err(SdkError::ServiceError(e)) if e.err().is_not_found() => {} | 470 | Err(SdkError::ServiceError(e)) if e.err().is_not_found() => {} |
| 471 | _ => { | 471 | Err(e) => { |
| 472 | println!("Failed to HeadObject (repo {repo}, OID {}): {e}", obj.oid); | ||
| 472 | return Some(BatchResponseObject::error( | 473 | return Some(BatchResponseObject::error( |
| 473 | obj, | 474 | obj, |
| 474 | StatusCode::INTERNAL_SERVER_ERROR, | 475 | StatusCode::INTERNAL_SERVER_ERROR, |
| @@ -539,12 +540,13 @@ async fn handle_download_object( | |||
| 539 | .await | 540 | .await |
| 540 | { | 541 | { |
| 541 | Ok(result) => result, | 542 | Ok(result) => result, |
| 542 | Err(_) => { | 543 | Err(e) => { |
| 544 | println!("Failed to HeadObject (repo {repo}, OID {}): {e}", obj.oid); | ||
| 543 | return BatchResponseObject::error( | 545 | return BatchResponseObject::error( |
| 544 | obj, | 546 | obj, |
| 545 | StatusCode::INTERNAL_SERVER_ERROR, | 547 | StatusCode::INTERNAL_SERVER_ERROR, |
| 546 | "Failed to query object information".to_string(), | 548 | "Failed to query object information".to_string(), |
| 547 | ) | 549 | ); |
| 548 | } | 550 | } |
| 549 | }; | 551 | }; |
| 550 | 552 | ||
| @@ -908,7 +910,8 @@ async fn obj_download( | |||
| 908 | .await | 910 | .await |
| 909 | { | 911 | { |
| 910 | Ok(result) => result, | 912 | Ok(result) => result, |
| 911 | Err(_) => { | 913 | Err(e) => { |
| 914 | println!("Failed to GetObject (repo {repo}, OID {oid}): {e}"); | ||
| 912 | return ( | 915 | return ( |
| 913 | StatusCode::INTERNAL_SERVER_ERROR, | 916 | StatusCode::INTERNAL_SERVER_ERROR, |
| 914 | "Failed to query object information", | 917 | "Failed to query object information", |