Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

Risk display wrongly show low risk for a few seconds before changing #1185

Closed
simon-anders opened this issue Sep 19, 2020 · 11 comments
Closed
Assignees
Labels
bug Something isn't working mirrored-to-jira This item is also tracked internally in JIRA

Comments

@simon-anders
Copy link

simon-anders commented Sep 19, 2020

A few days ago, i got an exposure notification.

Now, when I open the app, the following happens:

  • a green screen, entitled "low risk" is shown (screenshot 1), with a wrong notification of "no exposures" according to an update "yesterday" -- even though yesterday's update already showed the exposures.

  • after a few seconds, the screen switches to red (screenshot 2), showing the same update time stamp, but now notifying me of 2 exposures.

Obviously, a user who only briefly looks at the app will only see the green screen and get a wrong all-clear.

Frequency: seems to happen only the first time of opening the app per day

Version: 1.3.0

Android version: EMUI 9.1.0 on Huawei ANE-LX1

Screenshot 1:

Screenshot_20200919_110044_de rki coronawarnapp

Screenshot 2:

Screenshot_20200919_110048_de rki coronawarnapp


Internal Tracking ID: EXPOSUREAPP-2873, EXPOSUREAPP-2270

@simon-anders simon-anders added the bug Something isn't working label Sep 19, 2020
@thomasaugsten
Copy link
Member

Hi can you provide the export of the exposure log in the Android exposure notification settings?

@simon-anders
Copy link
Author

Exposure check log:
https://drive.google.com/file/d/1iFhnnksCH6KOurmpUXheZtYBoK-N0UdI/view?usp=drivesdk

@akuckartz
Copy link

I had similar effects when I had "Low Risk" encounters. The displayed nummer was changing about half a second after the Initial display whenever I opened the app.

@daimpi
Copy link

daimpi commented Sep 19, 2020

@simon-anders

seems to happen only the first time of opening the app per day

Looking at your ENF log and your screenshots, it looks like the update today happened exactly when you opened CWA at 11:00. This might explain why it took a few seconds to update to screen 2 (but it doesn't explain why screen 1 was green in the first place). Probably when you open your app now, it shows you "Updated: Today, 11:00" with two matches and a red screen immediately, right?

Looking further into your log it seems that the timestamps when an update was performed are all over the place in general. This could indicate, that automatic background updates are not working properly on your phone and the updates therefore only happen once you open CWA manually.

Does it sound plausible to you that you might have manually opened CWA in the past few days at the following times?

Date Time
19.9.2020 11:00
18.9.2020 13:03
17.9.2020 02:45
16.9.2020 13:15
15.9.2020 12:20
14.9.2020 16:12

Btw: do you have "Prioritized Background Activity" enabled in your CWA settings?

@simon-anders
Copy link
Author

Yes, that might be plausible.
And, yes, "prioritized background activity" is on, and "battery optimization" i "not allowed" for CWA

@daimpi
Copy link

daimpi commented Sep 19, 2020

@simon-anders
Ok, thanks for the info 🙂.

This means it's quite plausible that background updates are indeed not working properly on your device in which case your problem might be related to what @vaubaehn laid out here:

1) The mainView/risk card is not completely refreshed, when

  • the automatic background update has not been carried out for the current day, and

  • CWA is openend by the user, and

  • the diagnostic keys are exchanged in the background at the same time.

In this case @Jo-Achim can see the message "Risiko-Ermittlung nicht möglich, Risiko-Ermittlung einschalten", even the update succeeded. In my case, I see a green risk card, showing last update was done yesterday.
When you then navigate to a different screen/view, no matter whether inside or outside CWA, on return/onResume, the mainView is refreshed, and the correct status is displayed in the risk card.

Possible fix:

  • Take care, that risk card shows "updating", when ProvideDiagnosisKeys is called during a foreground process.

  • Make sure, that on receiving broadcast Action_Exposure_State_Updated app is in foreground, mainView is active, and refresh the view.

  • If the above listener is not sufficient, additionally create a listener for Action_Exposure_Not_Found (as far as I could see from the code, no listener is set yet.) and refresh view on retrieval.

And the act of taking the screenshots (which usually involves a context switch) might have been enough to trigger a screen update subsequently.

Not sure why your screen 1 is green though, as your app already showed a red screen yesterday 🤷.

If you want to test something: maybe tomorrow you could try to open CWA and don't interact with your phone afterwards (except for preventing screensaver) for ~1 min and check whether your app stays stuck on screen 1 for this time?

Btw: what is the behavior if you open the app now i.e. after today's update has been performed?

@heinezen
Copy link
Member

@simon-anders Thank you for reporting the issue. We have created an internal Jira ticket (EXPOSUREAPP-2873) for the bug. If there are any updates on the status of the bug, you will get notified here.

Regards,
CH


Corona-Warn-App Open Source Team

@ghost
Copy link

ghost commented Sep 29, 2020

This issue has been marked as obsolete and will be closed now.

@ghost ghost closed this as completed Sep 29, 2020
@daimpi
Copy link

daimpi commented Sep 29, 2020

@GPclips

This issue has been marked as obsolete and will be closed now.

Could you elaborate on why this has been marked as "obsolete"? Has this issue been fixed and if so could you kindly provide a link? 🙂

@ghost
Copy link

ghost commented Sep 29, 2020

Hello @daimpi ,

I just checked and the issue now will be tracked in the following item: EXPOSUREAPP-2270. This one is still in the status analyzing. I will go on and update the task.

Thanks,
LMM

Corona-Warn-App Open Source Team

@ghost ghost reopened this Sep 29, 2020
@heinezen
Copy link
Member

heinezen commented Mar 9, 2021

Hello everyone,

We cannot reproduce this anymore in the latest CWA versions. The issue last occured in CWA 1.7 for us. This means that the problem should hopefully be gone for everyone. We will therefore close this issue.

If you are still experiencing this problem, please make a comment in this thread.


Corona-Warn-App Open Source Team

@heinezen heinezen closed this as completed Mar 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working mirrored-to-jira This item is also tracked internally in JIRA
Projects
None yet
Development

No branches or pull requests

6 participants