View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0007688 | Taler | libeufin-bank webui | public | 2023-02-13 19:32 | 2023-09-23 13:09 |
| Reporter | sebasjm | Assigned To | Florian Dold | 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 | major | Reproducibility | always |
| Status | closed | Resolution | won't fix | ||
| Product Version | git (master) | ||||
| Target Version | 0.9.3 | Fixed in Version | 0.9.3 | ||
| Summary | 0007688: withdrawing from bank with amount less or equal than withdraw fee fails | ||||
| Description | should be bank check or have a configuration to not allow withdrawing an amount that does not make sense. NOTE: withdrawal fee is a configuration in the exchange that the bank should be aware of | ||||
| Tags | No tags attached. | ||||
| Well, theoretically the exchange exposes this via /keys, but it's not something we can reasonably expect banks to query. And even with tight integration, the customer could always be sending the money to another exchange that isn't the one that is pre-configured. Finally, I think this is largely a demo issue: in practice, the smallest units customers would bother to transfer will be larger, and it's OK if the *wallet* is then the one that fails. But, I think we should: 1) make sure the wallet generates a NICE error message (this amount is insanely tiny, try something larger), and 2) Change the demo to have libeufin refuse transfers less than 0.01 KUDOS (if it doesn't yet, didn't check), and 3) Change the demo to make 0.01 KUDOS the smallest coin/unit/fee instead of 0.1 KUDOS. That way, we can still test that the wallet complains if the amount is just 0.01 KUDOS (so we can still test the error), but it is unlikely that users would stumble over it unless they are _trying_ extreme cases. WDYT? | |
| Sandox needs to be configured with a default exchange, that when it make sense to check for this value. We can make the libeufin sandbox 1.- check which is the minimum amount to be withdrawn with taler when the default exchange is added 2.- provide a way to add this configuration using libeufin cli Either way, it should be provided to the bank ui in some way. I think this is better that the bankui checking directly to the exchange. | |
| cc @ms | |
| I agree that banks should be as unaware as possible about Taler specific aspects, even with tight integration. | |
| A real-world deployment of the bank also would not know anything about the denomination structure of the exchange. But the real-world would not allow sub-cent transfers. And the exchange will very, very likely know this and have to adjust the denomination structure accordingly. In summary, it makes sense for the behavior of the demobank to be close to what we would expect a real bank to do, and thus it doesn't make sense to add any exchange-specific checks in here. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2023-02-13 19:32 | sebasjm | New Issue | |
| 2023-02-13 19:32 | sebasjm | Status | new => assigned |
| 2023-02-13 19:32 | sebasjm | Assigned To | => Christian Grothoff |
| 2023-02-13 20:10 | Christian Grothoff | Note Added: 0019829 | |
| 2023-02-13 20:10 | Christian Grothoff | Assigned To | Christian Grothoff => Florian Dold |
| 2023-02-13 20:10 | Christian Grothoff | Status | assigned => feedback |
| 2023-02-14 14:57 | sebasjm | Note Added: 0019837 | |
| 2023-02-14 14:57 | sebasjm | Status | feedback => assigned |
| 2023-02-14 14:57 | sebasjm | Note Added: 0019838 | |
| 2023-02-14 15:21 | MS | Note Added: 0019839 | |
| 2023-02-14 15:21 | MS | Target Version | 0.9.2 => |
| 2023-02-19 22:45 | Florian Dold | Target Version | => 0.9.3 |
| 2023-04-04 10:00 | Florian Dold | Status | assigned => informing |
| 2023-04-04 10:00 | Florian Dold | Resolution | open => won't fix |
| 2023-04-04 10:00 | Florian Dold | Note Added: 0020006 | |
| 2023-09-23 13:07 | Christian Grothoff | Fixed in Version | => 0.9.3 |
| 2023-09-23 13:09 | Christian Grothoff | Status | informing => closed |
| 2023-12-03 00:21 | Christian Grothoff | Category | demobank-ui => libeufin-bank-ui |
| 2024-01-12 13:10 | Christian Grothoff | Category | libeufin-bank-ui => libeufin-bank-ui (SPA) |
| 2026-08-23 14:18 | Florian Dold | Category | libeufin-bank-ui (SPA) => libeufin-bank webui |