Skip to content
This repository has been archived by the owner on Dec 11, 2023. It is now read-only.

WMS request: parameter name can be switched to upper case #143

Open
martin-bom opened this issue Apr 15, 2020 · 4 comments
Open

WMS request: parameter name can be switched to upper case #143

martin-bom opened this issue Apr 15, 2020 · 4 comments

Comments

@martin-bom
Copy link
Contributor

Hi Ana,

TerriaJS/terriajs#4223

Recently we tried to submit a WMS request to ALA Australia through Data Viewer but returned the following error:
Missing valid BBOX parameter

The underlying request to ALA does have the parameter "bbox" but is in lowercase.
May I know if there is a configuration in json which can convert the parameter name to uppercase?
Thank you.

Regards,
Martin

@AnaBelgun AnaBelgun mentioned this issue Apr 17, 2020
3 tasks
@KeyboardSounds
Copy link
Contributor

KeyboardSounds commented Jun 11, 2020

Hi @martin-bom,
Which WMS layer were you trying to access? Could you please give me the url?

@nickdos
Copy link

nickdos commented Jun 11, 2020

I'm not sure if this will help but I ran into this problem when I upgraded Leaflet to 1.4 (from 0.7.7) and the following code (from memory) was a work-around for the bug listed above.

https://github.com/AtlasOfLivingAustralia/regions/blob/master/grails-app/assets/javascripts/regions_page.js#L294-L313

Note the separation of params being sent via the URL (sldParam) and via Leaflet's params (layerParams) argument with the addition of Leaflet's uppercase: true option.

L.tileLayer.wms(config.spatialCacheUrl + sldParam, layerParams)

This will only help if TerriaJS is using LeafletJS code for WMS implementation...

@KeyboardSounds
Copy link
Contributor

We do use Leaflet for 2D, but our primary renderer is Cesium. Super useful to know though!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants