Thanks, I appreciate the clean up. Added some feedback for you to consider on your PR. -- Jody Garnett
On Fri, 21 May 2021 at 00:09, Roar Brænden <roar.brenden...@gmail.com> wrote: > Hi, > > Recently I got aware of a bug with the WMTS cascading. The bug appears > whenever a url set for wmts have it's own query parameters. Those would be > omitted. This is solved by this PR: > https://github.com/geotools/geotools/pull/3520 > > I see there is a release going on right now, and this PR might come a > little late. But it would be nice if someone took a look at it. > > When dealing with that bug, I discovered that some of the code could be > cleaned up. I have created a JIRA ticket for this. > > https://osgeo-org.atlassian.net/jira/software/c/projects/GEOT/issues/GEOT-6895 > > Now I'm wondering if anyone have any feedback on this, or if I should just > go on with my proposed changes. > > Best regards, > > Roar Brænden > _______________________________________________ > GeoTools-Devel mailing list > GeoTools-Devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/geotools-devel >
_______________________________________________ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel