View Issue Details

IDProjectCategoryView StatusLast Update
0008724Talermechant backendpublic2024-05-15 18:01
ReporterEmmanuel Assigned Tosebasjm  
PriorityhighSeveritytweakReproducibilityhave not tried
Status closedResolutionfixed 
PlatformUbuntuOSLinux 
Product Versiongit (master) 
Target Version0.11Fixed in Version0.11 
Summary0008724: By configuring the bank account in the Taler Merchant Backend, https should be removed
DescriptionIn 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).
TagsNo tags attached.
Attached Files

Activities

Christian Grothoff

2024-04-10 13:53

manager   ~0022195

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!

Christian Grothoff

2024-04-10 13:54

manager   ~0022196

In any case, we should *also* give an example for what is expected under "Host". And never submit an invalid payto:// ;-)

Christian Grothoff

2024-04-10 14:00

manager   ~0022197

I think for a first iteration, let's only accept domain names (without schema://)

sebasjm

2024-04-15 18:56

developer   ~0022249

e07f801aa..ffd2e70fe

Issue History

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