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

URL encoding of '+' character in GetCoverage request #8

Closed
bermud opened this issue Jan 26, 2016 · 3 comments
Closed

URL encoding of '+' character in GetCoverage request #8

bermud opened this issue Jan 26, 2016 · 3 comments
Assignees
Labels
Milestone

Comments

@bermud
Copy link
Contributor

bermud commented Jan 26, 2016

When running cite tests for WCS 2.0.1, there is an error for test wcs2:get-kvp-core-req1-9.
when this request is send
http://?service=WCS&VERSION=2.0.1&request=GetCoverage&CoverageId=C0017&format=application/gml+xml

last elements are not correct, the '+' sign is not url encoded. it should be gml%2Bxml

@jinsongdi123
Copy link
Contributor

Thanks, The ctl is updated accordingly.

@coim32
Copy link
Contributor

coim32 commented Jan 27, 2016

you change was not correct.
change should apply on get request parameters, you applied for post requests.
for example line 326 of DataModel.xml

@jinsongdi123
Copy link
Contributor

Thanks, the ctl is updated accordingly

@bermud bermud modified the milestone: 1.10 Jan 29, 2016
@bermud bermud added bug and removed question labels Jan 29, 2016
@bermud bermud changed the title url encoding of '+' sign in GetCoverage request url encoding of '+' character in GetCoverage request Jan 29, 2016
@bermud bermud changed the title url encoding of '+' character in GetCoverage request URL encoding of '+' character in GetCoverage request Jan 29, 2016
@bermud bermud modified the milestones: 1.10, 1.9 Jan 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants