Michael McMahon | 6 Apr 2010 13:42
Picon

Re: Code Review 6648001: Cancelling HTTP authentication causes subsequent deadlocks

Christopher Hegarty -Sun Microsystems Ireland wrote:
> Hi Michael,
>
> This is a forward port of a fix that you put in 6uXX.
>
> By exposing the key used in the authentication cache outside of 
> AuthenticationInfo, this allows all requests to be terminated 
> correctly, including ones, that have no username/password returned.
>
> Webrev:
>   http://cr.openjdk.java.net/~chegar/6648001/webrev.00/webrev/
>
> -Chris.
Looks fine.

- Michael.

weijun.wang | 16 Apr 2010 04:06
Picon

hg: jdk7/tl/jdk: 6937978: let keytool -gencert generate the chain

Changeset: db4fd2fdf196
Author:    weijun
Date:      2010-04-16 10:06 +0800
URL:       http://hg.openjdk.java.net/jdk7/tl/jdk/rev/db4fd2fdf196

6937978: let keytool -gencert generate the chain
Reviewed-by: mullan

! src/share/classes/sun/security/tools/KeyTool.java
! test/sun/security/tools/keytool/selfissued.sh

Weijun Wang | 16 Apr 2010 05:12
Picon

Re: CR 6911951 Updated, weijun.wang now responsible engineer, P4 java/classes_secu NTLM should be a supported Java SASL mechanism

Vinnie

Please take a review on this webrev:

   cr.openjdk.java.net/~weijun/6911951/webrev.00/

I've updated the spec a little by making NTLMv2 as the default version. It has been supported for a long time
and now default with Windows 7 and Server 2008 R2.

Networking guys, are you OK with the rewrite of NTLMAuthentication?

Thanks
Max

On Jan 4, 2010, at 12:30 PM, Weijun.Wang@... wrote:

>   
> *Change Request ID*: 6911951
> 
> *Synopsis*: NTLM should be a supported Java SASL mechanism
> 
> === *Description* ============================================================
> The NTLM support currently available for HTTP authentication should be generalized
> and made available as a Java SASL mechanism for use with other protocols.

lana.steuck | 18 Apr 2010 00:11
Picon

hg: jdk7/tl/corba: 4 new changesets

Changeset: 09a41111a401
Author:    mikejwre
Date:      2010-03-18 13:52 -0700
URL:       http://hg.openjdk.java.net/jdk7/tl/corba/rev/09a41111a401

Added tag jdk7-b86 for changeset 6253e28826d1

! .hgtags

Changeset: 39e14d2da687
Author:    mikejwre
Date:      2010-03-25 15:05 -0700
URL:       http://hg.openjdk.java.net/jdk7/tl/corba/rev/39e14d2da687

Added tag jdk7-b87 for changeset 09a41111a401

! .hgtags

Changeset: bb4424c5e778
Author:    mikejwre
Date:      2010-04-08 17:02 -0700
URL:       http://hg.openjdk.java.net/jdk7/tl/corba/rev/bb4424c5e778

Added tag jdk7-b88 for changeset 39e14d2da687

! .hgtags

Changeset: 56ce07b0eb47
Author:    mikejwre
Date:      2010-04-15 13:54 -0700
(Continue reading)

lana.steuck | 18 Apr 2010 00:19
Picon

hg: jdk7/tl/jaxws: 4 new changesets

Changeset: 3febd6fab2ac
Author:    mikejwre
Date:      2010-03-18 13:52 -0700
URL:       http://hg.openjdk.java.net/jdk7/tl/jaxws/rev/3febd6fab2ac

Added tag jdk7-b86 for changeset 512b0e924a5a

! .hgtags

Changeset: 8c666f8f3565
Author:    mikejwre
Date:      2010-03-25 15:05 -0700
URL:       http://hg.openjdk.java.net/jdk7/tl/jaxws/rev/8c666f8f3565

Added tag jdk7-b87 for changeset 3febd6fab2ac

! .hgtags

Changeset: bf3675aa7f20
Author:    mikejwre
Date:      2010-04-08 17:02 -0700
URL:       http://hg.openjdk.java.net/jdk7/tl/jaxws/rev/bf3675aa7f20

