author | weijun |
Thu, 14 Dec 2017 20:19:34 +0800 | |
changeset 48333 | f47c18852172 |
permissions | -rw-r--r-- |
48333
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
1 |
/* |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
2 |
* Copyright (c) 2017, Oracle and/or its affiliates. All rights reserved. |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
4 |
* |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
5 |
* This code is free software; you can redistribute it and/or modify it |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
7 |
* published by the Free Software Foundation. |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
8 |
* |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
13 |
* accompanied this code). |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
14 |
* |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
15 |
* You should have received a copy of the GNU General Public License version |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
18 |
* |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
21 |
* questions. |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
22 |
*/ |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
23 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
24 |
/* |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
25 |
* @test |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
26 |
* @bug 8010125 8192988 |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
27 |
* @summary keytool should support -storepasswd for pkcs12 keystores |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
28 |
* @library /test/lib |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
29 |
* @build jdk.test.lib.SecurityTools |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
30 |
* jdk.test.lib.Utils |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
31 |
* jdk.test.lib.Asserts |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
32 |
* jdk.test.lib.JDKToolFinder |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
33 |
* jdk.test.lib.JDKToolLauncher |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
34 |
* jdk.test.lib.Platform |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
35 |
* jdk.test.lib.process.* |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
36 |
* @run main JKStoPKCS12 |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
37 |
*/ |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
38 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
39 |
import jdk.test.lib.Asserts; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
40 |
import jdk.test.lib.SecurityTools; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
41 |
import jdk.test.lib.process.OutputAnalyzer; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
42 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
43 |
import java.io.File; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
44 |
import java.nio.file.Files; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
45 |
import java.nio.file.Paths; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
46 |
import java.security.KeyStore; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
47 |
import java.util.Collections; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
48 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
49 |
public class JKStoPKCS12 { |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
50 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
51 |
static String srcStorePass, srcKeyPass; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
52 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
53 |
public static void main(String[] args) throws Exception { |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
54 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
55 |
// Part 1: JKS keystore with same storepass and keypass |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
56 |
genJKS("pass1111", "pass1111"); |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
57 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
58 |
// Change storepass, keypass also changes |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
59 |
convert("pass2222", null); |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
60 |
// You can keep storepass unchanged |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
61 |
convert("pass1111", null); |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
62 |
// Or change storepass and keypass both, explicitly |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
63 |
convert("pass2222", "pass2222"); |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
64 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
65 |
// Part 2: JKS keystore with different storepass and keypass |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
66 |
Files.delete(Paths.get("jks")); |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
67 |
genJKS("pass1111", "pass2222"); |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
68 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
69 |
// Can use old keypass as new storepass so new storepass and keypass are same |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
70 |
convert("pass2222", null); |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
71 |
// Or specify both storepass and keypass to brand new ones |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
72 |
convert("pass3333", "pass3333"); |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
73 |
// Or change storepass, keypass also changes. Remember to provide srckeypass |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
74 |
convert("pass1111", null); |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
75 |
} |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
76 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
77 |
// Generate JKS keystore with srcStorePass and srcKeyPass |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
78 |
static void genJKS(String storePass, String keyPass) |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
79 |
throws Exception { |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
80 |
srcStorePass = storePass; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
81 |
srcKeyPass = keyPass; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
82 |
kt("-genkeypair -keystore jks -storetype jks " |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
83 |
+ "-alias me -dname CN=Me -keyalg rsa " |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
84 |
+ "-storepass " + srcStorePass + " -keypass " + srcKeyPass) |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
85 |
.shouldHaveExitValue(0); |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
86 |
} |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
87 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
88 |
// Convert JKS to PKCS12 with destStorePass and destKeyPass (optional) |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
89 |
static void convert(String destStorePass, String destKeyPass) |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
90 |
throws Exception { |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
91 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
92 |
String cmd = "-importkeystore -noprompt" |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
93 |
+ " -srcstoretype jks -srckeystore jks" |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
94 |
+ " -destkeystore p12 -deststoretype pkcs12" |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
95 |
+ " -srcstorepass " + srcStorePass |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
96 |
+ " -deststorepass " + destStorePass; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
97 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
98 |
// Must import by alias (-srckeypass not available when importing all) |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
99 |
if (!srcStorePass.equals(srcKeyPass)) { |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
100 |
cmd += " -srcalias me"; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
101 |
cmd += " -srckeypass " + srcKeyPass; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
102 |
} |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
103 |
if (destKeyPass != null) { |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
104 |
cmd += " -destkeypass " + destKeyPass; |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
105 |
} |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
106 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
107 |
kt(cmd).shouldHaveExitValue(0); |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
108 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
109 |
// Confirms the storepass and keypass are all correct |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
110 |
KeyStore.getInstance(new File("p12"), destStorePass.toCharArray()) |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
111 |
.getKey("me", destStorePass.toCharArray()); |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
112 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
113 |
Files.delete(Paths.get("p12")); |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
114 |
} |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
115 |
|
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
116 |
static OutputAnalyzer kt(String arg) throws Exception { |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
117 |
return SecurityTools.keytool(arg); |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
118 |
} |
f47c18852172
8192988: keytool should support -storepasswd for pkcs12 keystores
weijun
parents:
diff
changeset
|
119 |
} |