View Issue Details

IDProjectCategoryView StatusLast Update
0003973Talerwallet (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
PrioritynormalSeverityfeatureReproducibilityN/A
Status closedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Version0.1 
Target Version0.3Fixed in Version0.3 
Summary0003973: wallet needs to check exchange satisfies merchant requirements
DescriptionThe merchant may impose constraints on the set of mints it is willing to do business with (cost, auditor). The wallet needs to check if its available coins meet those requirements.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Relationships

related to 0004027 closedMarcello Stanisci Provide multi auditors, multi exchange feature 

Activities

Florian Dold

2016-03-02 01:19

manager   ~0010209

Not sure what this is supposed to mean ... the wallet already checks if enough coins of a suitable denomination are available.

Christian Grothoff

2016-03-02 01:39

manager   ~0010212

The contract from the merchant may specify a set of acceptable auditors and/or exchanges. Once we have multiple exchanges, it is conceivable that some are not acceptable to the merchant, even if the fee structure is OK, i.e. because they are not sufficiently audited. So we need to not only check if we have enough coins, but also if we have enough coins from acceptable exchanges. Furthermore, right now, all coins must be from the same exchange (we could lift this restriction in the future).

Florian Dold

2016-03-02 01:41

manager   ~0010213

I see.

The wallet already does what you describe, except for the auditor part.

Christian Grothoff

2016-03-02 03:10

manager   ~0010214

Yes, the auditor part was what this was mostly about.

Christian Grothoff

2017-02-01 12:52

manager   ~0011684

We also need a new fee (in /keys, in contract), which is the maximum (or actual) aggregation fee that the exchange charges per aggregate wire transfer (as we learned that can be a significant expense).

Florian Dold

2017-04-26 12:13

manager   ~0012070

Fixed in 3c563c7, the wallet's coin selection now considers the auditors that the merchant's proposal terms include.

Issue History

Date ModifiedUsernameFieldChange
2015-09-17 12:17Christian GrothoffNew Issue
2015-09-17 12:17Christian GrothoffStatusnew => implementing
2015-09-19 18:49Christian GrothoffTarget Version0.1 => 0.3
2015-11-17 13:21Christian GrothoffAssigned To => tg
2015-11-17 13:21Christian GrothoffStatusimplementing => assigned
2015-11-17 14:14Christian GrothoffTarget Version0.3 => 0.2
2015-11-29 12:52Christian GrothoffCategorymerchant API (HTTP-based) => merchant backend API (HTTP specification)
2015-11-29 12:58Christian GrothoffCategorymerchant backend API (HTTP specification) => wallet (browser-based)
2016-01-22 22:29Christian GrothoffCategorywallet (browser-based) => wallet (WebExtensions)
2016-03-01 13:46Christian GrothoffAssigned Totg => Florian Dold
2016-03-02 01:19Florian DoldNote Added: 0010209
2016-03-02 01:19Florian DoldStatusassigned => feedback
2016-03-02 01:39Christian GrothoffNote Added: 0010212
2016-03-02 01:39Christian GrothoffStatusfeedback => assigned
2016-03-02 01:41Florian DoldNote Added: 0010213
2016-03-02 03:10Christian GrothoffNote Added: 0010214
2016-03-20 20:53Torsten GrothoffIssue cloned: 0004272
2016-10-10 06:40Christian GrothoffSummarywallet needs to check mint satisfies merchant requirements => wallet needs to check exchange satisfies merchant requirements
2016-10-17 22:27Florian DoldTarget Version0.2 => 0.3
2017-01-13 15:11Christian GrothoffRelationship addedrelated to 0004027
2017-02-01 12:52Christian GrothoffNote Added: 0011684
2017-04-26 12:13Florian DoldStatusassigned => informing
2017-04-26 12:13Florian DoldResolutionopen => fixed
2017-04-26 12:13Florian DoldNote Added: 0012070
2017-05-03 00:16Christian GrothoffFixed in Version => 0.3
2017-06-06 12:18Christian GrothoffStatusinforming => closed
2023-04-13 18:37Florian DoldCategorywallet (WebExtensions) => wallet (WebExtension)