RFR: 8257722: Improve "keytool -printcert -jarfile" output [v3]
Hai-May Chao
hchao at openjdk.java.net
Thu Oct 28 16:17:51 UTC 2021
On Tue, 26 Oct 2021 23:23:35 GMT, Weijun Wang <weijun at openjdk.org> wrote:
>> Hai-May Chao has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Update output per review comment
>
> src/java.base/share/classes/sun/security/tools/keytool/Main.java line 2878:
>
>> 2876: out.println();
>> 2877: out.println();
>> 2878: out.println(rb.getString("Signature."));
>
> Please remove a newline as well. There needn't be 3 `println()` calls.
Done.
> src/java.base/share/classes/sun/security/tools/keytool/Resources.java line 469:
>
>> 467: {"one.in.many", "%1$s #%2$d of %3$d"},
>> 468: {"one.in.many1", "%1$s of signer #%2$d"},
>> 469: {"one.in.many2", "%1$s #%2$d of %3$d of signer #%4$d"},
>
> `certificate #1 of 2 of signer #1` is a little too complicated. The second number is not really necessary. I think `certificate #1 of signer #1` is enough.
Done. Changed the warning message.
-------------
PR: https://git.openjdk.java.net/jdk/pull/6126
More information about the security-dev
mailing list