WMSValidator should use attributes() instead of getAttributes()

Description

The WMSValidator, when trying to validate a FeatureInfo, looks for the geometry attribute binding through the getAttributes() method.

https://github.com/geoserver/geoserver/blob/bf0345d228111c15aeff6f7c1bc87b4ed230af96/src/wms/src/main/java/org/geoserver/wms/WMSValidator.java#L65

This may fail if the Resource has not been loaded and the configuration does not contain the type of the attributes.

In order to avoid this issue (which may cause an NPE) the WMSValidator should call the method attributes() instead.

Environment

None

Activity

Show:
Andrea Aime
February 15, 2017, 11:48 AM

Mass closing all resolved issues not modified in the last 4 weeks

Andrea Aime
July 22, 2016, 6:28 AM

A week seaside with my 2 kids is more like lobotomy than a vacation, but I guess some of the end results are the same

Ben Caradoc-Davies
July 21, 2016, 6:07 PM

I am pleased that you enjoyed your holiday.

My definition of a real holiday is that, when you return, you have forgotten your password. Forgetting the month is even better.

Andrea Aime
July 21, 2016, 5:51 PM

Bah, we are in July... forget about me

Andrea Aime
July 21, 2016, 5:50 PM

Hmm... I based that on the release schedule here, which says 2.9.1 got released in July: https://github.com/geoserver/geoserver/wiki/Release-Schedule

Fixed

Assignee

Unassigned

Reporter

Alessio Fabiani

Fix versions