View Issue Details

IDProjectCategoryView StatusLast Update
0007438Talerwallet-corepublic2023-01-26 21:53
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
PriorityhighSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Versiongit (master) 
Target Version0.9.1Fixed in Version0.9.1 
Summary0007438: failing P2P payments result in money being lost
DescriptionAttempt to send money to another wallet ends up with 'unexpected exception', 'got error response from exchange' when exchange has wrong base_url configured; FUNDS ARE LOST (coins are marked as spent in wallet DB and not refreshed despite 403 error from exchange!).

There may be other cases where failed P2P payments loose money. In particular, we should check that a wallet doing a push payment always attempts a refresh after the push payment expired.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Relationships

related to 0007424 closedFlorian Dold peer-to-peer payment operations should do retries and error handling 

Activities

Florian Dold

2023-01-13 11:39

manager   ~0019655

Retries are implemented now, so a failed purse creation request won't result in lost money.

Aborting a P2P payment is a different issue: 0007584 / 0007583

The UI of course also needs to support this: 0007568

Issue History

Date ModifiedUsernameFieldChange
2022-11-06 21:52Christian GrothoffNew Issue
2022-11-06 21:52Christian GrothoffStatusnew => assigned
2022-11-06 21:52Christian GrothoffAssigned To => Florian Dold
2022-11-06 22:15Christian GrothoffRelationship addedrelated to 0007424
2023-01-13 01:37Florian DoldStatusassigned => informing
2023-01-13 01:37Florian DoldResolutionopen => fixed
2023-01-13 11:39Florian DoldNote Added: 0019655
2023-01-23 21:25Christian GrothoffFixed in Version => 0.9.1
2023-01-26 21:53Christian GrothoffStatusinforming => closed
2023-04-13 18:36Florian DoldCategorywallet (TS core) => wallet-core