Re: RFR: 8292681: Add JMH for ProtectionDomain

2022-08-23 Thread David Schlosnagle
On Fri, 19 Aug 2022 18:48:10 GMT, Eric Caspole wrote: > Add a JMH for ProtectionDomain related to current work on JDK-8292375. Also, > add the InMemoryJavaCompiler to the JMH jar, to generate the classes needed > for this test and will be useful for future class loading JMH too. test/micro/org

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

2022-08-23 Thread Jayashree Huttanagoudar
> Could you please review the changes? > This patch is to address : > https://bugs.openjdk.org/browse/JDK-8215916?jql=labels%20%3D%20starter-bug Jayashree Huttanagoudar has updated the pull request incrementally with one additional commit since the last revision: Address review comment -

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

2022-08-23 Thread Jayashree Huttanagoudar
On Tue, 23 Aug 2022 13:06:12 GMT, Weijun Wang wrote: >> Jayashree Huttanagoudar has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Address review comment for import statements > > test/jdk/javax/security/auth/login/LoginContext/OptionalJaas

Re: RFR: 6205692: (spec) javax.crypto.MacSpi.engineUpdate(ByteBuffer input): NPE should be specified [v7]

2022-08-23 Thread Kevin Driver
> add @exception message to indicate an NPE might be thrown when the input > parameter is null Kevin Driver has updated the pull request incrementally with one additional commit since the last revision: review comments - Changes: - all: https://git.openjdk.org/jdk/pull/9859/fi

Re: RFR: 6205692: (spec) javax.crypto.MacSpi.engineUpdate(ByteBuffer input): NPE should be specified [v6]

2022-08-23 Thread Bradford Wetmore
On Wed, 24 Aug 2022 00:19:50 GMT, Kevin Driver wrote: >> add @exception message to indicate an NPE might be thrown when the input >> parameter is null > > Kevin Driver has updated the pull request incrementally with one additional > commit since the last revision: > > remove unnecessary code

Re: RFR: 6205692: (spec) javax.crypto.MacSpi.engineUpdate(ByteBuffer input): NPE should be specified [v6]

2022-08-23 Thread Bradford Wetmore
On Wed, 24 Aug 2022 00:19:50 GMT, Kevin Driver wrote: >> add @exception message to indicate an NPE might be thrown when the input >> parameter is null > > Kevin Driver has updated the pull request incrementally with one additional > commit since the last revision: > > remove unnecessary code

Re: RFR: 6205692: (spec) javax.crypto.MacSpi.engineUpdate(ByteBuffer input): NPE should be specified [v5]

2022-08-23 Thread Kevin Driver
On Tue, 23 Aug 2022 23:55:33 GMT, Valerie Peng wrote: >> Kevin Driver has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - move test >> - new test for MacSpi demo purposes > > test/jdk/com/sun/crypto/provider/Mac/Test6205692.java line 112:

Re: RFR: 6205692: (spec) javax.crypto.MacSpi.engineUpdate(ByteBuffer input): NPE should be specified [v6]

2022-08-23 Thread Kevin Driver
> add @exception message to indicate an NPE might be thrown when the input > parameter is null Kevin Driver has updated the pull request incrementally with one additional commit since the last revision: remove unnecessary code - Changes: - all: https://git.openjdk.org/jdk/pull

RFR: 8292739: Invalid legacy entries may be returned by Provider.getServices() call

2022-08-23 Thread Valerie Peng
Invalid legacy services are screened and removed in Provider.getService(...) call. This fix is to also screen and remove the invalid legacy services when Provider.getServices() is called. Leveraged existing test case to test this particular scenario by adding one invalid legacy registration. T

Re: RFR: 4958071: (spec) confusing exception list for javax.crypto.Cipher.init(...) for [v2]

2022-08-23 Thread Kevin Driver
On Fri, 12 Aug 2022 23:05:11 GMT, Bradford Wetmore wrote: >> Kevin Driver has updated the pull request incrementally with one additional >> commit since the last revision: >> >> adding test to demonstrate exception being thrown > > LGTM, but needs test cases. @bradfordwetmore test added ---

Re: RFR: 4958071: (spec) confusing exception list for javax.crypto.Cipher.init(...) for [v2]

2022-08-23 Thread Kevin Driver
> Fix for JDK-4958071 to add additional @throws stanza to javadocs for all init > methods Kevin Driver has updated the pull request incrementally with one additional commit since the last revision: adding test to demonstrate exception being thrown - Changes: - all: https://git

Re: RFR: 6205692: (spec) javax.crypto.MacSpi.engineUpdate(ByteBuffer input): NPE should be specified [v5]

