View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0008724 | Taler | mechant backend | public | 2024-04-10 13:34 | 2024-05-15 18:01 |
Reporter | Emmanuel | Assigned To | sebasjm | ||
Priority | high | Severity | tweak | Reproducibility | have not tried |
Status | closed | Resolution | fixed | ||
Platform | Ubuntu | OS | Linux | ||
Product Version | git (master) | ||||
Target Version | 0.11 | Fixed in Version | 0.11 | ||
Summary | 0008724: By configuring the bank account in the Taler Merchant Backend, https should be removed | ||||
Description | In the host field, I entered a URL (incl. HTTPS://) That generated the following message "payto URL not valid". This is not easy to understand. Would be better to have a direct feedback in the client (or even better, if client would remove "https://"-prefix directly). | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
|
I don't think https:// should be removed, it should be forbidden to enter anything but a domain name. Or, we need to properly handle full URIs and construct the correct paytos: Consider: http://bank.com/ # payto+http:// or not OK? Do we support payto+http? https://bank.com/api # auto-append "/"? or not OK https://bank.com/api/ https://bank.com/api/?q=foo # never OK We *also* clearly need to add a gana entry for x-taler-bank in the RFC 8905 registry! |
|
In any case, we should *also* give an example for what is expected under "Host". And never submit an invalid payto:// ;-) |
|
I think for a first iteration, let's only accept domain names (without schema://) |
|
e07f801aa..ffd2e70fe |
Date Modified | Username | Field | Change |
---|---|---|---|
2024-04-10 13:34 | Emmanuel | New Issue | |
2024-04-10 13:34 | Emmanuel | Status | new => assigned |
2024-04-10 13:34 | Emmanuel | Assigned To | => Christian Grothoff |
2024-04-10 13:34 | Emmanuel | File Added: Capture d’écran 2024-04-10 à 13.27.40.png | |
2024-04-10 13:53 | Christian Grothoff | Note Added: 0022195 | |
2024-04-10 13:54 | Christian Grothoff | Note Added: 0022196 | |
2024-04-10 13:54 | Christian Grothoff | Assigned To | Christian Grothoff => sebasjm |
2024-04-10 13:54 | Christian Grothoff | Priority | normal => high |
2024-04-10 13:54 | Christian Grothoff | Product Version | => git (master) |
2024-04-10 13:54 | Christian Grothoff | Target Version | => 0.11 |
2024-04-10 13:55 | Christian Grothoff | Severity | minor => tweak |
2024-04-10 14:00 | Christian Grothoff | Note Added: 0022197 | |
2024-04-15 18:56 | sebasjm | Status | assigned => resolved |
2024-04-15 18:56 | sebasjm | Resolution | open => fixed |
2024-04-15 18:56 | sebasjm | Note Added: 0022249 | |
2024-05-15 17:53 | Christian Grothoff | Fixed in Version | => 0.11 |
2024-05-15 18:01 | Christian Grothoff | Status | resolved => closed |