View Issue Details

IDProjectCategoryView StatusLast Update
0005052Talerwallet (WebExtension)public2017-06-06 12:18
ReporterChristian Grothoff 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
PrioritynormalSeveritymajorReproducibilitysometimes
Status closedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Versiongit (master) 
Target Version0.3Fixed in Version0.3 
Summary0005052: demo.taler.net shows wallet as installed only after refresh (F5)
DescriptionI had the wallet already installed, went to demo.taler.net, and got the "wallet not installed" version. Then I hit "F5" and I got the "Congrats".

This is with Chromium.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2017-06-01 21:17

manager   ~0012205

More funky: on the bank page (after logging in), I can push F5 to (sometimes) toggle between wallet found & wallet not-found. Oopsie.

Florian Dold

2017-06-02 00:18

manager   ~0012206

Fixed in a6035dd4.

Pretty easy to explain why this happened, and probably happened because we've moved quite some code around.

We check for the data-taler-nojs attribute to enable stylesheet injection for presence detection. Unfortunately this attribute was checked before we made sure the DOM was loaded, so sometimes the content script was "too fast" and stylesheets weren't injected.

Issue History

Date ModifiedUsernameFieldChange
2017-06-01 21:16Christian GrothoffNew Issue
2017-06-01 21:16Christian GrothoffStatusnew => assigned
2017-06-01 21:16Christian GrothoffAssigned To => Florian Dold
2017-06-01 21:17Christian GrothoffNote Added: 0012205
2017-06-02 00:18Florian DoldNote Added: 0012206
2017-06-02 00:18Florian DoldStatusassigned => informing
2017-06-02 00:18Florian DoldResolutionopen => fixed
2017-06-02 07:34Christian GrothoffFixed in Version => 0.3
2017-06-06 12:18Christian GrothoffStatusinforming => closed
2023-04-13 18:37Florian DoldCategorywallet (WebExtensions) => wallet (WebExtension)