Skip to content

Commit 0b16992

Browse files
authored
make dh group 31 default, support 22-24+31 (#12764)
1 parent 64ac082 commit 0b16992

3 files changed

Lines changed: 15 additions & 7 deletions

File tree

ui/src/views/network/CreateVpnCustomerGateway.vue

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -258,9 +258,13 @@ export default {
258258
'Group 15': 'modp3072',
259259
'Group 16': 'modp4096',
260260
'Group 17': 'modp6144',
261-
'Group 18': 'modp8192'
261+
'Group 18': 'modp8192',
262+
'Group 22': 'modp1024s160',
263+
'Group 23': 'modp2048s224',
264+
'Group 24': 'modp2048s256',
265+
'Group 31': 'curve25519'
262266
},
263-
ikeDhGroupInitialValue: 'Group 5(modp1536)',
267+
ikeDhGroupInitialValue: 'Group 31(curve25519)',
264268
isSubmitted: false,
265269
ikeversion: 'ike'
266270
}
@@ -275,12 +279,12 @@ export default {
275279
initForm () {
276280
this.formRef = ref()
277281
this.form = reactive({
278-
ikeEncryption: 'aes128',
282+
ikeEncryption: 'aes256',
279283
ikeHash: 'sha1',
280284
ikeversion: 'ike',
281-
ikeDh: 'Group 5(modp1536)',
282-
espEncryption: 'aes128',
283-
espHash: 'sha1',
285+
ikeDh: 'Group 31(curve 25519)',
286+
espEncryption: 'aes256',
287+
espHash: 'sha256',
284288
perfectForwardSecrecy: 'None',
285289
ikelifetime: '86400',
286290
esplifetime: '3600',

utils/src/main/java/com/cloud/utils/net/NetUtils.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1265,7 +1265,7 @@ public static boolean isValidS2SVpnPolicy(final String policyType, final String
12651265
if (group == null && policyType.toLowerCase().matches("ike")) {
12661266
return false; // StrongSwan requires a DH group for the IKE policy
12671267
}
1268-
if (group != null && !group.matches("modp1024|modp1536|modp2048|modp3072|modp4096|modp6144|modp8192")) {
1268+
if (group != null && !group.matches("modp1024|modp1536|modp2048|modp3072|modp4096|modp6144|modp8192|modp1024s160|modp2048s224|modp2048s256|curve25519")) {
12691269
return false;
12701270
}
12711271
}

utils/src/test/java/com/cloud/utils/net/NetUtilsTest.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -131,6 +131,10 @@ public void testIsValidS2SVpnPolicy() {
131131
assertTrue(NetUtils.isValidS2SVpnPolicy("ike", "3des-md5;modp1024"));
132132
assertTrue(NetUtils.isValidS2SVpnPolicy("ike", "3des-sha1;modp3072,aes128-sha1;modp1536"));
133133
assertTrue(NetUtils.isValidS2SVpnPolicy("ike", "3des-sha256;modp3072,aes128-sha512;modp1536"));
134+
assertTrue(NetUtils.isValidS2SVpnPolicy("ike", "aes256-sha256;modp1024s160"));
135+
assertTrue(NetUtils.isValidS2SVpnPolicy("ike", "aes256-sha256;modp2048s224"));
136+
assertTrue(NetUtils.isValidS2SVpnPolicy("ike", "aes256-sha256;modp2048s256"));
137+
assertTrue(NetUtils.isValidS2SVpnPolicy("ike", "aes256-sha256;curve25519"));
134138
assertFalse(NetUtils.isValidS2SVpnPolicy("ike", "aes128-sha1"));
135139
assertFalse(NetUtils.isValidS2SVpnPolicy("ike", "3des-sha1"));
136140
assertFalse(NetUtils.isValidS2SVpnPolicy("ike", "3des-sha1,aes256-sha1"));

0 commit comments

Comments
 (0)