segmentio/analytics.js-integrations

[segmentio] Drop the event when message exceeds the maximum size

gglavan-ellation opened this issue ยท 1 comments

Hi ๐Ÿ‘‹ . Is there any reason why the segmentio integration doesn't drop the event when the message exceeds the maximum size as the analytics-ios and analytics-android SDKs do? Looking through the files it seems like this was actually considered a year ago.

// Print a log statement when messages exceed the maximum size. In the future,
// we may consider dropping this event on the client entirely.

I'm willing to open a PR if that is still considered.

I have opened a PR to change this: #599