View Issue Details

IDProjectCategoryView StatusLast Update
0005172Talerspecificationpublic2024-01-12 13:03
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
PrioritynormalSeverityfeatureReproducibilityN/A
Status closedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Versiongit (master) 
Target Version0.5Fixed in Version0.5 
Summary0005172: Eliminate /admin/add/incoming
DescriptionThis API is bad style. In practice, no bank is going to notify _us_. We expect to have taler-exchange-wirewatch poll the bank and notify the exchange whenever there is a wire transfer (just like the aggregator does the outgoing transfers), and we should use wirewatch also with the Taler bank.

The bank now does have (the beginnings of?) a /history API to obtain transactions. We should make sure the 'test' plugin works with /history, get wirewatch to work with the Taler bank, and eliminate /admin/add/incoming.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

The plan

IDSummaryStatusAssigned ToEstimated effort (hours)
0005174UUIDs of taler-merchant-generate-payments are not unique, but always 1 closedChristian Grothoff
This issue (hours)
Its children (hours)0
Together (hours)0
Nothing is missing: the plan can be finished.

Relationships

parent of 0005174 closedChristian Grothoff UUIDs of taler-merchant-generate-payments are not unique, but always 1 
related to 0005191 closedMarcello Stanisci Python bank needs to implement /reject API (and related updates) 
related to 0005211 closedMarcello Stanisci deployment should use taler-exchange-wirewatch 

Activities

Christian Grothoff

2017-12-10 15:03

manager   ~0012653

d36de3b..6623afa removes the need for /admin/add/incoming from the exchange-lib testcase. exchange-benchmark, merchant-lib-test and merchant-benchmark are the remaining users.

Christian Grothoff

2017-12-10 20:43

manager   ~0012659

Done removing need for the API, except for taler-merchant-generate-payments, which still uses /admin/add/incoming. Addressing that one will also implicitly fix 0005174 ;-)

Christian Grothoff

2017-12-14 12:36

manager   ~0012681

Now just blocked on 0005211.

Christian Grothoff

2017-12-14 12:50

manager   ~0012682

Patch is ready, waiting on 0005211 before pushing it.

Christian Grothoff

2017-12-14 15:03

manager   ~0012690

229907c6e233609ccfafe1823f281581bf5e9680

Christian Grothoff

2021-09-02 16:14

manager   ~0018288

Fix committed to master branch.

Issue History

Date ModifiedUsernameFieldChange
2017-11-04 17:48Christian GrothoffNew Issue
2017-11-04 17:48Christian GrothoffStatusnew => assigned
2017-11-04 17:48Christian GrothoffAssigned To => Christian Grothoff
2017-11-04 17:49Christian GrothoffTarget Version => 0.7.1
2017-11-08 18:52Christian GrothoffRelationship addedparent of 0005174
2017-12-06 11:47Christian GrothoffRelationship addedrelated to 0005191
2017-12-10 15:03Christian GrothoffNote Added: 0012653
2017-12-10 20:43Christian GrothoffNote Added: 0012659
2017-12-10 20:46Christian GrothoffRelationship addedrelated to 0005211
2017-12-14 12:36Christian GrothoffNote Added: 0012681
2017-12-14 12:50Christian GrothoffTarget Version0.7.1 => 0.5
2017-12-14 12:50Christian GrothoffNote Added: 0012682
2017-12-14 15:03Christian GrothoffStatusassigned => informing
2017-12-14 15:03Christian GrothoffResolutionopen => fixed
2017-12-14 15:03Christian GrothoffFixed in Version => 0.5
2017-12-14 15:03Christian GrothoffNote Added: 0012690
2018-04-15 18:34Christian GrothoffStatusinforming => closed
2021-09-02 16:13Christian Grothoff
2021-09-02 16:14Christian GrothoffNote Added: 0018288
2024-01-12 13:03Christian GrothoffCategoryexchange API (HTTP specification) => specification