RFR: JDK-8285504 Minor cleanup could be done in javax.net [v4]

Bradford Wetmore wetmore at openjdk.java.net
Thu Apr 28 16:17:50 UTC 2022


On Thu, 28 Apr 2022 15:47:44 GMT, Weijun Wang <weijun at openjdk.org> wrote:

>> Mark Powers has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains eight additional commits since the last revision:
>> 
>>  - Merge
>>  - Max and Brad comments
>>  - jaikiran comments
>>  - Alan Bateman comments
>>  - second iteration
>>  - Merge
>>  - Merge
>>  - first iteration
>
> src/java.base/share/classes/javax/net/ssl/KeyManagerFactory.java line 70:
> 
>> 68:         String type;
>> 69:         type = GetPropertyAction.privilegedGetProperty(
>> 70:             "ssl.KeyManagerFactory.algorithm");
> 
> So sorry I got it wrong here, this is a security property. `GetPropertyAction.privilegedGetProperty` is for system properties.

I just noticed the same.

-------------

PR: https://git.openjdk.java.net/jdk/pull/8384



More information about the security-dev mailing list