MMS users: call for testing

lduboeuf · 31 Jul 2021, 14:34

Hi,

We need your help testing a new feature brought by @jezek regarding MMS and the ability to re-download the MMS afterward (if any failure or the cellular data was off when the MMS was received).
Please report here any issue found or success. ( Note that this feature will not correct any APN setting issues, or issues with the provider or device specific )

For the long story: https://forums.ubports.com/topic/5100/the-mms-lost-story

You need to be on devel or RC ( since 2021/06/09 )
From a terminal, run that commands one by one:

sudo ubports-qa install PR_nuntium_8
sudo ubports-qa install telepathy-ofono 20
sudo ubports-qa install history-service 35
sudo ubports-qa install telephony-service 20
sudo ubports-qa install messaging-app 260

And reboot

Test Plan example:
Make sure it works as before, send/receive MMS, multiple ones
Disable cellular-data, receive a MMS and check if MMS notification is here with the download button
Change proxy settings in APN or put wrong settings and check for notification reception

Keneda · 1 Aug 2021, 12:58

@lduboeuf
If test successfull, release in next OTA?

Emphrath · 1 Aug 2021, 07:51

@lduboeuf Tested this morning on FP2 halium-devel. It works, although not perfectly. First, it seems like I had to toggle WiFi along with cellular to receive the images. Might be related to data being limited to H+ on this sim. For each message sent, the receiving FP2 often (not always) displayed two different error messages: the usual one with a download button, and a new, shorter one, with the size of the attachment, and a download button. If i tap on the usual one after toggling data back on, it downloads, and deletes the other message. If i tap on the new one, it also downloads, but the other message is still displayed, and tapping the download button is useless.

Emphrath · 1 Aug 2021, 09:34

@lduboeuf haven't tried multiple mms yet

lduboeuf · 1 Aug 2021, 12:58

@keneda no idea

lduboeuf · 8 Jan 2021, 13:48

@emphrath are you trying to send a MMS to yourself ?
Any screenshot available ?
[Edit] : no need for screenshot. So there are two mms notifications instead of only one. Maybe @jezek have an idea

Emphrath · 1 Aug 2021, 21:59

@lduboeuf no, i was sending from another phone.

jezek · 8 Feb 2021, 19:38

@emphrath said in MMS users: call for testing:

Tested this morning on FP2 halium-devel. It works, although not perfectly. First, it seems like I had to toggle WiFi along with cellular to receive the images. Might be related to data being limited to H+ on this sim.

I'm pretty sure WiFi has nothing to do with MMS. Unless some new technology I missed, or the operator is very brave to put user messages on public internet. Or am I wrong?

For each message sent, the receiving FP2 often (not always) displayed two different error messages: the usual one with a download button, and a new, shorter one, with the size of the attachment, and a download button. If i tap on the usual one after toggling data back on, it downloads, and deletes the other message. If i tap on the new one, it also downloads, but the other message is still displayed, and tapping the download button is useless.

I've tried halium-devel on my FP2 yesterday and I can't reproduce the double message error behavior.

