Re: Code Review Request: 6953455 CookieStore.add() cannot handle null URI parameter, contrary to the API specification

2012-07-04 Thread Neil Richards
On Fri, 2011-09-30 at 10:08 -0700, Kurchi Hazra wrote: > > > Hi, > > The CookieStore.add() method throws a Null Pointer Exception when > null is passed as the uri parameter, although this is allowed > according to the method spec. > > The exception is thrown because uri.getHost() is c

Re: Code Review Request: 6953455 CookieStore.add() cannot handle null URI parameter, contrary to the API specification

2012-07-04 Thread Chris Hegarty
Seems like a reasonable candidate for a backport. Are you encountering it in 7? -Chris On 4 Jul 2012, at 15:19, Neil Richards wrote: > On Fri, 2011-09-30 at 10:08 -0700, Kurchi Hazra wrote: >> >> >> Hi, >> >>The CookieStore.add() method throws a Null Pointer Exception when >> null is pas

InetAddress should utilize networkaddress.cache.ttl for getLocalHost() too

2012-07-04 Thread Deven You
Hi All, I noticed that InetAddress.getLocalHost() uses cache to improve the performance. However the default implementation is caching local host within 5 seconds. From the spec, networkaddress.cache.ttl should be used to control the cache behaviour and I think it is a better solution. For