View Issue Details

IDProjectCategoryView StatusLast Update
0009197Talermerchant backendpublic2024-12-13 18:15
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
PriorityhighSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Versiongit (master) 
Target Version0.14Fixed in Version0.14 
Summary0009197: merchant order creation hangs when the only configured wire account is not supported by the exchange
Description$ taler-harness run-integrationtests wallet-balance

OId, expected behavior: Merchant creates the order, wallet complains about insufficient balance when trying to pay (no matching wire account between exchange and merchant).

Current behavior: Merchant order creation request hangs.

Question is, what is the *correct* behavior? Should the merchant outright refuse to create orders? What if the exchange that *would* support the configured wire account is just temporarily unavailable?
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2024-10-01 12:07

manager   ~0023415

12c8aeb66e1a1f8dc8b4cefa78c11459e6a657d3 changes the merchant code to make it give up after 250ms on /keys from exchanges. So the order creation will now fail -- but not take "forever" -- with a status code depending on what happens if we do not have /keys from (some of) the exchanges. The specific tests fails with a 409 as we don't have an exchange that could process the order.

Christian Grothoff

2024-10-01 12:22

manager   ~0023416

Assigning to Florian as test still fails...

Florian Dold

2024-10-28 11:18

manager   ~0023606

commit 9d2d254e7d0bbfc6112544ea3079621cb6d7a647 (HEAD -> master, origin/master, origin/HEAD)
Author: Florian Dold <florian@dold.me>
Date: Mon Oct 28 12:18:01 2024 +0100

    harness: fix wallet-balance test by using updated merchant API

Issue History

Date ModifiedUsernameFieldChange
2024-09-17 17:14Florian DoldNew Issue
2024-09-17 17:14Florian DoldStatusnew => assigned
2024-09-17 17:14Florian DoldAssigned To => Christian Grothoff
2024-10-01 12:07Christian GrothoffNote Added: 0023415
2024-10-01 12:21Christian GrothoffAssigned ToChristian Grothoff => Florian Dold
2024-10-01 12:22Christian GrothoffNote Added: 0023416
2024-10-28 11:18Florian DoldNote Added: 0023606
2024-10-28 11:18Florian DoldStatusassigned => informing
2024-10-28 11:18Florian DoldResolutionopen => fixed
2024-10-28 11:20Christian GrothoffProduct Version => git (master)
2024-10-28 11:20Christian GrothoffFixed in Version => 0.14
2024-10-28 11:20Christian GrothoffTarget Version1.0 => 0.14
2024-12-13 18:15Christian GrothoffStatusinforming => closed
2025-03-22 13:09Christian GrothoffCategorymechant backend => merchant backend