View Issue Details

IDProjectCategoryView StatusLast Update
0008509Talerwallet (Android App)public2024-04-15 19:32
ReporterFlorian Dold Assigned Togrote Resolution deadline
ImplementerReviewer 
Release signed byDeployed by 
VerifierCommunicator 
Data protection authority notification requiredNo  
CSIRT notification requiredNo  
Estimated effort (hours)Actual effort (hours)0 
Fast-trackNo
Work packagenone
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Target Version0.10Fixed in Version0.10 
Summary0008509: show observability events in Android UI in dev mode
DescriptionWhen in dev mode, instead of showing a spinner, we should render a list of observability events from wallet-core.

Also in the transactions view, we can show the observability events for tasks of that transaction.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

avalos

2024-03-15 21:53

developer   ~0021937

Have we agreed on a UI for this already? How should the observability events be rendered and which ones should be displayed when?

 I would not like this to be overtly obtrusive, not even for developers. It should never clutter the IU. What I'm thinking about is a global toolbar button that shows a popup with all the observability events displayed in real time. At what frequency are these events generated by wallet-core? Should I worry about the event list moving too fast that it will be hard to keep track of?

Florian Dold

2024-03-25 20:08

manager   ~0021991

The event list is likely to be moving rather fast! It shows every single HTTP request, crypto operation and even DB transaction that wallet-core does.

Marc had a prototype where he showed logs (that was before observability events!) in some kind of floating overlay window.
That sounds pretty similar to the popup that you're suggesting!

We could either do this or we can show it "inline" as a replacement of the full-screen loading indicator.

Florian Dold

2024-03-25 20:08

manager   ~0021992

Also, ideally we in the settings there should be an option to export the buffer of recent observability events, so we can send this around for diagnostics *after* a problem occurred.

avalos

2024-03-26 15:58

developer   ~0022000

Where's the documentation for this feature?

Christian Grothoff

2024-03-27 18:32

manager   ~0022015

We should soon show in the UIs what is happening whenever we're blocked/waiting/spinning. I just today got another report from Michael R, saying that he tried to do a P2P payment with a student and just got a spinning wheel (iOS). He blamed it on the network (after all, the saying goes that if you want Internet at our university, try turning OFF the wifi), but we cannot be sure that he was right without any information shown on the screen. So it would be good if we had a way to show ("waiting on DNS", "waiting on TLS", "waiting on HTTP", "computing" (crypto)) in small-ish but readable text below the spinner if just to help us diagnose such issues! Right now, I get these reports and cannot even say if it is the BFH network or a major bug in our code! So fixing 0008509 (and equivalent on iOS) matters!

avalos

2024-03-27 18:33

developer   ~0022016

I'm almost done with that, implementing right now, should be ready soon.

avalos

2024-03-27 20:46

developer   ~0022026

Initial implementation in dev/ivan-avalos/observability, awaiting Torsten feedback and review.

grote

2024-04-01 12:38

reporter   ~0022079

implemented as of https://git.taler.net/taler-android.git/commit/?id=e33bc723687328eec69e7580dd3e5698d4679167

Issue History

Date ModifiedUsernameFieldChange
2024-02-23 15:27Florian DoldNew Issue
2024-02-23 15:27Florian DoldStatusnew => assigned
2024-02-23 15:27Florian DoldAssigned To => avalos
2024-03-15 21:53avalosAssigned Toavalos => Florian Dold
2024-03-15 21:53avalosStatusassigned => feedback
2024-03-15 21:53avalosNote Added: 0021937
2024-03-25 20:08Florian DoldNote Added: 0021991
2024-03-25 20:08Florian DoldAssigned ToFlorian Dold => avalos
2024-03-25 20:08Florian DoldNote Added: 0021992
2024-03-25 20:08Florian DoldStatusfeedback => assigned
2024-03-26 15:58avalosNote Added: 0022000
2024-03-27 18:32Christian GrothoffNote Added: 0022015
2024-03-27 18:33avalosNote Added: 0022016
2024-03-27 20:46avalosAssigned Toavalos => grote
2024-03-27 20:46avalosStatusassigned => feedback
2024-03-27 20:46avalosNote Added: 0022026
2024-04-01 12:38groteStatusfeedback => informing
2024-04-01 12:38groteResolutionopen => fixed
2024-04-01 12:38groteNote Added: 0022079
2024-04-09 11:11Christian GrothoffFixed in Version => 0.10
2024-04-15 19:32Christian GrothoffStatusinforming => closed