View Issue Details

IDProjectCategoryView StatusLast Update
0006540Talerwallet (TS core)public2020-08-28 17:49
ReporterFlorian Dold Assigned ToFlorian Dold  
PrioritynormalSeveritytweakReproducibilityhave not tried
Status assignedResolutionopen 
Summary0006540: generate more info for insufficient balance on exchange/auditor mismatch
DescriptionIn the "insufficient-balance" response, the wallet should additionally return:

(1) The current available balance in the requested currency
(2) The current available balance that the merchant would effectively accept considering auditor/exchange acceptance
(3) List of accepted exchanges and auditors

This should be sufficient to generate less confusing error messages when there is an auditor/exchange mismatch for the same currency.
TagsNo tags attached.


Christian Grothoff

2020-08-28 13:17

manager   ~0016785

I am not sure the above would be less confusing. I propose something simpler: *if* there is no auditor, then when reporting these kinds of issues, we always write ${CURRENCY}-${EXCHANGE_URL} to properly identify the exchange that issued the currency (for the amount held by the wallet). Similarly, for merchants, we should add the list of accepted exchanges (or auditors) as you propose.

For *audited* currencies (exchange or merchant) we should very explicitly include text in the error message that points out that the set of trusted auditors does not intersect.

Florian Dold

2020-08-28 14:39

manager   ~0016786

Regarding unaudited/regional currencies: Agreed, we've already discussed this and it's tracked in 0006448, but scheduled post-0.8

For audited currencies, I can simply improve the error message text, and give both sets (merchant-trusted and wallet-trusted auditors for the currency) in the "insufficient-balance" response.

So I guess I agree, the additional balance info isn't really required.

Issue History

Date Modified Username Field Change
2020-08-28 11:05 Florian Dold New Issue
2020-08-28 11:05 Florian Dold Status new => assigned
2020-08-28 11:05 Florian Dold Assigned To => Florian Dold
2020-08-28 13:17 Christian Grothoff Note Added: 0016785
2020-08-28 14:39 Florian Dold Note Added: 0016786
2020-08-28 17:49 Christian Grothoff Severity minor => tweak