View Issue Details

IDProjectCategoryView StatusLast Update
0009974Talermerchant backendpublic2025-08-31 17:09
ReporterChristian Grothoff Assigned ToChristian Grothoff 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
PriorityurgentSeverityblockReproducibilityalways
Status closedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Version1.0 
Target Version1.0 stretch goalsFixed in Version1.0 stretch goals 
Summary0009974: payments fail in BFH deployment
DescriptionIt's completely unclear why, but somehow the deposit signatures of the wallet are not accepted by the (recently upgraded) merchant backend. They don't even reach the exchange.

I've checked, and it seems the contract hash may be different between wallet and backend (even though the JSON seems the same), but that still does not fully explain it because even if I manually hack the right contract hash into the backend the signature verification still fails.

Steps To ReproduceCreate order in the merchant backend, try to pay it.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

The plan

IDSummaryStatusAssigned ToEstimated effort (hours)
0009977upload Android wallet with wallet-core 1.0.5 closedavalos
0009978upload iOS wallet with wallet-core 1.0.4 closedMarcS
This issue (hours)
Its children (hours)0
Together (hours)0
Nothing is missing: the plan can be finished.

Relationships

related to 0009976 closedFlorian Dold upload webext wallet with wallet-core 1.0.4 
parent of 0009978 closedMarcS upload iOS wallet with wallet-core 1.0.4 
parent of 0009977 closedavalos upload Android wallet with wallet-core 1.0.5 
child of 0009940 closedChristian Grothoff migrate pre 1.0 deployments to 1.0 setup 

Activities

Christian Grothoff

2025-05-18 12:53

manager   ~0024926

Urgent not only because BFH deployment is down/not working, but also because we really need to figure this one out before migrating Netzbon!

Florian Dold

2025-05-19 13:08

manager   ~0024928

commit 6085e8f811f2ce2067213fbf6a7eb5a7527116d1 (HEAD -> master, origin/master, origin/HEAD)
Author: Florian Dold <florian@dold.me>
Date: Mon May 19 15:07:43 2025 +0200

    wallet: fix handling of extra contract terms fields
    
    Also adds a test.
    
    Issue: https://bugs.taler.net/n/9974

Florian Dold

2025-05-19 13:10

manager   ~0024929

Last edited: 2025-05-19 17:37

Leaving this open (but unassigned) until the wallets with wallet-core v1.0.4 are uploaded.

Christian Grothoff

2025-05-25 10:20

manager   ~0024985

Works now ;-).

Issue History

Date ModifiedUsernameFieldChange
2025-05-18 12:47Christian GrothoffNew Issue
2025-05-18 12:47Christian GrothoffStatusnew => assigned
2025-05-18 12:47Christian GrothoffAssigned To => Florian Dold
2025-05-18 12:52Christian GrothoffRelationship addedchild of 0009940
2025-05-18 12:53Christian GrothoffNote Added: 0024926
2025-05-19 13:08Florian DoldNote Added: 0024928
2025-05-19 13:10Florian DoldNote Added: 0024929
2025-05-19 17:36Florian DoldRelationship addedparent of 0009978
2025-05-19 17:36Florian DoldRelationship addedparent of 0009977
2025-05-19 17:36Florian DoldRelationship addedrelated to 0009976
2025-05-19 17:37Florian DoldNote Edited: 0024929
2025-05-19 17:37Florian DoldAssigned ToFlorian Dold =>
2025-05-25 10:20Christian GrothoffAssigned To => Christian Grothoff
2025-05-25 10:20Christian GrothoffStatusassigned => informing
2025-05-25 10:20Christian GrothoffResolutionopen => fixed
2025-05-25 10:20Christian GrothoffFixed in Version => 1.0 stretch goals
2025-05-25 10:20Christian GrothoffNote Added: 0024985
2025-08-31 17:09Christian GrothoffStatusinforming => closed