-
-
Notifications
You must be signed in to change notification settings - Fork 350
require that $schema
cannot contain a fragment
#1633
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -957,8 +957,8 @@ resources, unless such a resource itself declares a different dialect by | |
including the `$schema` keyword with a different value. | ||
|
||
The value of this keyword MUST be an | ||
[IRI](https://www.rfc-editor.org/info/rfc3987) (containing a scheme) and this | ||
IRI MUST be normalized. | ||
[IRI](https://www.rfc-editor.org/info/rfc3987) (containing a scheme). This | ||
IRI MUST be normalized and MUST NOT contain a fragment. | ||
Comment on lines
+960
to
+961
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This can be called an "absolute IRI" instead of just an "IRI". Then we don't have to make a normative statement that a fragment isn't allowed because that's part of the definition of "absolute IRI". However, it might be a good idea to clarify that fact since most people think "absolute" just means "non-relative". Which means we shouldn't need to clarify that it mean "containing a scheme" anymore. Maybe something like this would work,
Suggested change
[IRI](https://www.rfc-editor.org/info/rfc3987) (containing a scheme). This
IRI MUST be normalized and MUST NOT contain a fragment.
[absolute IRI](https://www.rfc-editor.org/info/rfc3987) (without a fragment). This
IRI MUST be normalized.
|
||
|
||
If this IRI identifies a retrievable resource, that resource SHOULD be of media | ||
type `application/schema+json`. | ||
|