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

Unable to load layer from QGIS Server WMS Demo #1268

Closed
simboss opened this issue Nov 13, 2016 · 6 comments
Closed

Unable to load layer from QGIS Server WMS Demo #1268

simboss opened this issue Nov 13, 2016 · 6 comments

Comments

@simboss simboss added this to the R42016 milestone Nov 13, 2016
mbarto added a commit to mbarto/MapStore2 that referenced this issue Nov 16, 2016
@mbarto
Copy link
Contributor

mbarto commented Nov 16, 2016

Fixed by #1283
That service cannot work because it doesn't provide EPSG:3857, but now the error is clearly shown.
Other QGIS services work (e.g. http://demo.qgis.org/cgi-bin/qgis_mapserv.fcgi/swellendam/?map=/web/demos/swellendam/Swellendam.qgs&SERVICE=WMS&REQUEST=GetCapabilities&VERSION=1.1.1)

@mbarto mbarto closed this as completed in 6234225 Nov 16, 2016
@ghost ghost assigned mbarto Nov 16, 2016
@ghost ghost removed the pending review label Nov 16, 2016
@simboss simboss reopened this Dec 7, 2016
@simboss simboss modified the milestones: R52016, R42016 Dec 7, 2016
@simboss
Copy link
Member Author

simboss commented Dec 7, 2016

This problem is still open.

If you check this as an admin you will see that while we are parsing the GetCapabilities correctly and showing the map, but once we save and reopen, we are losing the portion of the URL that indicates which map we are using (..&map=/web/demos/sap/police.qgs&..) and so WMS requests fails afterwards.

Moreover I noticed that although the WMS version that we have in the GetCapabilities above is 1.1.1 we then make requests using 1.3.0. The version should not be changed between the GetCapabilities and the GetMap!

@mricca
Copy link
Contributor

mricca commented Dec 7, 2016

On my side, I solved the issue in this way.
mricca@bc45349

@simboss
Copy link
Member Author

simboss commented Dec 7, 2016

@mricca a PR would be very much appreciated :)

@mricca
Copy link
Contributor

mricca commented Dec 12, 2016

This is my pull requests:
#1342

@mbarto
Copy link
Contributor

mbarto commented Dec 14, 2016

Fixed by #1342

@mbarto mbarto closed this as completed Dec 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants