Re: Code Review Request: 7169142: CookieHandler does not work with localhost

2013-06-13 Thread Chris Hegarty
I think your changes are correct here, and thanks for adding a test. -Chris. On 06/13/2013 12:09 AM, Kurchi Hazra wrote: Hi, The problem that this bug points out is that when a cookie is from a localhost and no domain is set, our implementation sets the default domain of the cookie to it

Code Review Request: 7169142: CookieHandler does not work with localhost

2013-06-12 Thread Kurchi Hazra
Hi, The problem that this bug points out is that when a cookie is from a localhost and no domain is set, our implementation sets the default domain of the cookie to its uri.getHost() [1], (which is localhost here), and since the domain now does not contain .local, the cookie gets rejected