account for process_events errors in capture_events_dropped_total and…

… log them
PostHog · Nov 21, 2023 · 9cd2deb · 9cd2deb
1 parent 6cee0a5
commit 9cd2deb
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/capture/src/capture.rs b/capture/src/capture.rs
@@ -143,7 +143,11 @@ pub async fn event(
 
     tracing::debug!(context=?context, events=?events, "decoded request");
 
-    process_events(state.sink.clone(), &events, &context).await?;
+    if let Err(err) = process_events(state.sink.clone(), &events, &context).await {
+        report_dropped_events("process_events_error", events.len() as u64);
+        tracing::log::warn!("rejected invalid payload: {}", err);
+        return Err(err)
+    }
 
     Ok(Json(CaptureResponse {
         status: CaptureResponseCode::Ok,