How many times does it happen?
Have you rebooted your phone after applying the ubports-qa commands?
Does any message got lost (you didn't receive the message in the end) aside of the double error message?
~~After restarting the phone does the useless error message disapear?~~ Edit: Ignore this question. There is no logic for such a behavior. The answer should be always "No".
Could you provide logs (nuntium.log, dbus.log, application-legacy-messaging-app-.log) from the case when the double error message happened?
Note: It would be helpful, when you delete the logs before the test to not pollute the test logs with noise (logs with events before test). I prefer to:
1. Delete (relevant) logs.
2. Reboot phone.
3. Do tests until the error occures.
4. Copy/paste logs to some service (pastebin, gist) and report the link.

Thank you for your cooperation.

Emphrath · 3 Aug 2021, 00:07

@jezek unfortunately i won't be able to test again. I used a friend's sim in the fp2, which is not my main device, and my present sim is stuck in my oneplus tray. I can try to get the logs from the fp2, but they won't be "clean". The fp2 was having some issues of its own, too, so maybe that's that. Sorry I can't help further

jezek · 3 Aug 2021, 13:41

@emphrath said in MMS users: call for testing:

@jezek unfortunately i won't be able to test again. I used a friend's sim in the fp2, which is not my main device, and my present sim is stuck in my oneplus tray. I can try to get the logs from the fp2, but they won't be "clean". The fp2 was having some issues of its own, too, so maybe that's that. Sorry I can't help further

It's OK. Just provide the logs, you have. But don't wait too long, the logs can get more polluted (and it will be more pain to go through them). Also keep in mind, that logs get rotated, so the relevant parts may be already zipped in another file (remembering the time of test will help). And try to answer the other questions, please. Thanks.

lduboeuf · 3 Aug 2021, 16:56

@jezek just as a reminder, the issue that i've found last time:
https://github.com/ubports/nuntium/pull/8#issuecomment-861480575

Emphrath · 8 Apr 2021, 10:05

@jezek I don't use that phone, and it's just sitting there with no wifi or cellular, so I guess what you'll see in the logs will be rather plain, mostly background daemon routines. I'll try to get them tonight. Now for the questions:
-I think I sent four mms in total and all of them displayed this behavior.
-I did reboot, since ubports-qa explicitly asked me to (and you did as well) but I saw some errors while passing the commands. I can't remember properly, but sth about sth not being sandboxed, and also about the system being ro... I didn't have to make it
rw beforehand, right ? Anyway i'll get you these errors asap
-no message got lost

lduboeuf · 4 Aug 2021, 22:43

@emphrath ubports-qa script often output errors even if ok

@jezek it seems nuntium received two wap push don't you think ? That could explain the two mms notif.
In case of bad signal if the wap push couldn't be acknowledged, the provider might send it twice t

Emphrath · 5 Aug 2021, 08:26

@lduboeuf Quite frankly I wouldn't look any further. My fairphone has so many hardware issues it's hard to get a proper idea about the culprit - noticeably network issues, so...

lduboeuf · 9 Aug 2021, 20:57

For the use case "cellular data off", it is working nicely except when i'm with bad signal ( see issue above ) .

This is already a huge step for MMS users, in its worst case scenario we will have the same state as it is today on Devel/RC/Stable ( just ask user to resend)

jezek · 8 Oct 2021, 00:25

@lduboeuf said in MMS users: call for testing:

For the use case "cellular data off", it is working nicely except when i'm with bad signal ( see issue above ) .
This is already a huge step for MMS users, in its worst case scenario we will have the same state as it is today on Devel/RC/Stable ( just ask user to resend)

I think it is not a showstopper for deploying to next OTA. It's working better than before and there are no regressions. Also it will motivate me more to fix the bad signal issues, when the changes land to stable. I will definitely tackle the issues, but I had to change my HW and installed manjaro on it and now I have a little fight with crossbuilder and the lack of apt and dpkg[-architecture,-parsechangelog] commands on the system.

Thank you all for support.

jezek · 11 Sept 2021, 14:57

I've discovered, that the patches are already in RC, so if you are on RC channel (and dev channel too), there is no need to apply the above mentioned ubports-qa commands. This also means that the patches will probably be in the next OTA. Just saying. I'm happy. Thank you all.

I've been wrong. Read furher, why.

AppLee · 11 Sept 2021, 15:13

@jezek said in MMS users: call for testing:

Thank you all.

Thank YOU :thumbs_up:

lduboeuf · 11 Sept 2021, 15:31

@jezek humm i probably miss something, i don't see the PR already merged

jezek · 13 Sept 2021, 14:59

@lduboeuf Or am I wrong? Is there a possibility that the patch is not in RC (and dev) and the applied ubports-qa changes persisted 2 RC upgrades and I drew a bad conclusion? Can anyone answer this conundrum, or do I have to do full reinstall to check?