Skip to content

Commit 31dfb2d

Browse files
authored
fix(api): dont 500 on cross dc response (#4449)
Fixes RVT-6105 # Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Type of change - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] This change requires a documentation update ## How Has This Been Tested? Please describe the tests that you ran to verify your changes. ## Checklist: - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes
1 parent 0ed764e commit 31dfb2d

1 file changed

Lines changed: 1 addition & 3 deletions

File tree

  • engine/packages/api-util/src

engine/packages/api-util/src/lib.rs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -182,9 +182,7 @@ pub async fn reqwest_to_axum_response(reqwest_response: reqwest::Response) -> Re
182182

183183
if !status.is_success() {
184184
let body_text = String::from_utf8_lossy(&body_bytes);
185-
anyhow::bail!(
186-
"remote dc returned error (status: {status}, ray_id: {ray_id:?}, body: {body_text})"
187-
);
185+
tracing::error!(?status, ?ray_id, %body_text, "remote dc returned error");
188186
}
189187

190188
let mut response = Response::builder()

0 commit comments

Comments
 (0)