Merge pull request #456 from answerbook/mdeltito/LOG-19799

fix(http): handle and report partial failures with 207 status
mezmo · May 1, 2024 · 0d680df · 0d680df
2 parents 0c91263 + 744386a
commit 0d680df
Showing 1 changed file with 14 additions and 4 deletions.
diff --git a/src/sinks/util/http.rs b/src/sinks/util/http.rs
@@ -787,16 +787,26 @@ where
 
 impl UserLoggingResponse for HttpResponse {
     fn log_msg(&self) -> Option<Value> {
-        if !self.http_response.status().is_success() {
+        let status = self.http_response.status();
+        if status.is_success() {
+            match status {
+                StatusCode::MULTI_STATUS => Some(
+                    format!(
+                        "Partial error returned from destination with status code: {}",
+                        status
+                    )
+                    .into(),
+                ),
+                _ => None,
+            }
+        } else {
             Some(
                 format!(
                     "Error returned from destination with status code: {}",
-                    self.http_response.status()
+                    status
                 )
                 .into(),
             )
-        } else {
-            None
         }
     }