[Geoserver-devel] Recent change with mime type handling breaking build and cite tests

Hi,
it seems the change landed after this pull request broke both the build and the
CITE tests:

https://github.com/geoserver/geoserver/pull/44

Build wise we have app-schema breaking with this error:

25-ott-2012 10.22.52 org.geoserver.test.WmsGetFeatureInfoTest
testGetFeatureInfoGML
INFO: WMS =GetFeatureInfo GML response:
<?xml version="1.0" encoding="UTF-8"?>
<ServiceExceptionReport version="1.3.0"
xmlns="http://www.opengis.net/ogc&quot;
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance&quot;
xsi:schemaLocation="http://www.opengis.net/ogc
http://localhost:8080/geoserver/schemas/wms/1.3.0/exceptions_1_3_0.xsd&quot;&gt;
    <ServiceException>
      java.lang.IllegalStateException: Cannot guess file extension for
invalid MIME type: 'geoserver-GetFeatureInfo'
Cannot guess file extension for invalid MIME type: 'geoserver-GetFeatureInfo'
</ServiceException>
</ServiceExceptionReport>

The CITE tests are also breaking on a mime type check, I don't have
proof that the commit
is what breaking them, but it would seem likely

Can we get a quick fix, or shall we revert the change?

Cheers
Andrea

--

Our support, Your Success! Visit http://opensdi.geo-solutions.it for
more information.

Ing. Andrea Aime
@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it

-------------------------------------------------------

Hmmm… I guess we should roll this back for now and revisit why the cite tests are breaking. Who wants to do that?

On Thu, Oct 25, 2012 at 3:27 AM, Andrea Aime <andrea.aime@anonymised.com> wrote:

Hi,
it seems the change landed after this pull request broke both the build and the
CITE tests:

https://github.com/geoserver/geoserver/pull/44

Build wise we have app-schema breaking with this error:

25-ott-2012 10.22.52 org.geoserver.test.WmsGetFeatureInfoTest
testGetFeatureInfoGML
INFO: WMS =GetFeatureInfo GML response:

<?xml version="1.0" encoding="UTF-8"?>



java.lang.IllegalStateException: Cannot guess file extension for
invalid MIME type: ‘geoserver-GetFeatureInfo’
Cannot guess file extension for invalid MIME type: ‘geoserver-GetFeatureInfo’

The CITE tests are also breaking on a mime type check, I don’t have
proof that the commit
is what breaking them, but it would seem likely

Can we get a quick fix, or shall we revert the change?

Cheers
Andrea

Our support, Your Success! Visit http://opensdi.geo-solutions.it for
more information.

Ing. Andrea Aime
@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it



Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_sfd2d_oct


Geoserver-devel mailing list
Geoserver-devel@anonymised.comsts.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel


Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.

Ok, trying to reproduce the issue on the CITE test server failed after a couple of tries, so I went ahead and reverted the changes to get the build green again. Locally, I was able to reproduce the problem in the application-schema tests so I investigated a bit more there.

After correcting the error message, I see this:

Cannot guess file extension for invalid MIME type: ‘application/vnd.ogc.gml/3.1.1’

According to my reading of http://portal.opengeospatial.org/files/?artifact_id=37743 the MIME type for GML3.1.1 should actually be “application/gml+xml; version=3.1.1” . Am I misunderstanding something? I don’t think it’s valid to have a slash in the type or subtype based on my reading of http://tools.ietf.org/html/rfc2045 .


David Winslow
OpenGeo - http://opengeo.org/

On Thu, Oct 25, 2012 at 10:13 AM, David Winslow <dwinslow@anonymised.com> wrote:

I’m happy to revert, but (of course) there’s a bug in the custom exception message I added - it is showing the partially constructed filename rather than the invalid MIME type string. It would probably be useful in tracking down the problem to rerun with the error string corrected.


David Winslow
OpenGeo - http://opengeo.org/

On Thu, Oct 25, 2012 at 10:09 AM, Justin Deoliveira <jdeolive@anonymised.com> wrote:

Hmmm… I guess we should roll this back for now and revisit why the cite tests are breaking. Who wants to do that?

On Thu, Oct 25, 2012 at 3:27 AM, Andrea Aime <andrea.aime@anonymised.com> wrote:

Hi,
it seems the change landed after this pull request broke both the build and the
CITE tests:

https://github.com/geoserver/geoserver/pull/44

Build wise we have app-schema breaking with this error:

25-ott-2012 10.22.52 org.geoserver.test.WmsGetFeatureInfoTest
testGetFeatureInfoGML
INFO: WMS =GetFeatureInfo GML response:

<?xml version="1.0" encoding="UTF-8"?>



java.lang.IllegalStateException: Cannot guess file extension for
invalid MIME type: ‘geoserver-GetFeatureInfo’
Cannot guess file extension for invalid MIME type: ‘geoserver-GetFeatureInfo’

The CITE tests are also breaking on a mime type check, I don’t have
proof that the commit
is what breaking them, but it would seem likely

Can we get a quick fix, or shall we revert the change?

Cheers
Andrea

Our support, Your Success! Visit http://opensdi.geo-solutions.it for
more information.

Ing. Andrea Aime
@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it



Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_sfd2d_oct


Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel


Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.


Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_sfd2d_oct


Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel