View Issue Details

IDProjectCategoryView StatusLast Update
0008724Talermerchant backendpublic2024-05-15 16:01
ReporterEmmanuel Assigned Tosebasjm Resolution deadline
ImplementerReviewer 
Release signed byDeployed by 
VerifierCommunicator 
Data protection authority notification requiredNo  
CSIRT notification requiredNo  
Estimated effort (hours)Actual effort (hours)0 
Fast-trackNo
Work packagenone
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

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2024-04-10 11: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 11: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 12:00

manager   ~0022197

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

sebasjm

2024-04-15 16:56

developer   ~0022249

e07f801aa..ffd2e70fe

Issue History

Date ModifiedUsernameFieldChange
2024-04-10 11:34EmmanuelNew Issue
2024-04-10 11:34EmmanuelStatusnew => assigned
2024-04-10 11:34EmmanuelAssigned To => Christian Grothoff
2024-04-10 11:34EmmanuelFile Added: Capture d’écran 2024-04-10 à 13.27.40.png
2024-04-10 11:53Christian GrothoffNote Added: 0022195
2024-04-10 11:54Christian GrothoffNote Added: 0022196
2024-04-10 11:54Christian GrothoffAssigned ToChristian Grothoff => sebasjm
2024-04-10 11:54Christian GrothoffPrioritynormal => high
2024-04-10 11:54Christian GrothoffProduct Version => git (master)
2024-04-10 11:54Christian GrothoffTarget Version => 0.11
2024-04-10 11:55Christian GrothoffSeverityminor => tweak
2024-04-10 12:00Christian GrothoffNote Added: 0022197
2024-04-15 16:56sebasjmStatusassigned => informing
2024-04-15 16:56sebasjmResolutionopen => fixed
2024-04-15 16:56sebasjmNote Added: 0022249
2024-05-15 15:53Christian GrothoffFixed in Version => 0.11
2024-05-15 16:01Christian GrothoffStatusinforming => closed
2025-03-22 13:09Christian GrothoffCategorymechant backend => merchant backend