View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0007810 | Taler | wallet-core | public | 2023-04-22 15:54 | 2023-11-29 00:30 |
| 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 | urgent | Severity | feature | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Target Version | 0.9.3 | Fixed in Version | 0.9.3 | ||
| Summary | 0007810: support conversion service URL and regex validation for /wire | ||||
| Description | We also need to distinguish between withdrawals and deposits, as in the conversion case, different restrictions for the payto:// URI apply. | ||||
| Tags | No tags attached. | ||||
| API change for /wire specified in docs.git -- 43a3dca..d3102b7. Once implemented, we MUST bump the protocol to "15:0:0" and add a (transient) protocol-version based switch in libtalerexchange. | |
| This is largely implemented (exchange, merchant, documentation). Wallet is missing (=> assigning to Florian). What is still missing is that the merchant (and wallet) currently ignores debit restrictions imposed by the exchange and would be perfectly happy to accept deposits for coins from exchanges where the exchange doesn't actually accept the merchant's bank account. To fix this, we need a working regex evaluator for this style of regex AND to modify the pay logic to actually check. | |
| In wallet-core, we now support: * using the conversion_url during (manual only!) withdrawal, allowing the UIs to display the external currency that the user needs to transfer to withdraw the regional currency * checking the debit restrictions on deposit However, obviously the wallet can't check deposit restrictions when doing a payment, since it doesn't know the payto URI of the merchant. What was the design here? I remember we discussed *something* here, but not sure what it was exactly anymore. | |
| (After giving it more thought, I think that the merchant must only include exchanges in the contract terms where the merchant account is allowed by the debit restrictions of the exchange.) | |
| Yes, and the merchant does so. Why did you re-assign to me? | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2023-04-22 15:54 | Florian Dold | New Issue | |
| 2023-04-22 15:54 | Florian Dold | Status | new => assigned |
| 2023-04-22 15:54 | Florian Dold | Assigned To | => Christian Grothoff |
| 2023-04-22 16:11 | Florian Dold | Description Updated | |
| 2023-04-22 20:08 | Christian Grothoff | Note Added: 0020126 | |
| 2023-04-22 20:09 | Christian Grothoff | Note Edited: 0020126 | |
| 2023-05-02 10:09 | Christian Grothoff | Note Added: 0020172 | |
| 2023-05-02 10:09 | Christian Grothoff | Assigned To | Christian Grothoff => Florian Dold |
| 2023-06-21 16:22 | Florian Dold | Target Version | 0.9.3 => 0.9.4 |
| 2023-07-16 14:05 | Christian Grothoff | Priority | normal => urgent |
| 2023-11-02 23:40 | Christian Grothoff | Category | exchange => wallet-core |
| 2023-11-13 17:39 | Florian Dold | Assigned To | Florian Dold => Christian Grothoff |
| 2023-11-13 17:39 | Florian Dold | Assigned To | Christian Grothoff => Florian Dold |
| 2023-11-22 17:19 | Florian Dold | Assigned To | Florian Dold => Christian Grothoff |
| 2023-11-22 17:21 | Florian Dold | Note Added: 0020657 | |
| 2023-11-22 17:28 | Florian Dold | Note Added: 0020658 | |
| 2023-11-22 19:04 | Christian Grothoff | Note Added: 0020659 | |
| 2023-11-22 19:15 | Christian Grothoff | Status | assigned => informing |
| 2023-11-22 19:15 | Christian Grothoff | Resolution | open => fixed |
| 2023-11-22 19:15 | Christian Grothoff | Fixed in Version | => 0.9.4 |
| 2023-11-29 00:28 | Christian Grothoff | Target Version | 0.9.4 => 0.9.3 |
| 2023-11-29 00:29 | Christian Grothoff | Fixed in Version | 0.9.4 => 0.9.3 |
| 2023-11-29 00:30 | Christian Grothoff | Status | informing => closed |