Re: RFR: 8277976: Break up SEQUENCE in X509Certificate::getSubjectAlternativeNames and X509Certificate::getIssuerAlternativeNames in otherName [v6]

2022-02-16 Thread Weijun Wang
On Thu, 17 Feb 2022 03:07:12 GMT, Weijun Wang wrote: >> src/java.base/share/classes/java/security/cert/X509Certificate.java line 595: >> >>> 593: * returned as byte arrays containing the ASN.1 DER encoded form >>> of the >>> 594: * name. There may be a third entry in the list for thei

Re: RFR: 8277976: Break up SEQUENCE in X509Certificate::getSubjectAlternativeNames and X509Certificate::getIssuerAlternativeNames in otherName [v6]

2022-02-16 Thread Weijun Wang
On Wed, 16 Feb 2022 21:56:36 GMT, Sean Mullan wrote: >> Weijun Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> retell the formats > > src/java.base/share/classes/java/security/cert/X509Certificate.java line 595: > >> 593: * re

Re: RFR: 8277976: Break up SEQUENCE in X509Certificate::getSubjectAlternativeNames and X509Certificate::getIssuerAlternativeNames in otherName [v6]

2022-02-16 Thread Sean Mullan
On Tue, 15 Feb 2022 19:51:57 GMT, Weijun Wang wrote: >> The enhancement adds two extra items in the `getSubjectAlternativeNames()` >> output for an OtherName. >> >> It also fix several errors: >> 1. In `OtherName.java`, `nameValue` should be the value inside `CONTEXT [0]` >> without the tag an

Re: RFR: 8277976: Break up SEQUENCE in X509Certificate::getSubjectAlternativeNames and X509Certificate::getIssuerAlternativeNames in otherName [v6]

2022-02-16 Thread Sean Mullan
On Tue, 15 Feb 2022 19:51:57 GMT, Weijun Wang wrote: >> The enhancement adds two extra items in the `getSubjectAlternativeNames()` >> output for an OtherName. >> >> It also fix several errors: >> 1. In `OtherName.java`, `nameValue` should be the value inside `CONTEXT [0]` >> without the tag an

Re: RFR: 8282017: sun/net/www/protocol/https/HttpsURLConnection/B6216082.java fails with "SocketException: Unexpected end of file from server" [v2]

2022-02-16 Thread Daniel Fuchs
> A seemingly innocuous change from > [JDK-8061729](https://bugs.openjdk.java.net/browse/JDK-8061729) is causing a > test failure. The fix is to simply revert that small change. Daniel Fuchs has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev

RFR: 8282017: sun/net/www/protocol/https/HttpsURLConnection/B6216082.java fails with "SocketException: Unexpected end of file from server"

2022-02-16 Thread Daniel Fuchs
A seemingly innocuous change from [JDK-8061729](https://bugs.openjdk.java.net/browse/JDK-8061729) is causing a test failure. The fix is to simply revert that small change. - Commit messages: - 8282017: sun/net/www/protocol/https/HttpsURLConnection/B6216082.java fails with "Socket

Re: RFR: 8282017: sun/net/www/protocol/https/HttpsURLConnection/B6216082.java fails with "SocketException: Unexpected end of file from server"

2022-02-16 Thread Daniel Fuchs
On Wed, 16 Feb 2022 21:10:41 GMT, Daniel Fuchs wrote: > A seemingly innocuous change from > [JDK-8061729](https://bugs.openjdk.java.net/browse/JDK-8061729) is causing a > test failure. The fix is to simply revert that small change. I am currently running tier2 tests in the CI -

Re: RFR: 8280494: (D)TLS signature schemes [v17]

2022-02-16 Thread Xue-Lei Andrew Fan
On Wed, 16 Feb 2022 19:21:58 GMT, Xue-Lei Andrew Fan wrote: >> This update is to support signature schemes customization for individual >> (D)TLS connection. Please review the CSR as well: >> CSR: https://bugs.openjdk.java.net/browse/JDK-8280495 >> RFE: https://bugs.openjdk.java.net/browse/JDK-

Re: RFR: 8280494: (D)TLS signature schemes [v17]

2022-02-16 Thread Xue-Lei Andrew Fan
> This update is to support signature schemes customization for individual > (D)TLS connection. Please review the CSR as well: > CSR: https://bugs.openjdk.java.net/browse/JDK-8280495 > RFE: https://bugs.openjdk.java.net/browse/JDK-8280494 > Release-note: https://bugs.openjdk.java.net/browse/JDK-8

JEP Review Request: TLS Certificate Compression

2022-02-16 Thread xueleifan(XueleiFan)
Hi all, The JDK Enhancement Proposal, TLS Certificate Compression, has been opened for community review. Detailed, please refer to the draft: https://bugs.openjdk.java.net/browse/JDK-8281710 Feel free to make comment and send your feedback to the alias. I may submit this JEP in the begin

deprecation of SecurityManager JEP 411

2022-02-16 Thread Bowes, David
Unfortunately I have to completely disagree with deprecating the SecurityManager: JEP411 “The Security Manager, then, had ambitions to protect against two kinds of threat: Malicious intent, especially in remote code, and accidental vulnerabilities, especially in local code. The threat of m

Integrated: JDK-8061729 : Update java/net tests to eliminate dependency on sun.net.www.MessageHeader and some other internal APIs

2022-02-16 Thread Mahendra Chhipa
On Wed, 13 Oct 2021 22:04:23 GMT, Mahendra Chhipa wrote: > There are some regression tests depending on sun.net.www.MessageHeader, the > internal API dependency should be removed. Some of other internal API > dependancies are removed in following issues : > JDK-8273142 > JDK-8268464 > JDK-82681

Re: RFR: JDK-8061729 : Update java/net tests to eliminate dependency on sun.net.www.MessageHeader and some other internal APIs [v6]

2022-02-16 Thread Daniel Fuchs
On Wed, 16 Feb 2022 13:55:43 GMT, Mahendra Chhipa wrote: >> There are some regression tests depending on sun.net.www.MessageHeader, the >> internal API dependency should be removed. Some of other internal API >> dependancies are removed in following issues : >> JDK-8273142 >> JDK-8268464 >> JDK

Re: RFR: JDK-8061729 : Update java/net tests to eliminate dependency on sun.net.www.MessageHeader and some other internal APIs [v6]

2022-02-16 Thread Mahendra Chhipa
s per comments. > 3. Corrected java doc comments. > 4. Removed use of is.available() method. Mach5 results https://mach5.us.oracle.com/mdash/jobs/mahendrachhipa-jdk-20220216-1342-29188746 - PR: https://git.openjdk.java.net/jdk/pull/5937

Re: RFR: JDK-8061729 : Update java/net tests to eliminate dependency on sun.net.www.MessageHeader and some other internal APIs [v6]

2022-02-16 Thread Mahendra Chhipa
> There are some regression tests depending on sun.net.www.MessageHeader, the > internal API dependency should be removed. Some of other internal API > dependancies are removed in following issues : > JDK-8273142 > JDK-8268464 > JDK-8268133 Mahendra Chhipa has updated the pull request incrementa