Breadcrumb Duplication for Multi-Transaction Applications about sentry-native HOT 5 CLOSED

But it would make sense to only add breadcrumbs to transactions where the timestamp is within the transaction boundaries. Would this solve your current issue? I still have to ensure that this correlates with the expected behavior of the API or, rather, the implementations in other SDKs.

@supervacuus This makes sense and I agree that adding breadcrumbs only to transactions within the transaction boundaries would indeed resolve the current issue.

from sentry-native.

@tiny-dancer there is a limit which can be configured via max_breadcrumbs and defaults to 100 (docs). We use a ring buffer implementation to ensure this limit. If you can reproduce an issue with the SDK exceeding this limit, please open up a separate issue 🙏

from sentry-native.

Hi @jerin-omcare, thanks for the report. I can understand why this seems illogical. At this point, the Native SDK treats transactions like any other event when applying the scope before capturing. The only difference is that we don't send a stack trace and module list with each transaction.

But it would make sense to only add breadcrumbs to transactions where the timestamp is within the transaction boundaries. Would this solve your current issue? I still have to ensure that this correlates with the expected behavior of the API or, rather, the implementations in other SDKs.

from sentry-native.

Hey @jerin-omcare thanks for writing in - the behaviour you describe is intentional.

The goal of breadcrumbs is to give context information of events that happened before or during a transaction, and can include a variety of data (e.g. changes in network state, http requests, apps becoming active or going into background, etc.). Many of these events are relevant for transactions even though they happen outside (before) the time frame of the transaction. Therefore all our SDKs do not scope breadcrumbs to specific transactions.

You can read more about how we envision breadcrumbs to be used in the docs and in our develop docs.

from sentry-native.

@kahest thank you for the response, we've observed the breadcrumbs are queued and added all the way back to app start up depending on our configured breadcrumb limit. Is there a built in mechanism to ensure the global breadcrumb size doesn't grow forever?

from sentry-native.