2022-08-23 Thread Valerie Peng
On Tue, 23 Aug 2022 23:34:27 GMT, Kevin Driver wrote: >> add @exception message to indicate an NPE might be thrown when the input >> parameter is null > > Kevin Driver has updated the pull request incrementally with two additional > commits since the last revision: > > - move test > - new te

Re: RFR: 6205692: (spec) javax.crypto.MacSpi.engineUpdate(ByteBuffer input): NPE should be specified [v5]

2022-08-23 Thread Kevin Driver
> add @exception message to indicate an NPE might be thrown when the input > parameter is null Kevin Driver has updated the pull request incrementally with two additional commits since the last revision: - move test - new test for MacSpi demo purposes - Changes: - all: https:/

Re: RFR: 8133816: Display extra SSLServerSocket info in debug mode [v3]

2022-08-23 Thread Xue-Lei Andrew Fan
On Tue, 23 Aug 2022 20:03:19 GMT, Sean Coffey wrote: >> So, do you want to make the log where the configuration happens? Logging in >> one place cannot have the accuracy debug log where the problem happens, and >> cannot easy the analysis of the debug. One just gets the configuration >> info

Re: RFR: 6205692: (spec) javax.crypto.MacSpi.engineUpdate(ByteBuffer input): NPE should be specified [v4]

2022-08-23 Thread Kevin Driver
> add @exception message to indicate an NPE might be thrown when the input > parameter is null Kevin Driver has updated the pull request incrementally with one additional commit since the last revision: code review comments -- replace all occurences of @exception with @throws -

Re: RFR: 8133816: Display extra SSLServerSocket info in debug mode [v2]

2022-08-23 Thread Sean Coffey
On Mon, 22 Aug 2022 12:52:54 GMT, Sean Mullan wrote: >> Weibing Xiao has updated the pull request incrementally with one additional >> commit since the last revision: >> >> add or remove the blank line according to the comments > > src/java.base/share/classes/sun/security/ssl/ServerHello.java

Re: RFR: 8133816: Display extra SSLServerSocket info in debug mode [v3]

2022-08-23 Thread Sean Coffey
On Thu, 18 Aug 2022 15:36:43 GMT, Xue-Lei Andrew Fan wrote: >> Intend to log all of the collected information in one place. It is easy for >> the developer to see the configuration of the server socket. Previously when >> the handshake is failing, no information is available for the server sock

Re: RFR: 8276660: Scalability bottleneck in java.security.Provider.getService() [v6]

2022-08-23 Thread Valerie Peng
On Tue, 23 Aug 2022 10:25:19 GMT, Josef Eisl wrote: >> Hmm, could be. Let me check into it and I will have to file a separate bug >> to address this since the changes have already been integrated. Thanks for >> the comments. > > Thanks for looking into this! Please let me if you open a new bug

Re: RFR: 8133816: Display extra SSLServerSocket info in debug mode [v3]

2022-08-23 Thread Weibing Xiao
On Tue, 23 Aug 2022 15:04:44 GMT, Weibing Xiao wrote: >> Log the debugging info for server cipher suites when setting javax.net.debug >> == ssl, handshake. > > Weibing Xiao has updated the pull request incrementally with one additional > commit since the last revision: > > update the code ac

Re: RFR: JDK-8291509 Minor cleanup could be done in sun.security

2022-08-23 Thread Mark Powers
On Mon, 22 Aug 2022 21:45:39 GMT, Mark Powers wrote: > https://bugs.openjdk.org/browse/JDK-8291509 Mach5 tier1 and tier2 tests all pass on all platforms. - PR: https://git.openjdk.org/jdk/pull/9972

Re: RFR: 8133816: Display extra SSLServerSocket info in debug mode [v3]

2022-08-23 Thread Weibing Xiao
> Log the debugging info for server cipher suites when setting javax.net.debug > == ssl, handshake. Weibing Xiao has updated the pull request incrementally with one additional commit since the last revision: update the code according to review comments - Changes: - all: https:

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

2022-08-23 Thread Weijun Wang
On Tue, 23 Aug 2022 06:39:44 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 > > Jayashree Huttanagoudar has updated the pull request incrementally with one >

Re: RFR: 8276660: Scalability bottleneck in java.security.Provider.getService() [v6]

2022-08-23 Thread Josef Eisl
On Fri, 19 Aug 2022 23:32:17 GMT, Valerie Peng wrote: >> src/java.base/share/classes/java/security/Provider.java line 1276: >> >>> 1274: } >>> 1275: if (serviceSet == null) { >>> 1276: ensureLegacyParsed(); >> >> Hi @valeriepeng! I believe that with this change, `get