Right, saw it.
On Tue, Dec 20, 2011 at 9:56 PM, Gabriel Roldan <[email protected]> wrote:
> On Wed, Dec 21, 2011 at 1:54 AM, Justin Deoliveira <[email protected]>
> wrote:
> > Thanks for getting that in Gabriel.
> >
> > So if that one resolves http://jira.codehaus.org/browse/GEOS-4908 for
> the
> > time being i will push out the release tomorrow.
> Yes, just beware there's just another commit after that fixing a but
> in SimpleHttpClient to respect the connect/read timeouts.
>
> Cheers,
> Gabriel
> >
> > -Justin
> >
> >
> > On Tue, Dec 20, 2011 at 9:39 PM, Gabriel Roldan <[email protected]>
> wrote:
> >>
> >> Hi Andrea,
> >>
> >> On Tue, Dec 20, 2011 at 11:20 AM, Andrea Aime
> >> <[email protected]> wrote:
> >> >> There are two things that might still be considered regressions in
> >> >> there:
> >> >> - are we sure the password has to be specified along with the
> username?
> >> >> It's a bit of a stretch, but why don't we let it go and have the
> proxy
> >> >> decide if the password is required or not?
> >>
> >> ok, the patch committed to GeoTools just takes that approach.
> >>
> >> >> - the http.nonProxyHosts is not supported. In simple setups it's not
> >> >> required, and I agree it might not be
> >> >> common in usage, but consider a case in which you have a server
> >> >> that's
> >> >> cascading two other servers,
> >> >> one that is local (a legacy in house WMS server that GeoServer is
> >> >> acting
> >> >> as a front-end for) and
> >> >> another is remote, with the proxy you can access the remote but not
> >> >> the
> >> >> local, and vice versa.
> >> >>
> >> >> It might be a good idea to have some sort of configuration allowing
> the
> >> >> usage of the java based http client instead so that also this case is
> >> >> covered, or we can try to match the nonProxyHost list
> >> >> (which would require some parsing and some regex usage, as the doc
> say
> >> >> "The value can be a list of hosts, each seperated by a |, and in
> >> >> addition a
> >> >> wildcard character (*) can be used for matching. For example:
> >> >> -Dhttp.nonProxyHosts="*.foo.com|localhost".")
> >> >
> >> Supporting nonProxyHosts would be good, but too risky this close to
> >> the release. At least I don't have the time to commit to make sure it
> >> works properly.
> >>
> >> But I do can add a config option in the GeoServer UI to enable one or
> >> the other (called "Use connection pooling" for example, a checkbox
> >> that enables/disables the maxConnections input, and under the hood
> >> instructs which http backend to use?).
> >>
> >> For when you wake up that should already be up on svn. Let me know if
> >> there's something to change.
> >>
> >> Cheers,
> >> Gabriel
> >> >
> >> > Btw, if you decide to go either way (config or parsing) just go for
> it,
> >> > no
> >> > need for further reviews or discussions on my part.
> >>
> >>
> >>
> >> --
> >> Gabriel Roldan
> >> OpenGeo - http://opengeo.org
> >> Expert service straight from the developers.
> >
> >
> >
> >
> > --
> > Justin Deoliveira
> > OpenGeo - http://opengeo.org
> > Enterprise support for open source geospatial.
> >
>
>
>
> --
> Gabriel Roldan
> OpenGeo - http://opengeo.org
> Expert service straight from the developers.
>
--
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
------------------------------------------------------------------------------
Write once. Port to many.
Get the SDK and tools to simplify cross-platform app development. Create
new or port existing apps to sell to consumers worldwide. Explore the
Intel AppUpSM program developer opportunity. appdeveloper.intel.com/join
http://p.sf.net/sfu/intel-appdev
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
