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.

Sign up for GitHub

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

Jump to bottom

WMS provider expects SERVICE=WMTS keyword in uppercase to detect WMTS #36659

Closed
luqinghui opened this issue May 22, 2020 · 3 comments · Fixed by #36752
Closed

WMS provider expects SERVICE=WMTS keyword in uppercase to detect WMTS #36659

luqinghui opened this issue May 22, 2020 · 3 comments · Fixed by #36752
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Data Provider Related to specific vector, raster or mesh data providers

Comments

@luqinghui
Copy link

Describe the bug

I want to add a wms/wmts connection, but the QGIS report "failed to parse capabilities". the wmts url

How to Reproduce

  1. add a wmts service

image

  1. failed
    image

  2. layer -> add layer -> add wms/wmts layer -> connect
    image

  3. but when i paste the url to browser, the response:

image

QGIS and OS versions

Additional context
@luqinghui luqinghui added the Bug Either a bug report, or a bug fix. Let's hope for the latter! label May 22, 2020
@gioman
Copy link
Contributor

gioman commented May 22, 2020

@luqinghui I can't test this as from here (Europe) that URL does not open at all.

@gioman gioman added the Feedback Waiting on the submitter for answers label May 22, 2020
@luqinghui
Copy link
Author

@gioman thanks

@gioman gioman added Data Provider Related to specific vector, raster or mesh data providers and removed Feedback Waiting on the submitter for answers labels May 23, 2020
rouault added a commit to rouault/QGIS that referenced this issue May 26, 2020
@rouault
[WMS provider] Detection of SERVICE=WMTS in url must be case insensitive
cd437eb 
As mandated by OGC specs.

Fixes qgis#36659
@rouault rouault mentioned this issue May 26, 2020
Merged
@rouault
Copy link
Contributor

rouault commented May 26, 2020

the issue was related to "service=WMTS" in the connection URL. QGIS is currently case sensitive and expect "SERVICE=WMTS" full uppercase. Fix in #36752 to be case insensitive

@rouault rouault changed the title add WMTS service error WMS provider expects SERVICE=WMTS keyword in uppercase to detect WMTS May 26, 2020
rouault added a commit that referenced this issue May 27, 2020
@rouault
[WMS provider] Detection of SERVICE=WMTS in url must be case insensitive
97bac49 
As mandated by OGC specs.

Fixes #36659
nyalldawson pushed a commit to nyalldawson/QGIS that referenced this issue May 27, 2020
@rouault @nyalldawson
[WMS provider] Detection of SERVICE=WMTS in url must be case insensitive
29f0069 
As mandated by OGC specs.

Fixes qgis#36659

(cherry picked from commit 97bac49)
nyalldawson pushed a commit that referenced this issue Jun 19, 2020
@rouault @nyalldawson
[WMS provider] Detection of SERVICE=WMTS in url must be case insensitive
eef74d4 
As mandated by OGC specs.

Fixes #36659

(cherry picked from commit 97bac49)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Data Provider Related to specific vector, raster or mesh data providers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[WMS provider] Detection of SERVICE=WMTS in url must be case insensitive rouault/QGIS
3 participants
@rouault @gioman @luqinghui