View Issue Details

IDProjectCategoryView StatusLast Update
0005174Talermerchant backendpublic2021-09-02 16:22
ReporterChristian Grothoff Assigned ToChristian Grothoff 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
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Versiongit (master) 
Target Version0.5Fixed in Version0.5 
Summary0005174: UUIDs of taler-merchant-generate-payments are not unique, but always 1
DescriptionThis rightfully upsets the taler-auditor, when run against a DB primed by the payments generator. The code should be changed to not hard-code UUIDs, but instead each call to the 'admin add incoming' API should on-the-fly generate a 'fresh' UUID (i.e. by incrementing a counter).

Note that there is also 0005172, which deprecates the exchange's admin/add/incoming. So the next-generation of this code should use the *banks* API for triggering a wire transfer instead (see TALER_BANK_admin_add_incoming(), the library 'taler_bank_service.h' for this is in the exchange code).
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Relationships

child of 0005172 closedChristian Grothoff Eliminate /admin/add/incoming 

Activities

Christian Grothoff

2017-12-14 12:35

manager   ~0012680

Fixed and works as of 015dd3a..ee32930

Issue History

Date ModifiedUsernameFieldChange
2017-11-08 18:51Christian GrothoffNew Issue
2017-11-08 18:51Christian GrothoffStatusnew => assigned
2017-11-08 18:51Christian GrothoffAssigned To => Marcello Stanisci
2017-11-08 18:52Christian GrothoffRelationship addedchild of 0005172
2017-11-08 18:52Christian GrothoffAssigned ToMarcello Stanisci =>
2017-11-08 18:52Christian GrothoffStatusassigned => implementing
2017-11-08 18:52Christian GrothoffTarget Version => 0.7.1
2017-11-12 15:11Christian GrothoffAssigned To => Christian Grothoff
2017-11-12 15:11Christian GrothoffStatusimplementing => assigned
2017-12-14 12:35Christian GrothoffStatusassigned => informing
2017-12-14 12:35Christian GrothoffResolutionopen => fixed
2017-12-14 12:35Christian GrothoffFixed in Version => 0.5
2017-12-14 12:35Christian GrothoffNote Added: 0012680
2017-12-14 12:35Christian GrothoffTarget Version0.7.1 => 0.5
2018-04-15 18:34Christian GrothoffStatusinforming => closed
2021-09-02 16:22Christian Grothoff
2025-03-22 13:09Christian GrothoffCategorymechant backend => merchant backend