Zom-iOS-XMPP icon indicating copy to clipboard operation
Zom-iOS-XMPP copied to clipboard

Disable the user interface notification for bad server certification

Open tiffrobo opened this issue 6 years ago • 10 comments

img_4969 2

@N-Pex Device: iPhone8 OS: 11.3.1 App version: release build 130

tiffrobo avatar May 21 '18 19:05 tiffrobo

This is also happening to Rosa. Any progress?

n8fr8 avatar May 29 '18 14:05 n8fr8

@abeluck any thoughts on the server side of this possibly?

n8fr8 avatar May 29 '18 14:05 n8fr8

Yes, I am pretty sure I've fixed it (a new UUID was used every time, so instead of "reusing" the notification a new one was created). I just need to be sure that the fix is not affecting things to the other extreme, i.e. showing too few notifications because they are lumped together. Will try to do a new build soon, probably tomorrow.

N-Pex avatar May 29 '18 14:05 N-Pex

Can you tell me more about when it is triggered? Is it generated by a connection to the push.zom.im server or the home.zom.im?

Let's talk on the scrum today.

n8fr8 avatar May 29 '18 14:05 n8fr8

(oops tomorrow!)

n8fr8 avatar May 29 '18 14:05 n8fr8

@n8fr8 I haven't actually tried to chase the cause, I've only fixed the symptom (by faking a connection problem in the code, so that it tries to show a new notification when trying to reconnect). I've not seen this, but that's probably because I don't test on the home.zom.im server. I can give that a go tomorrow as well.

N-Pex avatar May 29 '18 14:05 N-Pex

Let's keep an eye out for this. @N-Pex Fixed the multiple push notifications and user should only see one, but we need to figure out if this is a false push notification and if so, triggered by what.

tiffrobo avatar May 31 '18 18:05 tiffrobo

Can we just disable this notification? Too many false positives.

n8fr8 avatar Jun 20 '18 19:06 n8fr8

Please retest in build 134.

N-Pex avatar Jun 25 '18 14:06 N-Pex

So! Kaci was seeing this error, but I she has the app store version. I haven't seen it during testing build 134 or 135.

Device: iPhone 6, iPhone 7 OS: 10.3.3, 11.2.2 App version: build 135

tiffrobo avatar Jun 29 '18 22:06 tiffrobo