geosolutions-it / MapStore2

The solution to create and share maps, dashboards, geostories with 3D support on the web. And it is open-source too!
https://mapstore.geosolutionsgroup.com/
Other
515 stars 403 forks source link

add missing 'FORMAT' parameter to WMTS GetFeatureInfo requests #10596

Closed landryb closed 1 month ago

landryb commented 1 month ago

this is required by the WMTS spec to be the same format as would be used for a GetTile request, and this allows GFI requests to succeed on https://data.geopf.fr/wmts.

Description

See #10595

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

Issue

What is the current behavior?

10595

What is the new behavior?

GetFeatureInfo requests on https://data.geopf.fr/wmts works.

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

Other useful information

tdipisa commented 1 month ago

Thank you so much for your contribution @landryb

allyoucanmap commented 1 month ago

@landryb thanks for the contribution. @ElenaGallo please test this fix on dev and let us know if we can backport on 2024.02.xx, thanks

ElenaGallo commented 1 month ago

Test passed, @allyoucanmap please backport to 2024.02.xx. Thanks

allyoucanmap commented 1 month ago

@ElenaGallo @tdipisa backport ready for review https://github.com/geosolutions-it/MapStore2/pull/10606