On Sat, Oct 12, 2013 at 11:16 AM, Justin Deoliveira <
jdeol...@boundlessgeo.com> wrote:
>
>
>
> On Sat, Oct 12, 2013 at 3:24 AM, Andrea Aime <andrea.a...@geo-solutions.it
> > wrote:
>
>> On Fri, Oct 11, 2013 at 11:46 PM, Ian Schneider <
>> ischnei...@boundlessgeo.com> wrote:
>>
>>> The complete death of the WfsDispatcher here:
>>> https://github.com/geoserver/geoserver/pull/356
>>>
>>
>> Hi Ian,
>> the patch looks good to me, those comments about supporting WfsDispatcher
>> are as old as GeoServer 1.4.0,
>> so I believe it's high time to let that thing go.
>>
>> I have a bit of hesitation about dropping support for WfsDispatcher on
>> 2.4.x, since it's a stable series,
>> but that url has been deprecated for so long that I'm good merging on
>> 2.4.x as well.
>>
>
> Agreed, my thought is that the only cases that this url would be used is
> cases like this where the client is simply following the url provided by
> the DescribeLayer endpoint. I vote for merge as is and if it really does
> cause a regression with clients needing the WfsDispatcher endpoint we can
> add the special case in the dispatcher (or perhaps hack it in with a
> callback) for the 2.4.x series.
>
As Justin notes, the only place the URL even showed up was the
DescribeLayer endpoint. Both workspace and layer virtual URLs (of the
...wfs/WfsDispatcher type) had been broken for some time, so my guess is
not many folks were actually using this.
>
>>
>>>
>>> Do you reckon it would be okay to cherry-pick to 2.3.x, too?
>>>
>>
>> 2.3.x won't does not have any more planned releases, so in a sense the
>> branch is dead, no objections
>> back porting this change there
>>
>
Cool, merged to 2.3.x, 2.4.x and master.
--
Ian Schneider
Software Engineer | Boundless
ischnei...@boundlessgeo.com
------------------------------------------------------------------------------
October Webinars: Code for Performance
Free Intel webinars can help you accelerate application performance.
Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from
the latest Intel processors and coprocessors. See abstracts and register >
http://pubads.g.doubleclick.net/gampad/clk?id=60134071&iu=/4140/ostg.clktrk
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
