Re: Code review request: 8005523: Unbound krb5 for TLS

2013-04-15 Thread Xuelei Fan
Using JSSE debug option, system property "javax.net.debug=all", and check the log about whether the session is resumed or not. If you want the check in test code, it may be ok to check the session ID (SSLSession.getId()). Xuelei On 4/16/2013 1:11 PM, Weijun Wang wrote: > I've found something str

Re: Code review request: 8005523: Unbound krb5 for TLS

2013-04-15 Thread Weijun Wang
I've found something strange. The test has tried two SSL connections. When server is unbound, the client always requests for an initiate ServicePermission. When server is bound, only the first connection requests for the permission. Is it possible the server invalidate the SSLSession when it's

hg: jdk8/tl/jdk: 8011800: Add java.util.Objects.requireNonNull(T, Supplier)

2013-04-15 Thread joe . darcy
Changeset: baaa706d7677 Author:darcy Date: 2013-04-15 18:31 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/baaa706d7677 8011800: Add java.util.Objects.requireNonNull(T, Supplier) Reviewed-by: alanb, dholmes, mduigou ! src/share/classes/java/util/Objects.java ! test/java/util

hg: jdk8/tl/jdk: 8008509: 6588413 changed JNIEXPORT visibility for GCC on HSX, jdk's jni_md.h needs similar change

2013-04-15 Thread martinrb
Changeset: 4ed143ddbb8a Author:martin Date: 2013-04-15 14:07 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/4ed143ddbb8a 8008509: 6588413 changed JNIEXPORT visibility for GCC on HSX, jdk's jni_md.h needs similar change Summary: Define JNIEXPORT to use "default" visibility wh

hg: jdk8/tl/nashorn: 9 new changesets

2013-04-15 Thread sundararajan . athijegannathan
Changeset: 635a93b61d34 Author:hannesw Date: 2013-04-10 14:00 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/nashorn/rev/635a93b61d34 8011714: Regexp decimal escape handling still not correct Reviewed-by: lagergren, attila ! src/jdk/nashorn/internal/runtime/regexp/RegExpScanner.java

hg: jdk8/tl/langtools: 7015104: use new subtype of TypeSymbol for type parameters

2013-04-15 Thread joel . franck
Changeset: 137994c189e5 Author:jfranck Date: 2013-04-12 12:05 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/137994c189e5 7015104: use new subtype of TypeSymbol for type parameters Reviewed-by: jjg, mcimadamore ! src/share/classes/com/sun/tools/javac/code/Symbol.java !

hg: jdk8/tl/langtools: 8009686: Generated javadoc documentation should be able to display type annotation on an array

2013-04-15 Thread bhavesh . x . patel
Changeset: f10cffab99b4 Author:bpatel Date: 2013-04-13 18:48 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/f10cffab99b4 8009686: Generated javadoc documentation should be able to display type annotation on an array Reviewed-by: jjg ! src/share/classes/com/sun/javadoc

Re: Ignore SSL server_name extension alerts (Bug 7127374)

2013-04-15 Thread Bernd Eckenfels
Hello, Will answer in detail, but just make sure People are Not confused: the TSA seems to be fixed meanwhile. Gruss Bernd -- bernd.eckenfels.net Am 15.04.2013 um 05:40 schrieb Xuelei Fan : > On 1/21/2013 7:25 AM, Bernd Eckenfels wrote: >> Hello, >> >> quite some time back I reported a bug,

hg: jdk8/tl/langtools: 6 new changesets

2013-04-15 Thread maurizio . cimadamore
Changeset: b26f36a7ae3b Author:mcimadamore Date: 2013-04-15 14:11 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/b26f36a7ae3b 8011383: Symbol.getModifiers omits ACC_ABSTRACT from interface with default methods Summary: Fixup for default method modifiers erroneously app

Re: Code review request: 8005523: Unbound krb5 for TLS

2013-04-15 Thread Xuelei Fan
On 4/15/2013 8:57 PM, Weijun Wang wrote: >>> http://cr.openjdk.java.net/~weijun/8005523/webrev.01/ >>> >> >> setupKerberosKeys() @ServerHandshaker.java: >> --- >> I would like to reserve the permission checking for bound krb5 here. >> The checking is don

Re: Code review request: 8005523: Unbound krb5 for TLS

2013-04-15 Thread Weijun Wang
http://cr.openjdk.java.net/~weijun/8005523/webrev.01/ setupKerberosKeys() @ServerHandshaker.java: --- I would like to reserve the permission checking for bound krb5 here. The checking is done while parse ClientHello, and is useful to select the right

Re: Code review request: 8005523: Unbound krb5 for TLS

2013-04-15 Thread Xuelei Fan
On 4/15/2013 2:18 PM, Weijun Wang wrote: > > > On 4/15/13 1:38 PM, Xuelei Fan wrote: Do you want to file a simple enhancement request (CCC)? >>> >>> Why CCC? This is all internal. >>> >> Yes, it is optional. I think, now it can accept unbound principal in >> server side, it is an enhancement