Closed jkmassel closed 4 years ago
cc @diegoreymendez @guarani @frosty
Pasting repro steps here from Slack...
First go to Settings.app → Privacy → Photos → WordPress and deselect any of the currently "allowed" photos. Then:
@frosty worth noting that Sentry is reporting this error occurring iOS 13 as well. (cc @designsimply )
For reference, those testing steps sound similar to https://github.com/WordPress/gutenberg/issues/24989 but that report doesn't mention a crash 🤔 and I'm having trouble replicating the steps (I don't see the "Done" option at top right for step 7 when I have not selected any photos yet as noted in step 6).
Tested and confirmed that WPiOS 15.8.0.0 TestFlight beta crashes for me on iOS 14 with the newly introduced photo privacy settings using slightly modified steps: I had to tap "Select more photo…" a 2nd time after step 7. (2m40s)
I could use some help checking whether this crashlog matches with the Sentry issue at https://sentry.io/share/issue/44064e4cd9964f09a5ff497b6a90970f/.
Tested and confirmed that WPiOS 15.8.0.0 TestFlight beta crashes for me on iOS 14 with the newly introduced photo privacy settings using slightly modified steps: I had to tap "Select more photo…" a 2nd time after step 7. (2m40s)
I also had to tap "Select more photos" a second time (like @designsimply above) to reproduce this crash on a fresh install.
After the crash, I reopened the app, went to the Media Lib, tapped +, tapped "Choose from my device" and the app crash 💣.
I could use some help checking whether this crashlog matches with the Sentry issue at https://sentry.io/share/issue/44064e4cd9964f09a5ff497b6a90970f/.
Yes, I think it matches. The log you provided is of an EXC_BAD_ACCESS (which is normally a memory issue) and it's in MediaLibraryViewController
. Looking at the grouping method used on that Sentry issue (under the "Event Grouping Information" section), it is "custom-fingerprint" of type "memory-corruption". So I would say that Sentry would add your crash as a new event to the issue you referenced.
(I had a look on Sentry for this exact crash but couldn't locate it using is:unresolved release:15.8.0.0 user.email:design5279+190313_1141@gmail.com
though 🤔 — not sure why Sentry didn't pick it up.)
The Sentry issue referenced above, https://sentry.io/share/issue/44064e4cd9964f09a5ff497b6a90970f, is causing a lot of crashes again. The actual crash may be different this time, but Sentry's "server side fingerprinting" may be grouping any and all "memory corruption" crashes together under the one Sentry issue.
I noticed this Sentry issue happens weekly, primarily on Mon, Tue, and Wed:
The crash stack trace references "Weekly Roundup" which may explain why we see this crashes happen on a weekly schedule:
6 WordPress 0x204db80e0 Blog.title.getter (Blog+Title.swift:7)
7 WordPress 0x204664cbc [inlined] WeeklyRoundupNotificationScheduler.scheduleDynamicNotification (WeeklyRoundupBackgroundTask.swift:537)
8 WordPress 0x204664cbc WeeklyRoundupNotificationScheduler.scheduleDynamicNotification (WeeklyRoundupBackgroundTask.swift:536)
9 WordPress 0x2046645f4 WeeklyRoundupBackgroundTask.run (WeeklyRoundupBackgroundTask.swift:420)
👋 @frosty, given this relates to the Weekly Roundup code (which I think is also known as Blogging Reminders, correct me if I'm wrong), would you know who could look into this further?
I created a new issue for the crash mentioned in the above comment: https://github.com/wordpress-mobile/WordPress-iOS/issues/17956 cc @frosty
Sentry Url: https://sentry.io/share/issue/44064e4cd9964f09a5ff497b6a90970f/ User Count: 643 Count: 1347 First Release: 15.5.0.3 First Seen: 2020-08-13T12:01:15Z Last Seen: 2020-09-22T04:34:26Z 24 Hours: 81 30 Days: 1344 LogID: 7E14087A-C7D1-4093-BA54-E5EB92A62DDA