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

Can we use full URLs for CRS or abbreviations? #15

Closed
joanma747 opened this issue Jun 21, 2019 · 9 comments
Closed

Can we use full URLs for CRS or abbreviations? #15

joanma747 opened this issue Jun 21, 2019 · 9 comments

Comments

@joanma747
Copy link
Contributor

Features CRS extension uses Full CRS but we are using abbreviations. It seems that full URLS are considered ids. It might be better to use full URLs instead

@jerstlouis
Copy link
Member

The OGC NA policy on short-hand CURIEs forms of CRS URIs was applied this week ( opengeospatial/NamingAuthority#92 and opengeospatial/NamingAuthority#93 ).

See https://docs.opengeospatial.org/pol/09-048r6.html#toc23 .

We should include text explicitly allowing CURIEs as short-hand for full URIs in OGC API - Maps.

@jerstlouis
Copy link
Member

jerstlouis commented Nov 10, 2022

The policy describing the use of CURIEs is at https://docs.opengeospatial.org/pol/09-048r6.html#toc14 .
Currently it mentions:

OGC Standards should only allow safe CURIEs, unless support for other forms of CURIEs is inherited from another standard

Not sure what the value of safe CURIE is if there is no way that a non-safe CURIE could be misinterpreted?

Safe CURIE would force us to use:

https://example.org/foo/collections/bar/map?crs=[EPSG:3395]

instead of

https://example.org/foo/collections/bar/map?crs=EPSG:3395

@jerstlouis
Copy link
Member

jerstlouis commented Nov 18, 2022

It seems that the requirements in the specification should be that the server shall support safe CURIEs (e.g., crs=[EPSG:4326]), but we should include a permission that the server may also be tolerant and interpret unsafe CURIE, which would be compatible with the WMS crs=EPSG:4326 syntax.

@joanma747
Copy link
Contributor Author

I'm not sure that adding a "[" "]" makes a string more safe :-). In any case I'm starting to see this very confusing. Too many notations for the same thing only because we are trying to please too many people: the old WMS users, the NA people and the RDF people.

@jerstlouis
Copy link
Member

jerstlouis commented Nov 18, 2022

@joanma747

The square brackets may be used to prevent ambiguities between CURIEs and regular URIs, yielding so-called safe CURIEs.

from https://en.wikipedia.org/wiki/CURIE

See also https://www.w3.org/TR/curie/#sec_2.2 2.2. Ambiguities Between CURIEs and URIs

Too many notations for the same thing only because we are trying to please too many people:

Yes, the requirements, based on the policy, would express that servers shall support the two following options:

and a permission would allow the server implementations to also recognize simply EPSG:4326, making the crs query parameter compatible with WMS.

I feel it is good if we can make everybody happy ;)

@chris-little
Copy link

Can EPSG:4326 cause wrong HTTP and namespace behaviour, outside of a geospatial context?

@jerstlouis
Copy link
Member

@chris-little I don't believe so currently, but as @cportele points out in opengeospatial/ogcapi-features#632 (comment) new URI schemes are added all the time.

@chris-little
Copy link

@jerstlouis My preference is for the Naming Authority policy (safe CURIEs) to remain. I think putting square brackets is not burdensome, is simple, and clearly avoids ambiguity. I see many examples of geospatial software using them, even doubled, to avoid ambiguity in a WKT context, such as optionally supporting arrays rather than single values.
I am sympathetic to syntaxes and notation being global, generic and non-domain specific.
Safe CURIEs like [EPSG:nnnn]` is apolicy, so I suppose implementations are free to choose to be more generous, but I think Postel's Rule applies: accept generously, generate strictly.

@joanma747
Copy link
Contributor Author

Applied and closed in regular telco

joanma747 added a commit that referenced this issue Nov 25, 2022
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

No branches or pull requests

3 participants