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

Close #7: Uri-Path encoding of trailing slash #32

Merged
merged 3 commits into from
Jul 2, 2024
Merged

Close #7: Uri-Path encoding of trailing slash #32

merged 3 commits into from
Jul 2, 2024

Conversation

cabo
Copy link
Member

@cabo cabo commented Jun 17, 2024

Use text of Err4895 for a an additional note about trailing slashes in URI paths

@chrysn
Copy link
Member

chrysn commented Jun 19, 2024

From interim: Let's add examples.

@mrdeep1
Copy link

mrdeep1 commented Jun 20, 2024

So is coap://path///// different from coap://path/ ?

@cabo
Copy link
Member Author

cabo commented Jun 20, 2024

So is coap://path///// different from coap://path/ ?

Yes, and we have to clarify this.

The only place where we suppress the empty path component is between coap://authority/ and coap://authority -- these mean the same in CoAP, and the first is the one that is reconstituted from the CoAP Options.
Additional trailing slashes do always count.

@mrdeep1
Copy link

mrdeep1 commented Jun 20, 2024

OK - taking it further, coap://host/path////// and coap://host/path/ are different (which is was I meant to write in the first place - been a long day).

@chrysn
Copy link
Member

chrysn commented Jun 20, 2024 via email

@mrdeep1
Copy link

mrdeep1 commented Jun 20, 2024

I don't like it as well. But need to understand potential consequences.

@cabo
Copy link
Member Author

cabo commented Jul 2, 2024

From interim: Let's add examples.

Do we add examples to corrclar or to the formal addition to RFC7252?

@cabo cabo merged commit 897d1e5 into main Jul 2, 2024
2 checks passed
@cabo
Copy link
Member Author

cabo commented Jul 2, 2024

I added some examples and merged -- we can reopen the issue if it isn't completely covered yet.

cabo added a commit that referenced this pull request Sep 24, 2024
Update #32 from 2024-07-03 CoRE interim WG meeting discussion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants