Philipp Hofmann
Philipp Hofmann
Yes, please use the same approach as for these values https://github.com/getsentry/sentry-cocoa/blob/ceaa5790db2d26f454ad7962a9aaa62d09467449/Sources/Sentry/SentryCrashIntegration.m#L251-L253 They show up like this in Sentry: We need changes in Sentry to support that. Similar to these lines...
> Even though it happens in a different file, the approach in the end is the same, right? Sentry's UI just needs to know about it. Yes, exactly.
@ueman or @marandaneto, do you already send this for the dart SDK? If yes, which name did you choose for this property?
Maybe talk to @untitaker before shipping this.
I can't remember why I created this issue. If you, @untitaker, tell me we shouldn't use the field, I'm happy to close the issue.
Thanks, @marandaneto and @untitaker for the info.
Thanks for reporting this @sindresorhus. I noticed that KSCrash has a few fixes for [KSCrashMonitor_MachException](https://github.com/kstenerud/KSCrash/blob/master/Source/KSCrash/Recording/Monitors/KSCrashMonitor_MachException.c) over the past year, that we don't include in SentryCrash. I think we should apply...
I'm sorry about that, @sindresorhus. I will bump the priority for this issue.
Hey, @martindufort. We don't have plans to add flush soon. What you described should be solved with https://github.com/getsentry/sentry-cocoa/issues/316. So ideally, our SDK would handle everything, and you don't need to...
Thanks for the update, @martindufort.