RFR 8214179: Add groupname info into keytool -list and -genkeypair output
Sean Mullan
sean.mullan at oracle.com
Fri Nov 30 21:28:29 UTC 2018
Looks good.
--Sean
On 11/29/18 9:19 PM, Weijun Wang wrote:
> Webrev updated at
>
> https://cr.openjdk.java.net/~weijun/8214179/webrev.01/
>
> Very glad to remove the change of Resources.java.
>
> Thanks
> Max
>
>> On Nov 30, 2018, at 3:36 AM, Sean Mullan <sean.mullan at oracle.com> wrote:
>>
>> On 11/26/18 8:30 PM, Weijun Wang wrote:
>>> Ping
>>>> On Nov 21, 2018, at 9:56 PM, Weijun Wang <weijun.wang at oracle.com> wrote:
>>>>
>>>> Please take a review at
>>>>
>>>> https://cr.openjdk.java.net/~weijun/8214179/webrev.00/
>>>>
>>>> Before this change, `keytool -genkeypair -keyalg EC -groupname brainpoolP256r1` shows
>>>>
>>>> Generating -1 bit EC key pair and self-signed certificate...
>>>>
>>>> With this change, the message becomes
>>>>
>>>> Generating brainpoolP256r1 EC key pair and self-signed certificate...
>>
>> For the above, I would prefer to also see the size of the key, like in the message below, ex:
>>
>> Generating 256-bit EC (brainpoolP256r1) key pair and self-signed certificate...
>>
>>>>
>>>> Also, `keytool -list -v` can show
>>>>
>>>> Subject Public Key Algorithm: 256-bit EC(brainpoolP256r1) key
>>
>> I would put a space between "EC" and "(".
>>
>> --Sean
>>
>>>>
>>>> Thanks
>>>> Max
>>>>
>
More information about the security-dev
mailing list