View Issue Details

IDProjectCategoryView StatusLast Update
0005158Talermerchant backendpublic2018-04-15 18:36
ReporterChristian Grothoff 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
PriorityhighSeveritymajorReproducibilityN/A
Status closedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Versiongit (master) 
Target Version0.5Fixed in Version0.5 
Summary0005158: error handling for /pay not comprehensive
DescriptionSpecifically, what happens if we store in merchant_transactions some timestamp/exchange and then the deposits (partially) fail?

Do we undo the deposits? (auto-refund? how do we tell the wallet?)

Do we allow a 2nd round of /pay? (likely case if wallet was restored from backup causing double-spending; what does the wallet do?)

What happens if the exchange differs the 2nd time? (maybe the wallet has another exchange we _also_ accept this time?)
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Relationships

related to 0005206 closedFlorian Dold wallet / merchant should negotiate refunds on partially failed payments 

Activities

Christian Grothoff

2017-12-28 16:26

manager   ~0012707

Implementation and API spec is in principle now done, but testcase is missing.

Christian Grothoff

2018-01-04 11:00

manager   ~0012723

Tested as of 6b2297c..cc367e6 in merchant. So what is left is for the wallet to support this.

Florian Dold

2018-01-29 23:57

manager   ~0012829

Implemented in the wallet in 1a66e232a55df

Issue History

Date ModifiedUsernameFieldChange
2017-10-23 08:30Christian GrothoffNew Issue
2017-10-23 08:30Christian GrothoffStatusnew => assigned
2017-10-23 08:30Christian GrothoffAssigned To => Christian Grothoff
2017-11-04 17:05Christian GrothoffTarget Version0.7.1 => 0.6
2017-11-04 17:56Christian GrothoffPrioritynormal => high
2017-12-14 14:48Christian GrothoffRelationship addedrelated to 0005206
2017-12-28 16:26Christian GrothoffNote Added: 0012707
2018-01-04 11:00Christian GrothoffAssigned ToChristian Grothoff => Florian Dold
2018-01-04 11:00Christian GrothoffNote Added: 0012723
2018-01-15 20:13Christian GrothoffTarget Version0.6 => 0.5
2018-01-29 23:57Florian DoldStatusassigned => informing
2018-01-29 23:57Florian DoldResolutionopen => fixed
2018-01-29 23:57Florian DoldNote Added: 0012829
2018-04-15 18:35Christian GrothoffFixed in Version => 0.5
2018-04-15 18:36Christian GrothoffStatusinforming => closed
2025-03-22 13:09Christian GrothoffCategorymechant backend => merchant backend