From 605042ada8eeb60ba9dd339f9ebfd246fa95633a Mon Sep 17 00:00:00 2001 From: Ad Schellevis Date: Wed, 9 Aug 2023 17:09:34 +0200 Subject: [PATCH] VPN: OpenVPN - fix seperator for redirect-gateway attribute in Instances and CSO. closes https://github.com/opnsense/core/issues/6726 --- src/opnsense/mvc/app/models/OPNsense/OpenVPN/OpenVPN.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/opnsense/mvc/app/models/OPNsense/OpenVPN/OpenVPN.php b/src/opnsense/mvc/app/models/OPNsense/OpenVPN/OpenVPN.php index 46afad076..c15c3d260 100644 --- a/src/opnsense/mvc/app/models/OPNsense/OpenVPN/OpenVPN.php +++ b/src/opnsense/mvc/app/models/OPNsense/OpenVPN/OpenVPN.php @@ -138,7 +138,7 @@ class OpenVPN extends BaseModel $result['ovpn_servers'] = (string)$cso->servers; $result['common_name'] = (string)$cso->common_name; $result['description'] = (string)$cso->description; - $result['redirect_gateway'] = (string)$cso->redirect_gateway; + $result['redirect_gateway'] = str_replace(',', ' ', (string)$cso->redirect_gateway); $result['tunnel_network'] = (string)$cso->tunnel_network; $result['tunnel_networkv6'] = (string)$cso->tunnel_networkv6; @@ -447,7 +447,8 @@ class OpenVPN extends BaseModel // push options if (!empty((string)$node->redirect_gateway)) { - $options['push'][] = "\"redirect-gateway {$node->redirect_gateway}\""; + $redirect_gateway = str_replace(',', ' ', (string)$node->redirect_gateway); + $options['push'][] = "\"redirect-gateway {$redirect_gateway}\""; } if (!empty((string)$node->register_dns)) { $options['push'][] = "\"register-dns\"";