View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0007743 | Taler | libeufin-bank webui | public | 2023-03-05 01:22 | 2023-09-23 13:09 |
| Reporter | Florian Dold | Assigned To | sebasjm | 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 | ||
| Target Version | 0.9.3 | Fixed in Version | 0.9.3 | ||
| Summary | 0007743: "Confirm with bank" flow does not work with current demobank-ui | ||||
| Description | There is no way to tell the demobank-ui to continue with a certain withdrawal. Also, the confirm_transfer_url generated by the libeufin demobank doesn't contain any reference to the account that created the withdrawal operation. | ||||
| Tags | No tags attached. | ||||
| changed required * sandbox should set `confirm_transfer_url` to [1] in these endpoints [2] * spa should show an message if it is confirming a transfer with an account different than the current user optionally: if transfer_user !== current_user then ask password, this require sandbox to check of auth header when doing abort or cancel withdrawal operation [1] https://docs.taler.net/core/api-bank-access.html#get-$BANK_API_BASE_URL-accounts-$account_name-withdrawals-$withdrawal_id endpoint contains account name in the path it will be nice to have the accountName in the response object so SPA will not need to parse the path and potentially break in the future (also remove /accounts/${account_name} from the enpodint) endpoints (get, abort, cancel) do not require credentials so it will be possible to call them without asking for the password [2] https://docs.taler.net/core/api-bank-integration.html#withdrawing | |
| new docs https://git.taler.net/docs.git/commit/?id=3273debb32a00825e7c77b8468d09d070ff8f9cd which describe the new behavior of sandbox so confirm_transfer_url will have the withdrwal operation id that will redirect to the correct UI in the demobank SPA | |
| at 877a4bb1328661ae2e97d764c26e2bc0ff1f46fe | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2023-03-05 01:22 | Florian Dold | New Issue | |
| 2023-03-05 01:22 | Florian Dold | Status | new => assigned |
| 2023-03-05 01:22 | Florian Dold | Assigned To | => sebasjm |
| 2023-03-29 12:39 | sebasjm | Note Added: 0019987 | |
| 2023-04-04 18:30 | sebasjm | Note Added: 0020021 | |
| 2023-04-07 22:00 | sebasjm | Status | assigned => informing |
| 2023-04-07 22:00 | sebasjm | Resolution | open => fixed |
| 2023-04-07 22:00 | sebasjm | Note Added: 0020037 | |
| 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 |