Added tag jdk7-b88 for changeset 8c666f8f3565

! .hgtags

Changeset: ead7c4566a00
Author:    mikejwre
Date:      2010-04-15 13:54 -0700
(Continue reading)

bradford.wetmore | 20 Apr 2010 23:25
Picon
Favicon

hg: jdk7/tl/jdk: 6945604: wrong error message in CardImpl.java

Changeset: d8ad2da3ecf3
Author:    wetmore
Date:      2010-04-20 14:24 -0700
URL:       http://hg.openjdk.java.net/jdk7/tl/jdk/rev/d8ad2da3ecf3

6945604: wrong error message in CardImpl.java
Reviewed-by: mullan

! src/share/classes/sun/security/smartcardio/CardImpl.java

alan.bateman | 24 Apr 2010 18:40
Picon

hg: jdk7/tl/corba: 6939646: Remove obsolete com.sun.corba.se.internal.io package

Changeset: cefae6b4a590
Author:    alanb
Date:      2010-04-24 17:09 +0100
URL:       http://hg.openjdk.java.net/jdk7/tl/corba/rev/cefae6b4a590

6939646: Remove obsolete com.sun.corba.se.internal.io package
Reviewed-by: ohair

! make/sun/corba/Makefile
- make/sun/corba/core/Makefile
- make/sun/corba/core/mapfile-vers
- src/share/classes/com/sun/corba/se/internal/io/IIOPInputStream.java
- src/share/classes/com/sun/corba/se/internal/io/IIOPOutputStream.java
- src/share/classes/com/sun/corba/se/internal/io/LibraryManager.java
- src/share/classes/com/sun/corba/se/internal/io/ObjectStreamClass.java
- src/share/native/com/sun/corba/se/internal/io/ioser.c

Chris Hegarty | 26 Apr 2010 11:54
Picon
Favicon

Request for Review 6886723: light weight http server doesn't return correct status code for HEAD requests

Hi Michael,

The HttpExchange implementation, sun.net.httpserver.ExchangeImpl, does 
not correctly handle HEAD requests. The test, described in the 
'description section' of 6886723, invokes sendResponseHeaders with a 0 
responseLength, i.e. chunked encoding. The "Transfer-encoding: chunked" 
header should be added to the response headers but no 
ChunkedOutputStream should be created. The current implementation does 
send an empty chunk causing subsequent requests on the same persistent 
connection to have problems.

The solution is to correctly set response headers, but create no 
corresponding OutputStreams when dealing with HEAD requests.

Webrev:
   http://cr.openjdk.java.net/~chegar/6886723/webrev.00/webrev/

Thanks,
-Chris

Chris Hegarty | 26 Apr 2010 15:02
Picon
Favicon

Request for Review 6718504: IN6_IS_ADDR_ANY tests only 12 bytes of 16-byte address

Hi Alan,

net_util_md.h incorrectly defines the macro IN6_IS_ADDR_ANY to only look 
at 12 bytes of the 16 byte address. This causes a bind to ::1 to be 
treated as if it were a bind to the wildcard address.

Note: with the addition of the dual stack native implementation on Vista 
this code is only executed when running IPv6 on Windows XP.

Webrev:
   http://cr.openjdk.java.net/~chegar/6718504/webrev.00/webrev/

-Chris.

Alan Bateman | 26 Apr 2010 15:37
Picon

Re: Request for Review 6718504: IN6_IS_ADDR_ANY tests only 12 bytes of 16-byte address

Chris Hegarty wrote:
> Hi Alan,
>
> net_util_md.h incorrectly defines the macro IN6_IS_ADDR_ANY to only 
> look at 12 bytes of the 16 byte address. This causes a bind to ::1 to 
> be treated as if it were a bind to the wildcard address.
>
> Note: with the addition of the dual stack native implementation on 
> Vista this code is only executed when running IPv6 on Windows XP.
>
> Webrev:
>   http://cr.openjdk.java.net/~chegar/6718504/webrev.00/webrev/
>
> -Chris.
The change looks fine to me. In the test it might be simpler to put the 
lookup of the IPv6 loopback address in the main method. Also, you can 
probably call getLocalAddress.

-Alan.


Gmane