View Issue Details

IDProjectCategoryView StatusLast Update
0004005Talerwallet (WebExtension)public2016-02-19 21:33
ReporterMarcello Stanisci Assigned ToFlorian Dold  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionno change required 
Product Version0.0 
Target Version0.0Fixed in Version0.0 
Summary0004005: Mark reserve as "defunct" after successful withdrawal
DescriptionInhibit the reserve, by marking it as empty or by removing
it (from DB), after the wallet withdraws all the money in it.
TagsNo tags attached.

Activities

Marcello Stanisci

2015-10-14 13:10

reporter   ~0009727

In the current implementation, the reserve has the three columns
'value, fraction, currency' indicating the amount wired to the mint
for its creation. Since every withdrawal goes asynchronously, there
is no "central" point where it would be possible to detect whether
any withdrawal went fine. A solution might be that each withdrawal just
subtracts its amount from the reserve's. That way, if the amount drops
to zero, then the reserve's row could be deleted (or accordingly managed),
otherwise, the wallet maintains a reference to a non-emptied reserve
(still to be accordingly managed).

Christian Grothoff

2015-10-18 00:58

manager   ~0009740

Actually, that doesn't work as you don't know if the user deposited more than indicated, or did another deposit later. The safest thing to do is to have a timeout, i.e. reserve not used for x months, and then check /reserve/status and if it is zero and the reserve has been unused for like a year, then we can drop the column.

Florian Dold

2016-02-01 15:57

manager   ~0010124

This should already work in the WebEx wallet. It needs more testing though ...

Florian Dold

2016-02-19 00:54

manager   ~0010158

Reserves in the WebEx wallet are not directly visible in the UI, all coins are automatically withdrawn.

Once 0004126 and 0004188 are fixed, reserves will always be completely emptied (even with restarts inbetween) and then deleted.

Issue History

Date Modified Username Field Change
2015-10-13 22:55 Marcello Stanisci New Issue
2015-10-13 22:55 Marcello Stanisci Status new => assigned
2015-10-13 22:55 Marcello Stanisci Assigned To => Marcello Stanisci
2015-10-14 13:10 Marcello Stanisci Note Added: 0009727
2015-10-16 16:21 Marcello Stanisci Target Version => 0.1
2015-10-18 00:58 Christian Grothoff Note Added: 0009740
2015-10-23 01:13 Christian Grothoff Product Version => 0.0
2016-01-22 13:33 Marcello Stanisci Assigned To Marcello Stanisci => Florian Dold
2016-01-22 13:33 Marcello Stanisci Category wallet (browser-based) => wallet (WebExtensions)
2016-02-01 15:57 Florian Dold Note Added: 0010124
2016-02-19 00:54 Florian Dold Note Added: 0010158
2016-02-19 00:54 Florian Dold Status assigned => resolved
2016-02-19 00:54 Florian Dold Resolution open => no change required
2016-02-19 21:33 Christian Grothoff Fixed in Version => 0.0
2016-02-19 21:33 Christian Grothoff Target Version 0.1 => 0.0
2016-02-19 21:33 Christian Grothoff Status resolved => closed
2023-04-13 20:37 Florian Dold Category wallet (WebExtensions) => wallet (WebExtension)