We always welcome PRs for open issues. This sounds as if there is a general
potential for SQL injection in the layer names that we should be protecting
against,

Ian

On Wed, 14 Jun 2023 at 10:09, Mike Bryant via GeoTools-Devel <
geotools-devel@lists.sourceforge.net> wrote:

> Dear all,
>
> https://osgeo-org.atlassian.net/browse/GEOT-6266
>
> I've recently run into GEOT-6266 attempting to use the GeoPackage export
> plugin with GeoServer 2.23.1, since some of our layer names contain
> hyphens.
>
> Looking at the relevant code in GeoPackage.java this could be resolved
> by quoting the table name in a few SQLite queries, and I'm happy to
> submit PRs for this if that would be welcome. However, perhaps there are
> other considerations here I'm not aware of? I guess there's the larger
> issue of compatibility and best-practices for layer naming but I'm not
> sure where that is supposed to be enforced.
>
> Many thanks,
> Mike
>
>
>
> _______________________________________________
> GeoTools-Devel mailing list
> GeoTools-Devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>


-- 
Ian Turton
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to