Re: RFR: 8007632: DES/3DES keys support in PKCS12 keystore [v3]

2020-10-27 Thread Hai-May Chao
On Tue, 27 Oct 2020 17:59:38 GMT, Weijun Wang  wrote:

>> Alexey Bakhtin has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Fix order of OIDs
>
> Marked as reviewed by weijun (Reviewer).

Change looks good.

-

PR: https://git.openjdk.java.net/jdk/pull/877


Re: RFR: 8007632: DES/3DES keys support in PKCS12 keystore [v3]

2020-10-27 Thread Weijun Wang
On Tue, 27 Oct 2020 17:58:35 GMT, Alexey Bakhtin  wrote:

>> Hi All,
>> 
>> DES and DESede keys are supported by JKS/JCEKS but not supported by PKCS#12 
>> keystores.
>> This issue prevents the migration of legacy applications to PKCS#12 
>> keystore. For example, an application has some old 3DES keys that are 
>> required for certain legacy features. Java PKCS12 keystore does not support 
>> DES/3DES keys, thus, application can’t migrate to PKCS#12
>> This patch adds OIDs for the DES/DESede algorithms. It is the only changes 
>> required to support DES/3DES keys in the PKCS#12 keystore.
>> sun/security/pkcs12/P12SecretKey test is updated to verify new secret keys 
>> in the PKCS#12 keystore.
>
> Alexey Bakhtin has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Fix order of OIDs

Marked as reviewed by weijun (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/877


Re: RFR: 8007632: DES/3DES keys support in PKCS12 keystore [v3]

2020-10-27 Thread Alexey Bakhtin
> Hi All,
> 
> DES and DESede keys are supported by JKS/JCEKS but not supported by PKCS#12 
> keystores.
> This issue prevents the migration of legacy applications to PKCS#12 keystore. 
> For example, an application has some old 3DES keys that are required for 
> certain legacy features. Java PKCS12 keystore does not support DES/3DES keys, 
> thus, application can’t migrate to PKCS#12
> This patch adds OIDs for the DES/DESede algorithms. It is the only changes 
> required to support DES/3DES keys in the PKCS#12 keystore.
> sun/security/pkcs12/P12SecretKey test is updated to verify new secret keys in 
> the PKCS#12 keystore.

Alexey Bakhtin has updated the pull request incrementally with one additional 
commit since the last revision:

  Fix order of OIDs

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/877/files
  - new: https://git.openjdk.java.net/jdk/pull/877/files/94423b3d..efbd5a4f

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk=877=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk=877=01-02

  Stats: 4 lines in 1 file changed: 2 ins; 2 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/877.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/877/head:pull/877

PR: https://git.openjdk.java.net/jdk/pull/877