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

URI Schemes term restriction is fairly draconian #310

Closed
hsolbrig opened this issue Nov 22, 2019 · 4 comments · Fixed by #315
Closed

URI Schemes term restriction is fairly draconian #310

hsolbrig opened this issue Nov 22, 2019 · 4 comments · Fixed by #315

Comments

@hsolbrig
Copy link

Section 9.1 Terms asserts:

"When used as the prefix in a Compact IRI, to avoid the potential ambiguity of a prefix being confused with an IRI scheme, terms MUST NOT come from the list of URI schemes as defined in [IANA-URI-SCHEMES]."

This appears to be quite restrictive, as prohibits "aaa", "nih" and more than 100 other seemingly innocuous terms. At the moment, the playground implementation doesn't enforce this (thank goodness). Should this constraint be weakened to a suggestion rather than a prohibition?

@gkellogg
Copy link
Member

This is really for publishers of JSON-LD data rather than consumers. There are other steps we take to prevent IRIs from being misinterpreted as compact IRIs, and we may not need to make such blanket statements.

That said, creating a term "https" which can be used as a prefix would be a bad idea.

@TallTed
Copy link
Member

TallTed commented Nov 25, 2019

This becomes more problematic when one considers the indeterminate future -- wherein any number of terms may be added to the list at [IANA-URI-SCHEMES] after having been used in JSON-LD documents created in the meantime.

Yes, using https (among others) would likely be a bad idea -- and some warning of the impact of such collisions, both current and future, should be included. But the current blanket forbiddance is problematic and unnecessarily strict.

At minimum, this MUST NOT should be reduced to a SHOULD NOT.

@pchampin pchampin added a11y-tracker Group bringing to attention of a11y, or tracked by the a11y Group but not needing response. and removed a11y-tracker Group bringing to attention of a11y, or tracked by the a11y Group but not needing response. labels Nov 25, 2019
@azaroth42
Copy link
Contributor

I believe this is an editorial oversight covered by previous resolutions. See:
#147, #148, #155, #177 for highly related issues.

MUST NOT is impossible to enforce, as discussed during the above issues and as an untestable feature we should have fixed it during the resolution of those issues.

@azaroth42
Copy link
Contributor

Resolved by #315. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants