RFR: 6447816: Provider filtering (getProviders) is not working with OR'd conditions [v5]
Weijun Wang
weijun at openjdk.org
Tue Sep 6 14:33:07 UTC 2022
On Mon, 29 Aug 2022 20:13:42 GMT, Valerie Peng <valeriep at openjdk.org> wrote:
>> Existing provider filtering code only handles two standard attribute "KeySize" and "ImplementedIn", the rest are compared by exact match. Over time, more standard attributes are added which contain multiple values separated by "|". We should enhance the provider filtering code to better compare these.
>>
>> Documentation update for this is tracked separately under https://bugs.openjdk.org/browse/JDK-6447817.
>>
>> Thanks in advance for review~
>
> Valerie Peng has updated the pull request incrementally with one additional commit since the last revision:
>
> more refactoring.
src/java.base/share/classes/java/security/Security.java line 895:
> 893: if (standardName != null) {
> 894: key = serviceName + "." + standardName +
> 895: (attrName != null ? ' ' + attrName : "");
I'd rather use `(' ' + attrName)`. Always not sure about operation precedence when `?:` is used.
-------------
PR: https://git.openjdk.org/jdk/pull/10008
More information about the security-dev
mailing list