View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0009183 | Taler | mechant backend | public | 2024-09-11 15:35 | 2024-09-17 18:24 |
| Reporter | Florian Dold | Assigned To | Florian Dold | ||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | resolved | Resolution | fixed | ||
| Target Version | 0.14 | ||||
| Summary | 0009183: merchant backend reports unexpected error code when wallet deposits coins with bad exchange base URL | ||||
| Description | The merchant is configured to accept exchange base URL with master pub K1 and base URL B1. The wallet, however, is configured to use the same exchange with K1 but a different base URL B2. The old merchant handled this scenario perfectly well. It's fine for the updated merchant (with the helper processes) to not handle this scenario, but the error response should not be a 5xx but a 4xx. | ||||
| Steps To Reproduce | $ taler-harness run-integrationtests merchant-exchange-confusion | ||||
| Additional Information | 2024-09-11T14:41:57.372496+0200 taler-merchant-httpd-128454(5Y7TEGP77AY77ZB23XEPDVA78G) INFO Processing payment with exchange http://localhost:9081/ 2024-09-11T14:41:57.372503+0200 taler-merchant-httpd-128454(5Y7TEGP77AY77ZB23XEPDVA78G) WARNING External protocol violation detected at taler-merchant-httpd_post-orders-ID-pay.c:1224. 2024-09-11T14:41:57.372533+0200 taler-merchant-httpd-128454(5Y7TEGP77AY77ZB23XEPDVA78G) INFO Processing /pay in phase 9 2024-09-11T14:41:57.372540+0200 taler-merchant-httpd-128454(5Y7TEGP77AY77ZB23XEPDVA78G) INFO Processing /pay in phase 11 2024-09-11T14:41:57.372591+0200 taler-merchant-httpd-128454(5Y7TEGP77AY77ZB23XEPDVA78G) INFO Request for `/orders/2024.255-01X9WSSR7Q7K6/pay' completed with HTTP status 504 (0) | ||||
| Tags | No tags attached. | ||||
|
|
As per 6191cf20..e2354ce6, you now get a 400 Bad request status back. Tests still fails though... |
|
|
I would like to package the merchant 0.14 today or tomorrow for Datenspuren.de-event. AFAIK this is the only failing test. Is this quickly fixable? |
|
|
commit 360c6e9267df2a0df9a2b5643e3cbf707fff36ed (HEAD -> master, origin/master, origin/HEAD) Author: Florian Dold <florian@dold.me> Date: Tue Sep 17 18:22:53 2024 +0200 harness: adjust merchant-exchange-confusion Merchant behavior has changed here. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2024-09-11 15:35 | Florian Dold | New Issue | |
| 2024-09-11 15:35 | Florian Dold | Status | new => assigned |
| 2024-09-11 15:35 | Florian Dold | Assigned To | => Christian Grothoff |
| 2024-09-14 00:49 | Christian Grothoff | Note Added: 0023288 | |
| 2024-09-14 00:49 | Christian Grothoff | Assigned To | Christian Grothoff => Florian Dold |
| 2024-09-16 14:21 | Christian Grothoff | Note Added: 0023301 | |
| 2024-09-17 18:24 | Florian Dold | Note Added: 0023322 | |
| 2024-09-17 18:24 | Florian Dold | Status | assigned => resolved |
| 2024-09-17 18:24 | Florian Dold | Resolution | open => fixed |
