View Issue Details

IDProjectCategoryView StatusLast Update
0009381Talerexchangepublic2024-12-13 18:15
ReporterFlorian Dold 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
PrioritynormalSeveritytextReproducibilityhave not tried
Status closedResolutionfixed 
Target Version0.14Fixed in Version0.14 
Summary0009381: lookup_deposit_wtid does not take reserves into consideration
Description... for the lookup of the target_pub.

That means in different parts of the exchange code, we have different notions of whether KYC auth is available or not.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Florian Dold

2024-12-11 16:31

manager   ~0023860

Two tests, *both* fail right now (even though one does the KYC auth transfer explicitly, and it ends up in the DB):

# Do auth explicitly via kyc auth ktransfer
$ taler-harness run-integrationtests kyc-deposit-aggregate

# Implicit auth via reserve.
$ taler-harness run-integrationtests kyc-deposit-aggregate-implicit-auth

Florian Dold

2024-12-11 16:50

manager   ~0023861

Fixed kyc-deposit-aggregate in b375ae8c.

The kyc-deposit-aggregate-implicit-auth test still fails as the query doesn't check the reserves table.

Christian Grothoff

2024-12-11 21:22

manager   ~0023867

API clarified in 30dda2f2..c9ba33eb.

Test still needs an update => Florian.

Florian Dold

2024-12-12 17:31

manager   ~0023874

Test fixed to take into account the previously unclear semantics of account_pub.

Issue History

Date ModifiedUsernameFieldChange
2024-12-11 15:21Florian DoldNew Issue
2024-12-11 15:21Florian DoldStatusnew => assigned
2024-12-11 15:21Florian DoldAssigned To => Christian Grothoff
2024-12-11 15:32Florian DoldDescription Updated
2024-12-11 16:31Florian DoldNote Added: 0023860
2024-12-11 16:50Florian DoldNote Added: 0023861
2024-12-11 20:57Christian GrothoffSeverityminor => text
2024-12-11 21:22Christian GrothoffStatusassigned => feedback
2024-12-11 21:22Christian GrothoffNote Added: 0023867
2024-12-11 22:37Christian GrothoffAssigned ToChristian Grothoff => Florian Dold
2024-12-12 17:31Florian DoldNote Added: 0023874
2024-12-12 17:31Florian DoldStatusfeedback => informing
2024-12-12 17:31Florian DoldResolutionopen => fixed
2024-12-13 00:59Christian GrothoffFixed in Version => 0.14
2024-12-13 18:15Christian GrothoffStatusinforming => closed