View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0008917 | Taler | wallet (Android App) | public | 2024-06-10 14:30 | 2024-08-28 21:31 |
| Reporter | avalos | Assigned To | avalos | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Target Version | 0.13 | Fixed in Version | 0.13 | ||
| Summary | 0008917: Withdrawal process is slow for new exchanges | ||||
| Description | It has been spotted that withdrawing from new exchanges takes significantly longer in Android when compared to withdrawing from exchanges already known by the wallet. The same is not true for iOS, even in older hardware. Investigate why is this happening, - is it the UI not responding to certain notifications? - is the native cURL client doing something inefficiently? - is it the spirits playing tricks on us? | ||||
| Tags | No tags attached. | ||||
| Apparently, the major bottlenecks are the multiple HTTP requests that result from withdrawing from a new exchange: 2024-06-10 08:43:49.599 Received notification from wallet-core: {"type":"request-observability-event","event":{"id":"req-1007","when":{"t_ms":1718030629593},"type":"http-fetch-start","url":"https://bank.taler.ar/taler-integration/config"}} 2024-06-10 08:43:51.214 Received notification from wallet-core: {"type":"request-observability-event","event":{"id":"req-1007","when":{"t_ms":1718030631204},"type":"http-fetch-finish-success","url":"https://bank.taler.ar/taler-integration/config","status":200}} --- 2024-06-10 08:43:51.219 Received notification from wallet-core: {"type":"request-observability-event","event":{"id":"req-1008","when":{"t_ms":1718030631206},"type":"http-fetch-start","url":"https://bank.taler.ar/taler-integration/withdrawal-operation/a8448c70-72df-4697-8df0-e8205b5e45ea"}} 2024-06-10 08:43:52.411 Received notification from wallet-core: {"type":"request-observability-event","event":{"id":"req-1008","when":{"t_ms":1718030632405},"type":"http-fetch-finish-success","url":"https://bank.taler.ar/taler-integration/withdrawal-operation/a8448c70-72df-4697-8df0-e8205b5e45ea","status":200}} --- 2024-06-10 08:43:52.489 Received notification from wallet-core: {"type":"request-observability-event","event":{"id":"req-1009","when":{"t_ms":1718030632482},"type":"http-fetch-start","url":"https://bank.taler.ar/conversion-info/cashin-rate?amount_credit=YEIN%3A10"}} 2024-06-10 08:43:53.686 Received notification from wallet-core: {"type":"request-observability-event","event":{"id":"req-1009","when":{"t_ms":1718030633678},"type":"http-fetch-finish-success","url":"https://bank.taler.ar/conversion-info/cashin-rate?amount_credit=YEIN%3A10","status":200}} --- 2024-06-10 08:43:53.689 Received notification from wallet-core: {"type":"request-observability-event","event":{"id":"req-1010","when":{"t_ms":1718030633679},"type":"http-fetch-start","url":"https://bank.taler.ar/conversion-info/config"}} 2024-06-10 08:43:54.891 Received notification from wallet-core: {"type":"request-observability-event","event":{"id":"req-1010","when":{"t_ms":1718030634884},"type":"http-fetch-finish-success","url":"https://bank.taler.ar/conversion-info/config","status":200}} --- 2024-06-10 08:43:54.915 Received notification from wallet-core: {"type":"request-observability-event","event":{"id":"req-1011","when":{"t_ms":1718030634913},"type":"http-fetch-start","url":"https://exchange.taler.ar/terms"}} 2024-06-10 08:43:56.521 Received notification from wallet-core: {"type":"request-observability-event","event":{"id":"req-1011","when":{"t_ms":1718030636518},"type":"http-fetch-finish-success","url":"https://exchange.taler.ar/terms","status":200}} | |
| Almost seven whole seconds are spent entirely doing HTTP fetches! | |
| This will (hopefully) be fixed in 0008920 | |
| Indeed, the speed went from ~7 seconds to ~2, sometimes even less than a second. | |
| As this ticket only covers the investigation part, I will close it now that we have results. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2024-06-10 14:30 | avalos | New Issue | |
| 2024-06-10 14:30 | avalos | Status | new => assigned |
| 2024-06-10 14:30 | avalos | Assigned To | => avalos |
| 2024-06-10 14:54 | avalos | Note Added: 0022568 | |
| 2024-06-10 14:56 | avalos | Note Added: 0022569 | |
| 2024-06-13 14:54 | avalos | Relationship added | related to 0008920 |
| 2024-06-13 14:55 | avalos | Note Added: 0022593 | |
| 2024-06-13 14:55 | avalos | Note Edited: 0022593 | |
| 2024-06-17 13:45 | avalos | Note Added: 0022645 | |
| 2024-06-17 13:47 | avalos | Status | assigned => informing |
| 2024-06-17 13:47 | avalos | Resolution | open => fixed |
| 2024-06-17 13:47 | avalos | Note Added: 0022646 | |
| 2024-08-26 20:29 | Christian Grothoff | Target Version | => 0.13 |
| 2024-08-26 20:29 | Christian Grothoff | Fixed in Version | => 0.13 |
| 2024-08-28 21:31 | Christian Grothoff | Status | informing => closed |