amplitude / Amplitude-JavaScript

JavaScript SDK for Amplitude
MIT License
315 stars 133 forks source link

fix: remove events when 200, no need to check response #572

Closed liuyang1520 closed 1 year ago

liuyang1520 commented 1 year ago

Summary

User might use the SDK to send events to their proxy server, we should only rely on the 200 status code, not the response content for removing successfully sent events. Otherwise, the same event might get sent to the ingestion host again and again.

Checklist