RFR: 8286259: Password cleanup after KeyStore.PasswordProtection in P11KeyStore

2022-06-22 Thread Hai-May Chao
Please review the small update for JDK-8286259. - Commit messages: - 8286259: Password cleanup after KeyStore.PasswordProtection in P11KeyStore Changes: https://git.openjdk.org/jdk/pull/9254/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=9254&range=00 Issue: https://bugs.

Re: RFR: 8288985: P11TlsKeyMaterialGenerator works with ChaCha20-Poly1305 [v2]

2022-06-22 Thread zzambers
> TLS `*_CHACHA20_POLY1305_*` cipher suites are currently broken when > configuration with SunPKCS11 provider is used. I discovered this by my > ssl-tests testsuite [1]. > > > make TEST_PKCS11_FIPS=1 > SSLTESTS_SSL_CONFIG_FILTER=SunJSSE,Default,TLSv1.2,TLS_DHE_RSA_WITH_CHACHA20_POLY1305_SHA256

RFR: 8288985: P11TlsKeyMaterialGenerator works with ChaCha20-Poly1305

2022-06-22 Thread zzambers
TLS `*_CHACHA20_POLY1305_*` cipher suites are currently broken when configuration with SunPKCS11 provider is used. I discovered this by my ssl-tests testsuite [1]. make TEST_PKCS11_FIPS=1 SSLTESTS_SSL_CONFIG_FILTER=SunJSSE,Default,TLSv1.2,TLS_DHE_RSA_WITH_CHACHA20_POLY1305_SHA256 SSLTESTS_CUS

Re: RFR: 8215916: The failure reason of an optional JAAS LoginModule is not logged

2022-06-22 Thread Weijun Wang
On Wed, 22 Jun 2022 15:10:50 GMT, Weijun Wang wrote: >> Thanks you both for taking a look at this PR. >> I will investigate further for suitable suggested changes. >> Could you please suggest me how I can quickly check whether the changes I >> made are reflecting properly as expected ? > > There

Re: RFR: 8215916: The failure reason of an optional JAAS LoginModule is not logged

2022-06-22 Thread Weijun Wang
On Wed, 22 Jun 2022 14:12:34 GMT, Jayashree Huttanagoudar wrote: >> I have the same suggestion as Sean. In JAAS, login could succeed even if one >> optional LoginModule failed, and in this case the reason for that failure is >> lost (even with your current fix). Logging it somewhere might help

Re: RFR: 8215916: The failure reason of an optional JAAS LoginModule is not logged

2022-06-22 Thread Jayashree Huttanagoudar
On Wed, 22 Jun 2022 13:52:13 GMT, Weijun Wang wrote: >> src/java.base/share/classes/javax/security/auth/login/LoginContext.java line >> 887: >> >>> 885: (new java.io.PrintWriter(sw)); >>> 886: sw.flush(); >>> 887: le = new Logi

Re: RFR: 8215916: The failure reason of an optional JAAS LoginModule is not logged

2022-06-22 Thread Weijun Wang
On Wed, 22 Jun 2022 13:10:24 GMT, Sean Mullan wrote: >> Could you please review the changes? >> This patch is to address : >> https://bugs.openjdk.org/browse/JDK-8215916?jql=labels%20%3D%20starter-bug > > src/java.base/share/classes/javax/security/auth/login/LoginContext.java line > 887: > >>

Re: RFR: 8215916: The failure reason of an optional JAAS LoginModule is not logged

2022-06-22 Thread Sean Mullan
On Tue, 14 Jun 2022 19:07:24 GMT, Jayashree Huttanagoudar wrote: > Could you please review the changes? > This patch is to address : > https://bugs.openjdk.org/browse/JDK-8215916?jql=labels%20%3D%20starter-bug src/java.base/share/classes/javax/security/auth/login/LoginContext.java line 887:

Re: JSSE: SSLEngine reporting HandshakeStatus.FINISHED, disabling NewSessionTicket

2022-06-22 Thread Ben Smyth
On Tue, 24 May 2022 at 17:20, Ben Smyth wrote: > Javadoc advises HandshakeStatus.FINISHED is reported when "a call to > SSLEngine.wrap() / unwrap() ... finishes a handshake." As expected, > > * OpenJDK SSLEngine.wrap() reports HandshakeStatus.FINISHED on wrapping a > client's (TLS) FINISHED messag