View Issue Details

IDProjectCategoryView StatusLast Update
0009966Talerwallet-corepublic2025-08-31 17:09
ReporterBohdan 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
PrioritynormalSeverityminorReproducibilityrandom
Status closedResolutionfixed 
Product Version1.0 
Target Version1.0 stretch goalsFixed in Version1.0 stretch goals 
Summary0009966: Apparently wallet core can try to claim the order which was already paid [3h]
DescriptionI believe the issue comes from the fact that I have tried to claim the order several times because of errors, and when I finally succeeded and paid it, the order ID that was in the DB to be retried with the same order id has not been removed from the retry state and still asks to claim it... every 5 minutes

I also believe this could be a root of the problem if the user tries to make a refund for such an order, and it will fail...
Tagsvivy

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Florian Dold

2025-05-15 14:35

manager   ~0024919

Is this reproducible?

Bohdan

2025-05-15 15:16

developer   ~0024921

I believe yes... especially with dev setup of wallet core

To reproduce it myself, I need to make a faulty merchant than do a couple of requests where merchant will return error, or rather to fully reproduce the exchange must return error that merchant can't get the money deposited(the problem of iban and x-taler-bank), and then try to pay it one more time...

Bohdan

2025-05-15 15:17

developer   ~0024922

Strong feeling that same order id can be in list of failed and to be retried, and in the same time to be actually paid, and not removed from the other list

Florian Dold

2025-08-02 09:45

manager   ~0025609

Still could not reproduce it, but the following commit could in theory fix it (not yet available in published wallets).
If the problem persists, please re-report with instructions on how to reproduce.

commit 6df6fd6ee7c9835956f861354cfd90b789a79df1 (HEAD -> master)
Author: Florian Dold <florian@dold.me>
Date: Sat Aug 2 11:44:12 2025 +0200

    wallet-core: fix matching of existing purchase transactions
    
    Should fix https://bugs.taler.net/9966

Issue History

Date ModifiedUsernameFieldChange
2025-05-15 13:49BohdanNew Issue
2025-05-15 14:35Florian DoldNote Added: 0024919
2025-05-15 15:16BohdanNote Added: 0024921
2025-05-15 15:17BohdanNote Added: 0024922
2025-05-15 19:56Christian GrothoffAssigned To => Florian Dold
2025-05-15 19:56Christian GrothoffStatusnew => assigned
2025-05-19 17:32Florian DoldProduct Version1.0 => 1.1
2025-05-19 17:32Florian DoldProduct Version1.1 =>
2025-05-19 17:32Florian DoldTarget Version => 1.1
2025-06-05 11:58Florian DoldSummaryApparently wallet core can try to claim the order which was already paid => Apparently wallet core can try to claim the order which was already paid [3h]
2025-07-30 09:29Christian GrothoffTag Attached: vivy
2025-08-02 09:45Florian DoldNote Added: 0025609
2025-08-02 09:46Florian DoldStatusassigned => informing
2025-08-02 09:46Florian DoldResolutionopen => fixed
2025-08-04 20:24Christian GrothoffProduct Version => 1.0
2025-08-04 20:24Christian GrothoffFixed in Version => 1.0 stretch goals
2025-08-04 20:24Christian GrothoffTarget Version1.1 => 1.0 stretch goals
2025-08-31 17:09Christian GrothoffStatusinforming => closed