View Issue Details

IDProjectCategoryView StatusLast Update
0004221Talerotherpublic2016-10-11 15:28
ReporterFlorian Dold Assigned ToFlorian Dold 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 
Product Version0.0 
Target Version0.1Fixed in Version0.1 
Summary0004221: remove wallet load/unload events from spec and code
Description... there is basically no way to implement those reliably, especially with WebExtensions.

Instead, timeouts should be used to detect non-presence of a wallet, and websites may periodically probe (let's say every 300ms) to detect presence.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Florian Dold

2016-03-02 12:47

manager   ~0010216

taler-presence.js should abstract over these things (timeouts, re-querying) and provide simple callbacks.

Christian Grothoff

2016-03-19 18:42

manager   ~0010252

I thought you figured out a way to do unload-detection. While I agree with that taler-presence.js should abstract it out, I don't think that's a reason to not have it in the spec. Please clarify what's left to do here.

Florian Dold

2016-09-12 21:49

manager   ~0011109

All code that interfaces with the wallet now uses the on{Present,Absent} events from the wallet lib.

We don't have the load/unload events, as they're unreliable at least in Chrome.

Issue History

Date ModifiedUsernameFieldChange
2016-03-02 04:17Florian DoldNew Issue
2016-03-02 12:47Florian DoldNote Added: 0010216
2016-03-02 12:47Florian DoldAssigned To => Florian Dold
2016-03-02 12:47Florian DoldStatusnew => assigned
2016-03-08 20:26Christian GrothoffProduct Version => 0.0
2016-03-08 20:26Christian GrothoffTarget Version => 0.1
2016-03-19 18:42Christian GrothoffNote Added: 0010252
2016-03-20 20:53Torsten GrothoffIssue cloned: 0004259
2016-09-12 21:49Florian DoldNote Added: 0011109
2016-09-12 21:49Florian DoldStatusassigned => informing
2016-09-12 21:49Florian DoldResolutionopen => fixed
2016-09-18 22:53Christian GrothoffFixed in Version => 0.1
2016-10-11 15:28Christian GrothoffStatusinforming => closed