Skip to content
This repository has been archived by the owner on Feb 17, 2024. It is now read-only.

Remove trailing slashes from Secutity baseUris #12

Open
dmartinezg opened this issue Oct 28, 2013 · 4 comments
Open

Remove trailing slashes from Secutity baseUris #12

dmartinezg opened this issue Oct 28, 2013 · 4 comments

Comments

@dmartinezg
Copy link

baseUri's should not end in slashes (unless they are meant to be there specifically), because they result in double slashes at the beginning of resource URIs.

@usarid
Copy link
Contributor

usarid commented Oct 29, 2013

Is this an error in the spec somewhere, or a restriction you want to add, or something else?

On Mon, Oct 28, 2013 at 4:05 PM, Damian Martinez Gelabert
notifications@github.com wrote:

baseUri's should not end in slashes (unless they are meant to be there specifically), because they result in double slashes at the beginning of resource URIs.

Reply to this email directly or view it on GitHub:
#12

@dmartinezg
Copy link
Author

There was a discussion in projects using RAML, that since the JS parser does not strip ending slashes from the baseUri, when consumers want to use the output, resource URLs end up with double slashes.

People have asked that we mandate that RAML processors strip ending slashes from the baseUri. This issue is to at least remove it from the examples.

@dmartinezg
Copy link
Author

I just pushed 4421bfe with this change.

But we still need to discuss whether RAML should mandate that processors strip trailing slashes from the baseUri or resource URLs, etc.

@usarid
Copy link
Contributor

usarid commented Nov 16, 2013

I don't know about mandating its removal, as it's a valid albeit odd URL. One place you see this is in proxy usecases, where the URL of the call to be proxied is appended to the proxy's URL.

Amusingly, Tim Berners-Lee even expressed regret at having double slashes in 'http://' (http://www.zdnet.com/blog/igeneration/double-slash-in-web-addresses-a-bit-of-a-mistake/3090) but it's too late for that right now.

But perhaps we can indicate in the spec that processors should warn about it as a likely source of errors.

The ultimate interpretation of double slashes is up to the server implementing the API. If this were important enough we could find some way to indicate in a RAML spec that a specific API will treat consecutive slashes as a single slash, or that it will treat them as significant, but I'm not sure it's important enough.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

No branches or pull requests

3 participants