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.

Sign up for GitHub

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

Jump to bottom

"rel" in links should be mandatory #358

Closed
cportele opened this issue Mar 30, 2020 · 0 comments · Fixed by #359
Closed

"rel" in links should be mandatory #358

cportele opened this issue Mar 30, 2020 · 0 comments · Fixed by #359
Assignees
@cportele
Labels
bug OGC API: Common Issue related to general resources or requirements (see #190) Part 1: Core Issue related to Part 1 - Core
Milestone
Part 1, Version 1...

Comments

@cportele
Copy link
Member

In the discussion in #343 we have identified that in RFC 8288 (a normative reference) both rel and href are required, but in the link object only href is currently required. This error should be corrected in version 1.0.1. Note that all links used in the standard have normative language about the link relation type, so this will not have an impact on conformant implementations.

Note that if someone is using link objects without rel, these are not consistent with the standard for Web Linking and should not be links. Most likely these should just be URIs (in JSON Schema: strings with a uri-reference format).

(Created as a result of the discussion in the meeting on 2020-03-30.)
@cportele cportele added bug Part 1: Core Issue related to Part 1 - Core OGC API: Common Issue related to general resources or requirements (see #190) labels Mar 30, 2020
@cportele cportele added this to the Part 1, Version 1.0.1 milestone Mar 30, 2020
@cportele cportele self-assigned this Mar 30, 2020
@cportele cportele mentioned this issue Mar 30, 2020
Open
cportele added a commit that referenced this issue Mar 30, 2020
@cportele
make rel required in links
b21cb75 
fixes #358
@cportele cportele mentioned this issue Mar 30, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cportele cportele

Labels
bug OGC API: Common Issue related to general resources or requirements (see #190) Part 1: Core Issue related to Part 1 - Core
Projects
Features Part 1: Core
Status: Done
Milestone
Part 1, Version 1.0.1
Development

Successfully merging a pull request may close this issue.

make rel required in links opengeospatial/ogcapi-features
1 participant
@cportele