View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006904 | Taler | merchant backend | public | 2021-06-16 10:12 | 2024-01-12 13:04 |
| Reporter | Florian Dold | Assigned To | Christian Grothoff | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Product Version | git (master) | ||||
| Target Version | 0.8 | Fixed in Version | 0.8 | ||
| Summary | 0006904: contract claiming fails with database error for certain contract terms | ||||
| Description | When the salt for a forgettable field is specified as "true", the merchant backend is supposed to generate a salt. However, the contract claiming simply fails. Here is the order specification I've used: { summary: "Buy me!", amount: "TESTKUDOS:5", fulfillment_url: "taler://fulfillment-success/thx", extra: { foo: { bar: "baz" }, $forgettable: { foo: true, }, }, } | ||||
| Steps To Reproduce | First, make sure that a current version of the exchange is installed, as the forgettable fields implementation changed. $ taler-wallet-cli testing run-integrationtests payment-forgettable | ||||
| Additional Information | { "type": "error", "operation": "preparePay", "id": "reqid-1", "error": { "code": 7005, "hint": "Error: WALLET_UNEXPECTED_REQUEST_ERROR", "details": { "requestUrl": "http://localhost:8083/orders/2021.167-G1ZAG5107XA0M/claim", "httpStatusCode": 500, "errorResponse": { "code": 54, "hint": "The service encountered an error event to commit the database transaction (hard, unrecoverable error).", "detail": null } }, "message": "Unexpected error code in response" } } | ||||
| Tags | No tags attached. | ||||
| Forgettable 'true' to 'random' conversion was never implemented AFAIK. 23b4c5e..0f66ec8 modifies the code to notice bad 'salt' values, so that the test now fails during order creation. Next: implement salt generation! | |
| Fixed in 0f66ec8..5a838bf | |
| Fix committed to master branch. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2021-06-16 10:12 | Florian Dold | New Issue | |
| 2021-06-16 10:12 | Florian Dold | Status | new => assigned |
| 2021-06-16 10:12 | Florian Dold | Assigned To | => Christian Grothoff |
| 2021-06-16 20:39 | Christian Grothoff | Note Added: 0017959 | |
| 2021-06-16 21:18 | Christian Grothoff | Note Added: 0017960 | |
| 2021-06-16 21:18 | Christian Grothoff | Status | assigned => informing |
| 2021-06-16 21:18 | Christian Grothoff | Resolution | open => fixed |
| 2021-06-16 21:18 | Christian Grothoff | Fixed in Version | => 0.9 |
| 2021-06-16 21:18 | Christian Grothoff | Product Version | => git (master) |
| 2021-06-16 21:18 | Christian Grothoff | Target Version | => 0.9 |
| 2021-07-30 11:57 | Christian Grothoff | Fixed in Version | 0.9 => 0.8.1 |
| 2021-07-30 11:59 | Christian Grothoff | Target Version | 0.9 => 0.8.1 |
| 2021-07-30 12:02 | Christian Grothoff | Fixed in Version | 0.8.1 => 0.8 |
| 2021-07-30 12:02 | Christian Grothoff | Target Version | 0.8.1 => 0.8 |
| 2021-08-24 14:22 | Christian Grothoff | Status | informing => closed |
| 2021-09-02 16:22 | Christian Grothoff | ||
| 2021-09-02 16:23 | Christian Grothoff | Note Added: 0018345 | |
| 2024-01-12 13:04 | Christian Grothoff | Category | merchant backend API (C) => mechant backend |
| 2025-03-22 13:09 | Christian Grothoff | Category | mechant backend => merchant backend |