View Issue Details

IDProjectCategoryView StatusLast Update
0009180Talerwallet-corepublic2024-12-13 18:15
ReporterMarcS 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
PriorityhighSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Product Version0.14 
Target Version0.14Fixed in Version0.14 
Summary0009180: checkPeerPushDebit needs scopeInfo parameter
DescriptionCurrently checkPeerPushDebit has only the amount as mandatory parameter (which just has the currency | ISO4217 designation).
The optional exchangeBaseUrl parameter is only to select a "Preferred Exchange".
When multiple Exchanges by sheer coincidence use the same name for a regional currency (such as demo.taler.net and head.taler.net, which both use KUDOS), wallet-core adds the balances and reports wrong values.

checkPeerPushDebit should have scopeInfo as second, mandatory parameter, then wallet-core can tell the currencies apart.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Florian Dold

2024-09-11 14:30

manager   ~0023242

Both checkPeerPushDebit and initiatePeerPushDebit now support the restrictScope arguments.

commit f401ab076bb037e46c4b12e371dfe13a777bf582 (HEAD -> master, origin/master, origin/HEAD)
Author: Florian Dold <florian@dold.me>
Date: Wed Sep 11 16:29:50 2024 +0200

    wallet-core: support restricting scope for peer push payments

Issue History

Date ModifiedUsernameFieldChange
2024-09-11 09:00MarcSNew Issue
2024-09-11 09:00MarcSStatusnew => assigned
2024-09-11 09:00MarcSAssigned To => Florian Dold
2024-09-11 14:30Florian DoldStatusassigned => informing
2024-09-11 14:30Florian DoldResolutionopen => fixed
2024-09-11 14:30Florian DoldNote Added: 0023242
2024-09-13 23:08Christian GrothoffFixed in Version => 0.14
2024-12-13 18:15Christian GrothoffStatusinforming => closed