[15] RFR JDK-8242151 Improve OID mapping and reuse among JDK security providers for aliases registration
Valerie Peng
valerie.peng at oracle.com
Mon May 18 18:44:35 UTC 2020
Great, thanks much for the thorough review~
Valerie
On 5/15/2020 8:57 PM, Weijun Wang wrote:
> Well done. Everything looks fine to me.
>
> --Max
>
>> On May 16, 2020, at 5:47 AM, Valerie Peng <valerie.peng at oracle.com> wrote:
>>
>> Hi Max,
>>
>> I have updated the webrev (http://cr.openjdk.java.net/~valeriep/8242151/webrev.05/) to address your suggestion below. Touched classes are NamedCurve, CurveDB, ConstraintsParameters, and SunEC. The result of using the single method looks pretty good - clean and shorter code. :)
>>
>>> CurveDB.getNamesByOID is only used in ConstraintsParameters.getNamedCurveFromKey(), but we already have a NamedCurve there and you can directly use it without converting to nc.getObjectId().
>>>
>>> In fact, it looks like nc.getAliases() and nc.getName() are always used together. Can we just remove these 2 and add a new method nc.getNameAndAliases()? Then there will be no compatibility impact for getName() at all!
>>>
>> Thanks,
>> Valerie
>>
More information about the security-dev
mailing list