View Issue Details

IDProjectCategoryView StatusLast Update
0005203Talerotherpublic2018-04-15 18:36
ReporterFlorian Dold Assigned ToFlorian Dold 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
PrioritynormalSeveritytextReproducibilityhave not tried
Status closedResolutionfixed 
Product Versiongit (master) 
Target Version0.5Fixed in Version0.5 
Summary0005203: use URL instead of URI consistently in code and docs
DescriptionWe currently mix both, which is bad. URL seems to be more common in the browser, even though URI would be more technically correct. However we rarely deal with URIs that are not technically URLs anyway.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2017-12-09 10:14

manager   ~0012642

This also likely applies in to some places in the *protocol* (see "fulfillment_url" field). Could you double-check for 0.5 / v2 of the protocol that at least all URIs are URLs in the protocol?

Florian Dold

2018-01-30 01:34

manager   ~0012833

Fixed in the docs, deployment, bank, exchange, merchant and wallet.

Issue History

Date ModifiedUsernameFieldChange
2017-12-09 00:55Florian DoldNew Issue
2017-12-09 10:07Christian GrothoffSeverityminor => text
2017-12-09 10:07Christian GrothoffStatusnew => implementing
2017-12-09 10:07Christian GrothoffProduct Version => git (master)
2017-12-09 10:07Christian GrothoffTarget Version => 0.5
2017-12-09 10:14Christian GrothoffNote Added: 0012642
2017-12-09 10:14Christian GrothoffAssigned To => Florian Dold
2017-12-09 10:14Christian GrothoffStatusimplementing => assigned
2018-01-30 01:34Florian DoldStatusassigned => informing
2018-01-30 01:34Florian DoldResolutionopen => fixed
2018-01-30 01:34Florian DoldNote Added: 0012833
2018-04-15 18:35Christian GrothoffFixed in Version => 0.5
2018-04-15 18:36Christian GrothoffStatusinforming => closed