Re: [PATCH] OpenVPN: Fix max-clients option

[Michael Tremer <michael.tremer@ipfire.org>]

[-- Attachment #1: Type: text/plain, Size: 1268 bytes --]

Reviewed-by: Michael Tremer <michael.tremer(a)ipfire.org>

Interesting is how long this has been in here unnoticed.

> On 4 Nov 2019, at 14:52, Erik Kapfer <ummeegge(a)ipfire.org> wrote:
> 
> Fix: Triggered by https://forum.ipfire.org/viewtopic.php?f=16&t=23551
> 
> Since the 'DHCP_WINS' cgiparam has been set for the max-client directive, changes in the WUI has not been adapted to server.conf.
> 
> Signed-off-by: Erik Kapfer <ummeegge(a)ipfire.org>
> ---
> html/cgi-bin/ovpnmain.cgi | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/html/cgi-bin/ovpnmain.cgi b/html/cgi-bin/ovpnmain.cgi
> index 970444a55..09f0dc42f 100644
> --- a/html/cgi-bin/ovpnmain.cgi
> +++ b/html/cgi-bin/ovpnmain.cgi
> @@ -355,10 +355,10 @@ sub writeserverconf {
>         print CONF "push \"dhcp-option WINS $sovpnsettings{DHCP_WINS}\"\n";
>     }
> 
> -    if ($sovpnsettings{DHCP_WINS} eq '') {
> +    if ($sovpnsettings{MAX_CLIENTS} eq '') {
> 	print CONF "max-clients 100\n";
>     }
> -    if ($sovpnsettings{DHCP_WINS} ne '') {
> +    if ($sovpnsettings{MAX_CLIENTS} ne '') {
> 	print CONF "max-clients $sovpnsettings{MAX_CLIENTS}\n";
>     }	
>     print CONF "tls-verify /usr/lib/openvpn/verify\n";
> -- 
> 2.12.2
>