Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Muting a topic in the notification center (for 1 hour / today) crashes the notification center #1519

Closed
ShadowJonathan opened this issue Aug 18, 2023 · 4 comments · Fixed by #1598
Labels
A-Notifications O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@ShadowJonathan
Copy link

Steps to reproduce

  1. Enable notifications
  2. Have a notification come in
  3. Swipe on the notification itself, to reveal options menu
  4. Go to sub-menu, press "Mute for today"

Outcome

What did you expect?

The conversation to get muted

What happened instead?

Notification center crashes

Your phone model

iPhone Pro Max 13

Operating system version

iOS 17 Public Beta 3 (But also spotted on iOS 16)

Application version

1.2.3 (64)

Homeserver

jboi.nl

Will you send logs?

No

@GizmoTjaz
Copy link

I can also reproduce this on iOS 16.6 only with EX notifications.

@ShadowJonathan
Copy link
Author

I noticed today the same occurs on watchOS (together with some... pretty interesting failure modes on compilations after the frontend restarted, had to reboot it to fix)

@stefanceriu stefanceriu added S-Minor Impairs non-critical functionality or suitable workarounds exist O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience A-Notifications labels Aug 29, 2023
@stefanceriu
Copy link
Member

I can easily reproduce this as well but I honestly have absolutely no idea what's causing it.

Crashing the simulator results in this nice but useless stacktrace. Presumably there's something about our NSE or NCE configs

Responsible Process: SimulatorTrampoline [3493]

Date/Time:           2023-08-29 17:44:42.2613 +0300
Launch Time:         2023-08-29 17:43:39.3054 +0300
OS Version:          macOS 13.4.1 (22F770820d)
Release Type:        User
Report Version:      104

Exception Type:  EXC_CRASH (SIGABRT)
Exception Codes: 0x0000000000000000, 0x0000000000000000
Triggered by Thread:  6

Last Exception Backtrace:
0   CoreFoundation                	       0x180437324 __exceptionPreprocess + 160
1   libobjc.A.dylib               	       0x180051274 objc_exception_throw + 56
2   CoreFoundation                	       0x180437040 -[NSException init] + 0
3   Foundation                    	       0x180b1d1f0 -[NSObject(NSKeyValueCoding) valueForUndefinedKey:] + 188
4   Foundation                    	       0x180b1c114 -[NSObject(NSKeyValueCoding) valueForKey:] + 300
5   Foundation                    	       0x180b1e03c -[NSDictionary(NSKeyValueCoding) valueForKey:] + 140
6   BulletinBoard                 	       0x19366aa0c -[BBThreadsMuteAssertion getNextExpirationDate:wasPurged:] + 184
7   BulletinBoard                 	       0x19366b1a8 -[BBMuteExpirationManager cleanAndWatchExpirationsForSectionInfo:] + 92
8   BulletinBoard                 	       0x193630ee4 -[BBServer _setSectionInfo:forSectionID:] + 172
9   BulletinBoard                 	       0x193630dd4 -[BBServer _setSectionInfoNoteSettingsChanged:forSectionID:] + 72
10  BulletinBoard                 	       0x1936396f8 __52-[BBServer setSectionInfo:forSectionID:withHandler:]_block_invoke + 288
11  libdispatch.dylib             	       0x180132ee4 _dispatch_call_block_and_release + 24
12  libdispatch.dylib             	       0x180134708 _dispatch_client_callout + 16

@Velin92
Copy link
Member

Velin92 commented Aug 30, 2023

Looking at it.
It seems to be caused by the INSendMessageIntent used by the NSE to create the communication notification.

We are currently trying to understand what in the configuration and use of the intent might be go wrong and triggering this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Notifications O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants