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

Fix tests/test_capabilities.py with correct check of request case #665

Merged

Conversation

gschwind
Copy link
Collaborator

@gschwind gschwind commented Nov 3, 2022

As far as I know, the OGC standard state that request parameters name such as 'service' or 'request' are not case sensitive, but the values of those are case sensitive [1].

[1] OGC 06-121r3 §11.5.1

Overview

Related Issue / Discussion

Additional Information

This contribution is supported by MINES ParisTech.

Contribution Agreement

(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

  • I'd like to contribute [feature X|bugfix Y|docs|something else] to PyWPS. I confirm that my contributions to PyWPS will be compatible with the PyWPS license guidelines at the time of contribution.
  • I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

The OGC standard state that request parameters name such as 'service' or
'request' are not case sensitive, but the values of those are case
sensitive [1].

[1] OGC 06-121r3 §11.5.1
@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.053% when pulling 89e7a9f on gschwind:service-parameter-is-case-sensitive into 85ca819 on geopython:main.

@cehbrecht cehbrecht merged commit 15d23e4 into geopython:main Jan 27, 2023
@cehbrecht
Copy link
Collaborator

@gschwind thanks :)

@gschwind gschwind deleted the service-parameter-is-case-sensitive branch September 14, 2023 09:45
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

Successfully merging this pull request may close these issues.

None yet

4 participants