sentry-cocoa
sentry-cocoa copied to clipboard
Crash reports not generated on watchOS
Platform:
- [ ] iOS
- [ ] tvOS
- [ ] MacOS
- [x ] watchOS
Swift:
- [x 5.1] Yes -> If yes, which version?
- [ ] No
sentry-cocoa installed with:
- [ ] CocoaPods
- [x ] Carthage
- [ ] Manually
Version of sentry-cocoa: 4.5.0
I have following issue: I have sentry installed in the watchkitextension and enabled. When I trigger a crash with Client.shared?.crash()
it crashes but isn't generating a report. I am monitoring the files via the simulator. I am running in the simulator but without the debugger attached and looking at the process logs through Console.
2020-03-12 14:53:07.317602-0500 WatchCrashlytics WatchKit Extension[10206:10166742] Sentry - Debug:: SentryCrashHandler started
2020-03-12 14:53:07.323403-0500 WatchCrashlytics WatchKit Extension[10206:10166742] Sentry - Debug:: Sent 0 crash report(s)```
**Steps to reproduce:**
- I created a new watchkit + iphone app in Xcode. I then followed https://sentry.io/under-armour-l7/under-armour-l7/getting-started/cocoa-swift/.
**Actual result:**
- No crash reports are generated
**Expected result:**
- Crash reports should be generated
I have also tried this on 5.0.0-alpha.4 as well and it does not seem to run the [SentryCrash install]
. Adding breakpoints it shows it runs through [SentryCrashInstallation install]
but never actually goes into any of those methods, just seems to roll over them.
@jostster Can you share your example app with Sentry installed so we can reproduce this? Thanks
@HazAT is there an email address I can send it to?
My firstname @ sentry.io
Any new updates on this @HazAT or @jostster? Can you try it with version 5.x.x and let us know if you still have the issue present?
@HazAT How can I help to debug this? Running Sentry 5.1.8 on watchOS 6.2.8. Events reported with SentrySDK.capture(message: "Hello")
arrive, but SentrySDK.crash()
does not. I'm testing on device, not attached to the debugger.
Hey all,
any updates on this issue?
Have the same issues like @st3fan : SentrySDK.capture(message: "Hello")
works, but SentrySDK.crash()
not. Tested with different Watch Models and Simulator.
Thanks, @st3fan and @mu99el for reaching out to us about this issue. We are going to take a look soon and come back to you.
@mu99el thanks for reporting this. I can reproduce the issue. We are trying to fix this, but I can't give you an ETA at the moment.
We update the Readme https://github.com/getsentry/sentry-cocoa/pull/632 and our official docs for https://github.com/getsentry/sentry-docs/pull/1924 to have clear expectations for all devs at the moment.
Any progress on this? It feels like one is left quite alone when developing watchOS apps... :)
Sorry @LiMuBei no progress update on this one. Maybe we can learn from this issue https://github.com/firebase/firebase-ios-sdk/issues/4558 and this PR https://github.com/firebase/firebase-ios-sdk/pull/6262.
You may want to update your watchOS marketing page if this is still unavailable. I signed up thinking crash reporting was available until I found in the documentation that it wasn't in reality.
@aaronpearce Apologies for the confusion. We'll update the marketing pages to clarify the state of the support.
We’re running into the same issue after following your documentation to integrate Sentry v7.2.8 into our watchOS app. Very misleading.
Is there a timeline when this is expected to be resolved?
This was a regression to the docs (making content reusable between "Apple platforms") and again showed the crash bits for watchOS. Sorry once again for the confusion
Sorry about the confusion @triplef; I opened up a PR to fix this on the docs: https://github.com/getsentry/sentry-docs/pull/4175.
This issue has gone three weeks without activity. In another week, I will close it.
But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog
or Status: In Progress
, I will leave it alone ... forever!
"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀
Is there any ETA on when Sentry SDK for watchOS would handle and report crashes? We are using Sentry in our technology stack, but if it still doesn't support handling crashes we would need to use another service in our watchOS app :( Thanks!
Sorry to hear that @albertinad. I can't give you an ETA at the moment. You asking for this definitely gives this a bit of a higher priority.
@philipphofmann If asking is giving this a bit higher priority, then I'm asking as well.
The docs are extremely confusing, as they seem to be auto-generated for all OS', e.g. https://docs.sentry.io/platforms/apple/guides/watchos/ has two small hints that crash handling is not supported, but then there is this big Crash handling
section in here. Same goes for the marketing in https://sentry.io/for/watchos.
You should definitely communicate this more clearly.
Thanks for pointing that out, @EmDee. I raised an issue (https://github.com/getsentry/static-sites/issues/779) to fix this for our marketing https://sentry.io/for/watchos/. The repository for static sites is internal, though.
I also opened up a PR for the docs to remove the section on crash handling to avoid confusion https://github.com/getsentry/sentry-docs/pull/4448.
Have a look at KSCrash. Maybe they already added it.
Any update on this? We're evaluating to migrate from Crashlytics to Sentry and also assumed that watchOS is fully supported.
Sorry, we don't have an update on this as not many people ask for this. I'm happy to hear that you are evaluating moving to Sentry @weakfl. Is this a blocker for you?
not many people ask for this.
That kind of surprises me considering the ever increasing number of watchOS users.
Is this a blocker for you?
Well, kind of @philipphofmann. But given the lack of alternatives besides Crashlytics we'll probably have to make do. I hope you'll consider adding supporting for watchOS crashes nonetheless.
Of course, we consider adding support for watchOS crashes at some point, but right now, we don't see many people asking for it, which is why it has a lower priority. We are always happy to accept PRs if you are up for it.
Just been going through the code a little to understand what it would take to implement this, but it's quite a large codebase to digest :) I've found an interesting comment though: https://github.com/getsentry/sentry-cocoa/blob/e47eea57ef97e9b1a949a230b74219587619c33f/Sources/SentryCrash/Recording/SentryCrashSystemCapabilities.h#L130 And it seems due to this watchOS is excluded in the following define which seems to disable the crash reporting functionality. The respective commit for this is apparently 4 years old. Maybe this is not true anymore? I'm also wondering what this refers to? watchOS version 3.1?
@LiMuBei, that's exactly the problem. You can enable it back, at least for the last WatchOS version, looks like it is working based on some tests I've done. But we need to investigate it further to determine since which version it is ok to enable it, and if there is no other issue related to it, before we fix this in the SDK
we don't see many people asking for it
I'm just chiming in to ask for watchOS support! Thank you!