Using type "other" for MetadataURL will lead to invalid GetCapabilities for WMS 1.1.1

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.1.0
  • Fix Version/s: 2.1.3
  • Component/s: WMS
  • Labels:
    None

Description

If you set type to "other" in MetadataURL, it will lead to invalid XML in the WMS 1.1.1 GetCapabilities response and break parsing of the GetCapabilities in Internet Explorer. This is not an issue for WMS 1.3.0.

SystemID: http://localhost:8080/geoserver/wms?service=WMS&request=GetCapabilities&version=1.1.1
Engine name: Xerces
Severity: error
Description: Attribute "type" with value "other" must have a value from the list "TC211 FGDC ".
Start location: 200:27
End location: 200:34

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Gabriel Roldán added a comment -

Attached is the patch for 2.1.x. It is pretty straight forward, but in any case I've added a small note to the metadata link editor warning any non FGDC/TC211 link won't show up on the WMS 1.1.1 capabilities.

Show
Gabriel Roldán added a comment - Attached is the patch for 2.1.x. It is pretty straight forward, but in any case I've added a small note to the metadata link editor warning any non FGDC/TC211 link won't show up on the WMS 1.1.1 capabilities.
Hide
Permalink
Gabriel Roldán added a comment -

Just added jdeolive and aaime as watchers in case they can take a look at the patch/have some feedback.

Show
Gabriel Roldán added a comment - Just added jdeolive and aaime as watchers in case they can take a look at the patch/have some feedback.
Hide
Permalink
Justin Deoliveira added a comment -

Patch looks good to me.

Show
Justin Deoliveira added a comment - Patch looks good to me.
Hide
Permalink
Gabriel Roldán added a comment -

attached patch had wrong name. Called GEOS-4673.patch now

Show
Gabriel Roldán added a comment - attached patch had wrong name. Called GEOS-4673.patch now
Hide
Permalink
Gabriel Roldán added a comment -

fixed at r16414 on 2.1.x and r16415 on trunk

Show
Gabriel Roldán added a comment - fixed at r16414 on 2.1.x and r16415 on trunk

People

  • Assignee:
    Gabriel Roldán
    Reporter:
    Bart van den Eijnden
Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: