View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0010907 | Taler | merchant backend | public | 2026-01-22 15:44 | 2026-01-23 03:10 |
| Reporter | sebasjm | Assigned To | sebasjm | ||
| Priority | high | Severity | major | Reproducibility | have not tried |
| Status | feedback | Resolution | open | ||
| Product Version | git (master) | ||||
| Target Version | 1.4 | ||||
| Summary | 0010907: can create order on demo after adding bank accoutn | ||||
| Description | I created my account with self-provision (sebasjm) added a fake bank account I expect to be able to create order after that What happens is that KYC state is triggered even if exchange demo does not require KYC the kyc page of the backoffice shows "exchange unreachable" so I think we are doing 2 things wrong: 1) assuming that KYC is required: this confuse the user 2) not triggering exchange contact just after and ON-SYNC with the bank account addition. The exchange should be contact to validate that the this bank account is supported. | ||||
| Additional Information | { "kyc_data": [ { "payto_uri": "payto://iban/DE1231231231?receiver-name=seba", "h_wire": "8R76BJZGF7EA8JY507YKTB106NMR1G4HYQFYSGXQJ252GFQNFA4RT43PJ2Q97SAYC9P3DBEKWV3GJPNKQ884MJY2TJB3TT3BYJMEQD0", "status": "exchange-unreachable", "exchange_url": "https://exchange.taler-ops.ch/", "no_keys": true, "auth_conflict": false, "exchange_http_status": 0 }, { "payto_uri": "payto://iban/DE1231231231?receiver-name=seba", "h_wire": "8R76BJZGF7EA8JY507YKTB106NMR1G4HYQFYSGXQJ252GFQNFA4RT43PJ2Q97SAYC9P3DBEKWV3GJPNKQ884MJY2TJB3TT3BYJMEQD0", "status": "exchange-unreachable", "exchange_url": "https://exchange.demo.taler.net/", "no_keys": true, "auth_conflict": false, "exchange_http_status": 0 } ] } | ||||
| Tags | No tags attached. | ||||
| Attached Files | |||||
|
|
this is related https://ich.taler.net/t/cant-connect-selfhosted-merchant-to-official-kudos-exchange/622 |
|
|
my demo instance is still in this state, so something seems to be broken. the merchant may not be checking the connection to the exchaneg |
|
|
increasing priority since this may mean that non self provisioning instances are working |
|
|
we found an unrecoverable state in logs which is not treated as error Jan 22 16:18:50 d0f55933210e taler-merchant-kyccheck[1616621]: INFO Found account payto://iban/DE1231231231?receiver-name=seba of instance sebasjm with H_PAYTO RVMJAC Jan 22 16:18:50 d0f55933210e taler-merchant-kyccheck[1616621]: INFO Account payto://iban/DE1231231231?receiver-name=seba not eligible at exchange https://exchange.taler-ops.ch/ Jan 22 16:18:50 d0f55933210e taler-merchant-kyccheck[1616621]: INFO Account payto://iban/DE1231231231?receiver-name=seba not eligible at exchange https://exchange.demo.taler.net/ |
|
|
Can we have a taler-harness test? |
|
|
yes # taler-harness run-integrationtests merchant-bank-bad-wire-target |
|
|
I'm not sure what the report is about anymore. The issue the test surfaces is that you have an "IBAN" account and a TESTKUDOS currency with an x-taler-bank exchange. So that cannot work. I updated the POST /private/orders specification to extend the 451 error response with details about every exchange why it did not work. Note that nothing changes if the amount is simply too high, but if exchanges were not considered at all you get to see exactly why (currency, no keys, no wire method, zero deposit limit). Anyway, this at best should improve diagnostics. Second, with respect to that "unrecoverable state in logs", I again don't see how it is "unrecoverable": adding an exchange that supports the account should immediately make it work. Anyway, I did now modify the code to *expose* the status more clearly that the account is not supported by a particular exchange, so you now get a new "unsupported-account" case in the /kyc status response for the respective account and exchange (which was previously not communicated). Please clarify if anything else is required and otherwise adjust the test to make it pass... |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2026-01-22 15:44 | sebasjm | New Issue | |
| 2026-01-22 15:44 | sebasjm | Status | new => assigned |
| 2026-01-22 15:44 | sebasjm | Assigned To | => Christian Grothoff |
| 2026-01-22 15:44 | sebasjm | File Added: image.png | |
| 2026-01-22 15:44 | sebasjm | Note Added: 0027309 | |
| 2026-01-22 17:14 | sebasjm | Note Added: 0027312 | |
| 2026-01-22 17:14 | sebasjm | Priority | normal => high |
| 2026-01-22 17:15 | sebasjm | Note Added: 0027313 | |
| 2026-01-22 18:21 | sebasjm | Note Added: 0027314 | |
| 2026-01-23 00:30 | Christian Grothoff | Note Added: 0027316 | |
| 2026-01-23 00:58 | sebasjm | Note Added: 0027317 | |
| 2026-01-23 03:10 | Christian Grothoff | Note Added: 0027318 | |
| 2026-01-23 03:10 | Christian Grothoff | Assigned To | Christian Grothoff => sebasjm |
| 2026-01-23 03:10 | Christian Grothoff | Status | assigned => feedback |
| 2026-01-23 03:10 | Christian Grothoff | Product Version | => git (master) |
| 2026-01-23 03:10 | Christian Grothoff | Target Version | => 1.4 |