View Issue Details

Related ChangesetsJump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0005174Talermechant backendpublic2017-11-08 19:512021-09-02 18:22
ReporterChristian Grothoff Assigned ToChristian Grothoff  
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.

Relationships

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

Activities

Christian Grothoff

2017-12-14 13:35

manager   ~0012680

Fixed and works as of 015dd3a..ee32930

Related Changesets

merchant: master 9d79c8c2

2017-12-13 22:50

Christian Grothoff


Details Diff		 adaptations to taler-merchant-generate-payments to use (fakebank) admin/add/incoming instead of exchange; not quite there yet. Also fixes 0005174 Affected Issues
0005174
mod - src/merchant-tools/taler-merchant-generate-payments.c Diff File

Issue History

Date Modified Username Field Change
2017-11-08 19:51 Christian Grothoff New Issue
2017-11-08 19:51 Christian Grothoff Status new => assigned
2017-11-08 19:51 Christian Grothoff Assigned To => Marcello Stanisci
2017-11-08 19:52 Christian Grothoff Relationship added child of 0005172
2017-11-08 19:52 Christian Grothoff Assigned To Marcello Stanisci =>
2017-11-08 19:52 Christian Grothoff Status assigned => confirmed
2017-11-08 19:52 Christian Grothoff Target Version => 0.7.1
2017-11-12 16:11 Christian Grothoff Assigned To => Christian Grothoff
2017-11-12 16:11 Christian Grothoff Status confirmed => assigned
2017-12-14 13:35 Christian Grothoff Status assigned => resolved
2017-12-14 13:35 Christian Grothoff Resolution open => fixed
2017-12-14 13:35 Christian Grothoff Fixed in Version => 0.5
2017-12-14 13:35 Christian Grothoff Note Added: 0012680
2017-12-14 13:35 Christian Grothoff Target Version 0.7.1 => 0.5
2018-04-15 20:34 Christian Grothoff Status resolved => closed
2021-09-02 18:22 Christian Grothoff Changeset attached => Taler-merchant master 9d79c8c2