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

Crs not supported with small case #226

Open
GinaStavropoulou opened this issue Sep 19, 2023 · 1 comment
Open

Crs not supported with small case #226

GinaStavropoulou opened this issue Sep 19, 2023 · 1 comment
Assignees

Comments

@GinaStavropoulou
Copy link

I am getting this exception when I run the tests:

image

It seems that it does not recognize my CRS even though it starts exactly with the suggested string.
I believe the issue lies here.
In this line you are checking the CRS against http://www.opengis.net/def/crs/EPSG with EPSG in capital letters. And right bellow you suggest that it should be "starting with http://www.opengis.net/def/crs/epsg"

Shouldn't both capital and lower case letters be accepted?

@dstenger
Copy link
Contributor

Thank you for reporting.

You are right that the error message and the check should be adjusted.
However, as far as I see, capital letters should be used and the error message must be updated.
See CRS registry: https://www.opengis.net/def/crs

I will clarify this with the CITE team and provide feedback again.

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

No branches or pull requests

4 participants