getsentry / relay

Sentry event forwarding and ingestion service.
https://docs.sentry.io/product/relay/
Other
322 stars 91 forks source link

Document max size for Unreal Engine crash reports #4208

Open bruno-garcia opened 2 hours ago

bruno-garcia commented 2 hours ago

We document max payload size for different types of things here: https://develop.sentry.dev/sdk/data-model/envelopes/#size-limits

But there's no mention of the max size for the /unreal endpoint.

It came up on Discord that UE crash reports are getting dropped due to size limit. But it's not clear what that limit is and it's challenging for customers to even realize that they are losing crashes, or what part of the dump was the reason.

Since UE crash dumps have a minidump + context data + configuration + Logs. If we could apply the limits over each part indepdendently, possibly resulting in dropped logs only, instead of dropped minidump. That would improve things significanantly.

narthur157 commented 2 hours ago

Image You can see that issues are being dropped, but it is hard to tell what the solution is without knowing which limit was exceeded, given that are multiple limits involved

Eg it would be very useful for this to say Too Large: Event or Too Large: Attachment so we know where to look

Additionally it would be useful to be able to alert issues being dropped due to size, or maybe other conditions as well. It puts us into a situation of wanting some other error reporting solution for our error reporting.