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

Add text/html to advertised GetFeatureInfo formats #318

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hugcis
Copy link

@hugcis hugcis commented Aug 25, 2023

This PR addresses this issue: #188

As far as I understand, this was only a matter of advertising the format in GetCapabilities, since the actual request is forwarded to the backend WMS server.

@jmckenna
Copy link
Member

jmckenna commented Sep 8, 2023

@hugcis thanks, can you also add that format to /tests/expected/wms_capabilities.xml ?

@jmckenna jmckenna added this to the 1.14.1 release milestone Sep 8, 2023
@jratike80
Copy link

Excuse me, but if these are the default formats

  <Format>text/plain</Format>
  <Format>application/vnd.ogc.gml</Format>

does that mean that they are always included in the GetCapabilities? Because the WMS standard does not mandate any format for GetFeatureInfo, the formats should actually be configured for each layer independently. But I do not see such configuration possibility in https://mapserver.org/mapcache/config.html.

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

Successfully merging this pull request may close these issues.

3 participants