Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

How should %2F in resource URLs be interpreted? #381

Closed
RubenVerborgh opened this issue Mar 7, 2022 · 3 comments
Closed

How should %2F in resource URLs be interpreted? #381

RubenVerborgh opened this issue Mar 7, 2022 · 3 comments
Labels
category: revisit for test suite Issues that are resolved for spec, but requires testing doc: Protocol topic: resource access

Comments

@RubenVerborgh
Copy link
Contributor

RubenVerborgh commented Mar 7, 2022

No description provided.

@ThisIsMissEm
Copy link

This may be relevant: https://simonwillison.net/2022/Mar/5/dash-encoding/

@kjetilk
Copy link
Member

kjetilk commented Apr 11, 2022

I see you reference RFC 3986 there in a discussion around URI equivalence, but I think the relevant section is 2.1:

A percent-encoding mechanism is used to represent a data octet in a component when that octet's corresponding character [...] is being used as a delimiter of, or within, the component.

In this case, "/" is delimiter, as per the same RFC, and so the percent encoded character is a data octet, that is, "any other character". The URI normalization is also given there.

I'm not sure we should put this in the protocol spec, but we should write a test case for it in a future "relevant tests outside of the protocol spec" test suite to ensure that it does not influence the containment hierarchy in any way. Indeed, foo/bar and foo%2Fbar are two different resources.

@kjetilk
Copy link
Member

kjetilk commented May 23, 2022

I created a label to use for things that needs to be revisited for tests, and with that, I think it is OK to close this. Feel free to reopen if you don't agree.

@kjetilk kjetilk closed this as completed May 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: revisit for test suite Issues that are resolved for spec, but requires testing doc: Protocol topic: resource access
Projects
None yet
Development

No branches or pull requests

3 participants