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

WMS parameter values should not be case sensitive #7

Closed
alexgleith opened this issue May 24, 2018 · 4 comments
Closed

WMS parameter values should not be case sensitive #7

alexgleith opened this issue May 24, 2018 · 4 comments

Comments

@alexgleith
Copy link
Contributor

  • datacube-wms version: all

Description

Parameters values in this WMS implementation are case sensitive, and I don't think they should be.

The standard says:

6.4.1. Parameter Ordering and Case
Parameter names shall not be case sensitive, but parameter values shall be case sensitive.

So the implementation is technically correct...

But GeoServer, for example, is NOT case sensitive for most things, including parameter values, where it doesn't have to be.

What I Did

Try to get capabilities at ?service=wms&request=getcapabilities

Note that it only works with ?service=WMS&request=GetCapabilities.

@SpacemanPaul
Copy link
Contributor

I am sorely tempted to close this as "complies with the standard" and suggest you file a bug report with either GeoServer or the OGC. :)

But in the name of pragmatism I will reluctantly make the service and request parameter values case-insensitive.

Note that this is not a general precedent. Other parameter values will be considered on case-by-case basis as requested but will most likely remain case-sensitive.

@SpacemanPaul
Copy link
Contributor

SpacemanPaul commented May 31, 2018

(Or as my mother would put it: If Geoserver jumped off a cliff, would you?) :)

@SpacemanPaul
Copy link
Contributor

Fixed in commit 8e33b20

@alexgleith
Copy link
Contributor Author

hehe, thanks @SpacemanPaul

I think this makes it better, even if it does break the rules.

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

2 participants