View Issue Details

IDProjectCategoryView StatusLast Update
0008850Talerwallet (WebExtension)public2024-07-25 22:11
ReporterFlorian Dold Assigned Tosebasjm 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
PrioritynormalSeverityfeatureReproducibilityalways
Status closedResolutionwon't fix 
Target Version0.12Fixed in Version0.12 
Summary0008850: webext should call hintNetworkAvailability
DescriptionThe new wallet-core request

  hintNetworkAvailability({isNetworkAvailable: boolean})

should be used to tell wallet-core about network availability.

In case the network becomes available again, all pending tasks are retried immediately.
In the future, when the network becomes unavailable, wallet-core might stop emitting certain errors or not retry certain network requests.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

sebasjm

2024-05-28 17:32

developer   ~0022486

webex is listening two types of evenets
1.-
    window.addEventListener("offline", notifyOffline);
    window.addEventListener("online", notifyOnline);
https://developer.mozilla.org/en-US/docs/Web/API/Window/offline_event

2.-
      navigator.connection.addEventListener("change", notifyChange);
https://developer.mozilla.org/en-US/docs/Web/API/NetworkInformation/change_event

None seems reliable. Works when changing the value in the devTool but does not respond to system changes.

navigator.online has also wrong values
https://developer.mozilla.org/en-US/docs/Web/API/Navigator/onLine

It may work better in other OS but I think we can't relay on this feature.

sebasjm

2024-05-28 17:44

developer   ~0022487

IMHO wallet-core should report the network error with a TalerErrorCode so the UI can decide: if this is a network error and (in case that UI can check network status) we are offline, the do not show the error msg.
In browser it seems that we can't do it reliable on all platform.
Tested on Ubuntu 23.04 firefox 123.0.1 chrome 122.0.6261.111

Issue History

Date ModifiedUsernameFieldChange
2024-05-15 22:34Florian DoldNew Issue
2024-05-15 22:34Florian DoldStatusnew => assigned
2024-05-15 22:34Florian DoldAssigned To => sebasjm
2024-05-15 22:34Florian DoldIssue generated from: 0008849
2024-05-15 22:34Florian DoldIssue cloned: 0008851
2024-05-27 17:02sebasjmCategorywallet (Android App) => wallet (WebExtension)
2024-05-28 17:32sebasjmNote Added: 0022486
2024-05-28 17:33sebasjmStatusassigned => implementing
2024-05-28 17:44sebasjmStatusimplementing => informing
2024-05-28 17:44sebasjmResolutionopen => won't fix
2024-05-28 17:44sebasjmNote Added: 0022487
2024-07-25 22:11Christian GrothoffFixed in Version => 0.12
2024-07-25 22:11Christian GrothoffStatusinforming => closed