View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006855 | Taler | merchant backend | public | 2021-05-06 16:23 | 2021-09-02 16:14 |
| 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 | high | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Product Version | git (master) | ||||
| Target Version | 0.8 | Fixed in Version | 0.8 | ||
| Summary | 0006855: merchant backend should implement current version of forgettable information (DD18) | ||||
| Description | The merchant backend current doesn't correctly scrub salts and doesn't support forgetting primitives. Also, the merchant backend must check well-formedness of contract terms and generate the salt for orders. The wallet already implements the current version of DD18: https://git.taler.net/wallet-core.git/tree/packages/taler-wallet-core/src/util/contractTerms.ts Ideally we should also have some test vectors of the following form to easily test compatibility between wallet and merchant: { contract_terms_hashing: [{ contract_terms_str: "...", contract_terms_hash: "...", }, ... ] } | ||||
| Tags | No tags attached. | ||||
| 6ee58c99..684a2342 (exchange.git) implements salt-forgetting and improves the tests. | |
| 4713a43..4e406e0 (docs.git, dd13 branch (!)) adds a test vector to DD18. Specifically, { "k1": 1, "_forgettable": { "k1": "SALT" }, "k2": { "n1": true, "_forgettable": { "n1": "salt" } }, "k3": { "n1": "string" } } must result in the hash 287VXK8T6PXKD05W8Y94QJNEFCMRXBC9S7KNKTWGH2G2J2D7RYKPSHNH1HG9NT1K2HRHGC67W6QM6GEC4BSN1DPNEBCS0AVDT2DBP5G | |
| Re-assigning to Florian for interoperability testing with TypeScript. | |
| Merchant backend and wallet now agree on the contract terms hashing with forgettable fields. | |
| Fix committed to master branch. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2021-05-06 16:23 | Florian Dold | New Issue | |
| 2021-05-06 16:23 | Florian Dold | Status | new => assigned |
| 2021-05-06 16:23 | Florian Dold | Assigned To | => Christian Grothoff |
| 2021-05-09 10:53 | Christian Grothoff | Priority | normal => high |
| 2021-05-09 10:53 | Christian Grothoff | Product Version | => git (master) |
| 2021-05-09 10:53 | Christian Grothoff | Target Version | => 0.9 |
| 2021-05-09 11:44 | Christian Grothoff | Note Added: 0017801 | |
| 2021-05-09 11:45 | Christian Grothoff | Note Added: 0017802 | |
| 2021-05-09 11:45 | Christian Grothoff | Assigned To | Christian Grothoff => Florian Dold |
| 2021-05-09 11:45 | Christian Grothoff | Note Added: 0017803 | |
| 2021-06-16 10:19 | Florian Dold | Status | assigned => informing |
| 2021-06-16 10:19 | Florian Dold | Resolution | open => fixed |
| 2021-06-16 10:19 | Florian Dold | Note Added: 0017957 | |
| 2021-07-30 11:56 | Christian Grothoff | Fixed in Version | => 0.8 |
| 2021-07-30 11:59 | Christian Grothoff | Target Version | 0.9 => 0.8.1 |
| 2021-07-30 12:02 | Christian Grothoff | Target Version | 0.8.1 => 0.8 |
| 2021-08-24 14:23 | Christian Grothoff | Status | informing => closed |
| 2021-09-02 16:13 | Christian Grothoff | ||
| 2021-09-02 16:14 | Christian Grothoff | Note Added: 0018228 | |
| 2021-09-02 16:14 | Christian Grothoff | Assigned To | Florian Dold => Christian Grothoff |
| 2025-03-22 13:09 | Christian Grothoff | Category | mechant backend => merchant backend |