Re: [9] RFR 8056026 Debug security logging should print Provider used for each crypto operation

2014-09-15 Thread Sean Mullan
Can you also add similar log messages for MessageDigest, SecureRandom, and KeyStore? Otherwise looks good. Please add a noreg label. Also the fix is helpful to any platform and not just solaris/sparc so you should change those fields to be generic. --Sean On 09/12/2014 11:11 AM, Vincent Rya

Re: [9] RFR 8056026 Debug security logging should print Provider used for each crypto operation

2014-09-15 Thread Vincent Ryan
Originally I did support tracing for MessageDigest but removed it because of the huge quantity of log messages that were generated. Hashes are very widely used before an application even starts. SecureRandom is similar. Also I omitted KeyStore log messages because there is usually only a single

Re: [9] RFR 8056026 Debug security logging should print Provider used for each crypto operation

2014-09-15 Thread Sean Mullan
On 09/15/2014 11:34 AM, Vincent Ryan wrote: Originally I did support tracing for MessageDigest but removed it because of the huge quantity of log messages that were generated. Hashes are very widely used before an application even starts. SecureRandom is similar. Hmm, it would be nice to spec

Re: [9] RFR 8056026 Debug security logging should print Provider used for each crypto operation

2014-09-15 Thread Vincent Ryan
On 15 Sep 2014, at 16:50, Sean Mullan wrote: > On 09/15/2014 11:34 AM, Vincent Ryan wrote: >> Originally I did support tracing for MessageDigest but removed it because of >> the huge quantity of log messages that were generated. >> Hashes are very widely used before an application even starts.

RFR 8038089: TLS optional support for Kerberos cipher suites needs to be re-examine

2014-09-15 Thread Wang Weijun
Hi Xuelei Please review the latest code change at http://cr.openjdk.java.net/~weijun/8038089/webrev.04/ Compared with webrev.03, only the way the provider is loaded is changed, which is the static block on lines 50-71 of Krb5Helper.java. Thanks Max