View Issue Details

IDProjectCategoryView StatusLast Update
0010162Talerwallet-corepublic2025-08-31 17:09
ReporterFlorian Dold 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
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version1.0 
Target Version1.0 stretch goalsFixed in Version1.0 stretch goals 
Summary0010162: wallet-core should limit batch size of large deposit transactions [3h]
DescriptionExchange supports only a batch size of 64, but wallet-core tries larger sizes.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Florian Dold

2025-07-10 22:53

manager   ~0025490

Additionally, when aborting a deposit transaction (one that fails because of this very issue!), we don't store immediate results of refunds in the database. That means for a large deposit group, we run into throttling but won't ever recover properly, since potentially hundreds of refund requests would need to succeed in immediate succession.

Florian Dold

2025-07-14 21:20

manager   ~0025502

commit 3847b84436df8985b6143606e7eeced2141e9baa (HEAD -> master, origin/master, origin/HEAD)
Author: Florian Dold <florian@dold.me>
Date: Mon Jul 14 23:19:26 2025 +0200

    wallet-core: split deposit batches
    
    The exchange only accepts a limited number of deposits per batch
    deposit request, so we need to split batches.

Issue History

Date ModifiedUsernameFieldChange
2025-07-08 20:41Florian DoldNew Issue
2025-07-08 20:41Florian DoldStatusnew => assigned
2025-07-08 20:41Florian DoldAssigned To => Florian Dold
2025-07-08 23:58Florian DoldSummarywallet-core should batch large deposit transactions => wallet-core should limit batch size of large deposit transactions
2025-07-08 23:58Florian DoldDescription Updated
2025-07-08 23:58Florian DoldSummarywallet-core should limit batch size of large deposit transactions => wallet-core should limit batch size of large deposit transactions [3h]
2025-07-10 22:53Florian DoldNote Added: 0025490
2025-07-14 21:20Florian DoldStatusassigned => informing
2025-07-14 21:20Florian DoldResolutionopen => fixed
2025-07-14 21:20Florian DoldNote Added: 0025502
2025-07-15 00:31Christian GrothoffProduct Version => 1.0
2025-07-15 00:31Christian GrothoffFixed in Version => 1.0 stretch goals
2025-08-31 17:09Christian GrothoffStatusinforming => closed