View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0011657 | Taler | wallet-core | public | 2026-07-20 15:02 | 2026-07-28 12:10 |
| Reporter | Florian Dold | Assigned To | Florian Dold | ||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | resolved | Resolution | fixed | ||
| Target Version | 1.7 | ||||
| Summary | 0011657: add integration test for balance before/after aborted p2p payments | ||||
| Description | The 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. | ||||
| Tags | No tags attached. | ||||
|
|
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. |
|
|
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. |
| 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 |