Re: RFR: 8257497: Key identifier compliance issue [v5]

2021-02-16 Thread Hai-May Chao
On Tue, 16 Feb 2021 18:33:52 GMT, Sean Mullan wrote: >> Hai-May Chao has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Reduced one param to createV3Extensions > > src/java.base/share/classes/sun/security/tools/keytool/Main.java line 1484:

Re: RFR: 8257497: Key identifier compliance issue [v5]

2021-02-16 Thread Sean Mullan
On Fri, 12 Feb 2021 20:42:03 GMT, Hai-May Chao wrote: >> This change is made for compliance with RFC 5280 section 4.2.1.1 for >> Authority Key Identifier extension. > > Hai-May Chao has updated the pull request incrementally with one additional > commit since the last revision: > > Reduced

Re: RFR: 8257497: Key identifier compliance issue [v5]

2021-02-12 Thread Hai-May Chao
On Fri, 12 Feb 2021 21:01:48 GMT, Weijun Wang wrote: >> Hai-May Chao has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Reduced one param to createV3Extensions > > LGTM. Thanks for the review, Max! - PR:

Re: RFR: 8257497: Key identifier compliance issue [v5]

2021-02-12 Thread Weijun Wang
On Fri, 12 Feb 2021 20:42:03 GMT, Hai-May Chao wrote: >> This change is made for compliance with RFC 5280 section 4.2.1.1 for >> Authority Key Identifier extension. > > Hai-May Chao has updated the pull request incrementally with one additional > commit since the last revision: > > Reduced

Re: RFR: 8257497: Key identifier compliance issue [v5]

2021-02-12 Thread Hai-May Chao
> This change is made for compliance with RFC 5280 section 4.2.1.1 for > Authority Key Identifier extension. Hai-May Chao has updated the pull request incrementally with one additional commit since the last revision: Reduced one param to createV3Extensions - Changes: - all: