opensearch-project / dashboards-maps

:globe_with_meridians: User interface for the backend geospatial plugin which can be integrated with OpenSearch Dashboards and a home for all other map related features.
https://opensearch.org/docs/latest/dashboards/geojson-regionmaps/#custom-vector-maps-with-geojson
Apache License 2.0
17 stars 44 forks source link

[BUG] Maps produces WMS URLs without parameters after editing WMS layer configuration #601

Open lonnyj opened 5 months ago

lonnyj commented 5 months ago

Describe the bug

Editing a WMS layer in Maps causes Maps to generate WMS URLs without parameters.

To Reproduce Steps to reproduce the behavior:

  1. Ensure that you have some sort of data already loaded such as the Global Flight sample data.
  2. Go to Maps in Dashboards
  3. Click on Add layer
  4. Click on Custom map
  5. Populate the fields (WMS URL: https://mesonet.agron.iastate.edu/cgi-bin/wms/us/mrms_nn.cgi, WMS layers: mrms_p72h, WMS version: 1.3.0, WMS format: image/png, WMS CRS: EPSG:4326)
  6. Click on Settings
  7. Open the Developer Tools for your browser and select the Network tab
  8. Click the Update button
  9. Notice that the URLs contain parameters that resemble: https://mesonet.agron.iastate.edu/cgi-bin/wms/us/mrms_nn.cgi?service=WMS&version=1.3.0&request=GetMap&format=image/png&transparent=true&layers=mrms_p72h&styles=&crs=EPSG:4326&width=256&height=256&bbox=-11897270.578531113,5009377.085697312,-11271098.442818949,5635549.221409474
  10. Click on the newly created layer, New layer 2
  11. Change the WMS format to image/jpg.
  12. Click the Update button.
  13. Pan/zoom the map.
  14. Notice that the URLs are now all https://mesonet.agron.iastate.edu/cgi-bin/wms/us/mrms_nn.cgi without the parameters.

Expected behavior Users should be able to edit WMS layers and have Maps continue to populate the URL parameters.

OpenSearch Version 2.12.0

Dashboards Version 2.12.0

Plugins

None. The Security plugin is also disabled.

Screenshots

Screenshot 2024-03-17 at 9 44 45 AM

Host/Environment (please complete the following information):

Additional context

When working, I should be able to visualize the data like this: https://mesonet.agron.iastate.edu/cgi-bin/wms/us/mrms_nn.cgi?service=WMS&version=1.3.0&request=GetMap&format=image/png&transparentx=true&layers=mrms_p72h&styles=&crs=EPSG:4326&width=512&height=512&bbox=24,-126,50,-66

ashwin-pc commented 5 months ago

@opensearch-project/admin Can you move this to the maps dashboard plugin repo

dblock commented 2 months ago

Catch All Triage - 1 2 3 4 5