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

Use of colon in media types ... #759

Open
pvretano opened this issue Sep 1, 2022 · 5 comments
Open

Use of colon in media types ... #759

pvretano opened this issue Sep 1, 2022 · 5 comments
Labels
no change Part 1: Core Issue related to Part 1 - Core

Comments

@pvretano
Copy link
Contributor

pvretano commented Sep 1, 2022

From gitter ... @bradh pointed out: "In https://portal.ogc.org/files/99149#mediatypes, is that supposed to be a valid Media Type? I read RFC 6838 as saying it can't contain a : character." If @bradh is correct then we should patch the source documents and update Part 1 accordingly.

@cportele
Copy link
Member

cportele commented Sep 1, 2022

The comment is correct, but the issue cannot be solved by fixing Part 1. The issue is in the GML media type registration at IANA (https://www.iana.org/assignments/media-types/application/gml+xml) and that needs to be corrected first along with the OGC copy of it (https://portal.ogc.org/files/?artifact_id=59324).

Interesting that we did not detect this during the media type registration and that the issue was not raised in the IANA Expert Review process.

I am not aware that the parameter has been used outside of OGC, maybe we should simplify/change the "profile" specification to:

"profile": If provided, this parameter indicates the GML profiles that the GML document conforms to. This is consistent with the semantics of a profile as laid out in RFC 6906.

The parameter can also be used to provide protocol-specific operations, such as profile-based content negotiation in HTTP.

Valid profiles are published at http://www.opengis.net/def/profile/ogc/gml. The profiles follow the pattern 'http://www.opengis.net/def/profile/ogc/gml/{name}' where 'name' is the value of the "profile" parameter, e.g., 'gml-sf0'.

As a GML document may conform to more than one GML profile, the parameter SHALL be a whitespace-separated list of profile names.

Note: The original media type specification required the profile names to be URIs. This violates the requirements for media type parameter values, which only allow a restricted character set. Implementations should be updated to support this updated specification of the "profile" parameter.

I also notice that the OGC Definition Server has a problem, both http://www.opengis.net/def/profile/ogc/gml and http://www.opengis.net/def/profile both result in an error. This would need to be fixed before submitting anything to IANA.

@ogcscotts - I am not aware of a form to submit corrections to a registered media type, only new submissions. Are you aware of any other channel to update a media type registration? Any other thoughts?

@bradh
Copy link
Contributor

bradh commented Sep 2, 2022

I think this probably needs to be resolved at the OGC Naming Authority and a common approach pushed out to all applicable documents / SWGs.

See opengeospatial/NamingAuthority#194 for my report.

@bradh
Copy link
Contributor

bradh commented Sep 8, 2022

Looks like the source documents just need it to be quoted.

@cportele
Copy link
Member

cportele commented Sep 8, 2022

@bradh - I think the profile parameter values are all quoted. Can you point out where the quotes are missing?

Note: In v1.0.0 the quotes were missing, but that has been corrected in May 2020 (#382) and this fix is part of v1.0.1.

@bradh
Copy link
Contributor

bradh commented Sep 8, 2022

@cportele cportele added the Part 1: Core Issue related to Part 1 - Core label Sep 12, 2022
@cportele cportele added this to Backlog in Features Part 1: Core via automation Sep 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no change Part 1: Core Issue related to Part 1 - Core
Projects
Status: Backlog
Development

No branches or pull requests

3 participants