On vendredi 2 décembre 2016 18:22:38 CET Andrea Aime wrote:
> On Fri, Dec 2, 2016 at 6:19 PM, Even Rouault <even.roua...@spatialys.com>
>
> wrote:
> > On vendredi 2 décembre 2016 17:52:37 CET Andrea Aime wrote:
> > > Hi,
> > >
> > > recently I've been wrestling with QGIS sending really funny requests to
> > >
> > > GeoServer, that are causing
> > >
> > > a not so nice ClassCastException. Here is a sample, it's a POST request
> >
> > at
> >
> > > this URL:
> > FYI, I've been emailed about the same issue but from the perspective of
> > QGIS. Ultimately, this should be fixed in QGIS since such requests are
> > wrong and I wouldn't expect a WFS server to make sense of them.
>
> Interesting... wondering how you're going to implement that? Like, wiping
> out everything past the ? won't work with mapserver, that
> normally needs a reference to a mapfile.
> But maybe just removing service/request/version would do the trick.
There are 2 issues in fact:
- &SERVICE=WFS is always appended at the end of the URL, which is not necessary
(but
probably not critical)
- the other parameters come from the user. In past QGIS versions, folks used to
put full
GetFeature queries (or at least something with VERSION & SRSNAME), so some
sanitizing is
necessary in that case. But yes I would strip SERVICE, REQUEST, VERSION,
TYPENAME(s),
SRSNAME, anything that is a WFS KVP.
>
> Do you already know in which version of QGIS the fix will land?
I'm not sure. Perhaps this can be addressed during the next sponsored QGIS
bugfixing
session. A first step would be to have a ticket in the QGIS tracker about that.
Even
--
Spatialys - Geospatial professional services
http://www.spatialys.com
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
