View Issue Details

IDProjectCategoryView StatusLast Update
0011657Talerwallet-corepublic2026-07-28 12:10
ReporterFlorian Dold Assigned ToFlorian Dold  
PrioritynormalSeverityminorReproducibilityhave not tried
Status resolvedResolutionfixed 
Target Version1.7 
Summary0011657: add integration test for balance before/after aborted p2p payments
DescriptionThe underlying bug should be fixed, but we don't have a good integration test that checks the balance and that no money was lost yet.
TagsNo tags attached.

Activities

Florian Dold

2026-07-28 12:09

manager   ~0029246

commit 95e70cad7a1d56ee001064fad3168a78495aef62 (HEAD -> master, origin/master, origin/HEAD, durian/master, durian/HEAD)
Author: Florian Dold <dold@taler.net>
Date: Tue Jul 28 12:00:05 2026 +0200

    harness: check wallet balances around aborted p2p payments
    
    The exchange runs without fees, so an abort that gives the money back has to
    restore the balance exactly.

Florian Dold

2026-07-28 12:10

manager   ~0029247

This did actually find a bug in the aborts, fixed here:

commit 00bdbfe58348ed4d6cb70c85c656c86b2e2cd3fe
Author: Florian Dold <dold@taler.net>
Date: Tue Jul 28 11:59:59 2026 +0200

    wallet: refresh the payer's coins when a peer-pull-debit expires
    
    The coins are allocated when the invoice is accepted, so expiring without a
    refresh left them stuck on a dead transaction and the money was lost.

Issue History

Date Modified Username Field Change
2026-07-20 15:02 Florian Dold New Issue
2026-07-20 15:02 Florian Dold Status new => assigned
2026-07-20 15:02 Florian Dold Assigned To => Florian Dold
2026-07-28 12:09 Florian Dold Note Added: 0029246
2026-07-28 12:10 Florian Dold Note Added: 0029247
2026-07-28 12:10 Florian Dold Status assigned => resolved
2026-07-28 12:10 Florian Dold Resolution open => fixed