From 1dacbb13a7fff0fd941e1de61900df7320189740 Mon Sep 17 00:00:00 2001 From: Nikos Mavrogiannopoulos Date: Thu, 28 May 2020 22:55:26 +0200 Subject: [PATCH] tests: added test cases for no-route in group and main configuration Signed-off-by: Nikos Mavrogiannopoulos --- tests/Makefile.am | 5 +- tests/data/group-config/group1 | 1 + tests/data/no-route-default.config | 192 +++++++++++++++++++++++++++++ tests/data/no-route-group.config | 192 +++++++++++++++++++++++++++++ tests/no-route-default | 86 +++++++++++++ tests/no-route-group | 86 +++++++++++++ 6 files changed, 560 insertions(+), 2 deletions(-) create mode 100644 tests/data/group-config/group1 create mode 100644 tests/data/no-route-default.config create mode 100644 tests/data/no-route-group.config create mode 100755 tests/no-route-default create mode 100755 tests/no-route-group diff --git a/tests/Makefile.am b/tests/Makefile.am index 354a2715..2b440d56 100644 --- a/tests/Makefile.am +++ b/tests/Makefile.am @@ -39,7 +39,8 @@ EXTRA_DIST = certs/ca-key.pem certs/ca.pem ns.sh common.sh certs/server-cert.pem sleep-connect-script data/test-psk-negotiate.config data/test-group-name.config \ connect-ios-script data/apple-ios.config certs/kerberos-cert.pem \ data/kdc.conf data/krb5.conf data/k5.KERBEROS.TEST data/kadm5.acl \ - data/ipv6-iface.config + data/ipv6-iface.config data/no-route-default.config data/no-route-group.config \ + data/group-config/group1 SUBDIRS = docker-ocserv @@ -94,7 +95,7 @@ dist_check_SCRIPTS += test-pass test-pass-cert test-cert test-group-pass \ test-enc-key test-sighup-key-change test-get-cert test-san-cert \ test-gssapi test-pass-opt-cert test-cert-opt-pass test-gssapi-opt-pass \ test-gssapi-opt-cert haproxy-auth test-maintenance \ - test-group-name + test-group-name no-route-default no-route-group if HAVE_CWRAP_PAM dist_check_SCRIPTS += test-pam test-pam-noauth diff --git a/tests/data/group-config/group1 b/tests/data/group-config/group1 new file mode 100644 index 00000000..6ddd00c3 --- /dev/null +++ b/tests/data/group-config/group1 @@ -0,0 +1 @@ +no-route = 192.168.98.0/24 diff --git a/tests/data/no-route-default.config b/tests/data/no-route-default.config new file mode 100644 index 00000000..864345f5 --- /dev/null +++ b/tests/data/no-route-default.config @@ -0,0 +1,192 @@ +# User authentication method. Could be set multiple times and in that case +# all should succeed. +# Options: certificate, pam. +auth = "certificate" +#auth = "plain[@SRCDIR@/data/test-group.passwd]" +#auth = "pam" + +isolate-workers = @ISOLATE_WORKERS@ + +# A banner to be displayed on clients +#banner = "Welcome" + +# Use listen-host to limit to specific IPs or to the IPs of a provided hostname. +#listen-host = [IP|HOSTNAME] + +use-dbus = no + +# Limit the number of clients. Unset or set to zero for unlimited. +#max-clients = 1024 +max-clients = 16 + +# Limit the number of client connections to one every X milliseconds +# (X is the provided value). Set to zero for no limit. +#rate-limit-ms = 100 + +# Limit the number of identical clients (i.e., users connecting multiple times) +# Unset or set to zero for unlimited. +max-same-clients = 2 + +# TCP and UDP port number +tcp-port = @PORT@ +udp-port = @PORT@ + +# Keepalive in seconds +keepalive = 32400 + +# Dead peer detection in seconds +dpd = 440 + +# MTU discovery (DPD must be enabled) +try-mtu-discovery = false + +# The key and the certificates of the server +# The key may be a file, or any URL supported by GnuTLS (e.g., +# tpmkey:uuid=xxxxxxx-xxxx-xxxx-xxxx-xxxxxxxx;storage=user +# or pkcs11:object=my-vpn-key;object-type=private) +# +# There may be multiple certificate and key pairs and each key +# should correspond to the preceding certificate. +server-cert = @SRCDIR@/certs/server-cert.pem +server-key = @SRCDIR@/certs/server-key.pem + +# Diffie-Hellman parameters. Only needed if you require support +# for the DHE ciphersuites (by default this server supports ECDHE). +# Can be generated using: +# certtool --generate-dh-params --outfile /path/to/dh.pem +#dh-params = /path/to/dh.pem + +# If you have a certificate from a CA that provides an OCSP +# service you may provide a fresh OCSP status response within +# the TLS handshake. That will prevent the client from connecting +# independently on the OCSP server. +# You can update this response periodically using: +# ocsptool --ask --load-cert=your_cert --load-issuer=your_ca --outfile response +# Make sure that you replace the following file in an atomic way. +#ocsp-response = /path/to/ocsp.der + +# In case PKCS #11 or TPM keys are used the PINs should be available +# in files. The srk-pin-file is applicable to TPM keys only (It's the storage +# root key). +#pin-file = /path/to/pin.txt +#srk-pin-file = /path/to/srkpin.txt + +# The name of the group that if selected it would allow to use +# the assigned by default group. +default-select-group = group1 + +# Instead of specifying manually all the allowed groups, you may instruct +# ocserv to scan all available groups and include the full list. That +# option is only functional on plain authentication. +#auto-select-group = true + +select-group = group1 +select-group = group3[Do not tunnel access LAN subnet] + +# The Certificate Authority that will be used +# to verify clients if certificate authentication +# is set. +ca-cert = @SRCDIR@/certs/ca.pem + +# The object identifier that will be used to read the user ID in the client certificate. +# The object identifier should be part of the certificate's DN +# Useful OIDs are: +# CN = 2.5.4.3, UID = 0.9.2342.19200300.100.1.1 +cert-user-oid = 0.9.2342.19200300.100.1.1 + +# The object identifier that will be used to read the user group in the client +# certificate. The object identifier should be part of the certificate's DN +# Useful OIDs are: +# OU (organizational unit) = 2.5.4.11 +#cert-group-oid = 2.5.4.11 + +# A revocation list of ca-cert is set +#crl = /path/to/crl.pem + +# GnuTLS priority string +tls-priorities = "PERFORMANCE:%SERVER_PRECEDENCE:%COMPAT" + +# To enforce perfect forward secrecy (PFS) on the main channel. +#tls-priorities = "NORMAL:%SERVER_PRECEDENCE:%COMPAT:-RSA" + +# The time (in seconds) that a client is allowed to stay connected prior +# to authentication +auth-timeout = 40 + +# The time (in seconds) that a client is not allowed to reconnect after +# a failed authentication attempt. +#min-reauth-time = 2 + +# Script to call when a client connects and obtains an IP +# Parameters are passed on the environment. +# REASON, USERNAME, GROUPNAME, HOSTNAME (the hostname selected by client), +# DEVICE, IP_REAL (the real IP of the client), IP_LOCAL (the local IP +# in the P-t-P connection), IP_REMOTE (the VPN IP of the client). REASON +# may be "connect" or "disconnect". +#connect-script = /usr/bin/myscript +#disconnect-script = /usr/bin/myscript + +# UTMP +use-utmp = true + +# PID file +pid-file = ./ocserv.pid + +# The default server directory. Does not require any devices present. +#chroot-dir = /path/to/chroot + +# socket file used for IPC, will be appended with .PID +# It must be accessible within the chroot environment (if any) +socket-file = ./ocserv-socket + +# The user the worker processes will be run as. It should be +# unique (no other services run as this user). +run-as-user = @USERNAME@ +run-as-group = @GROUP@ + +# Network settings + +device = vpns + +# The default domain to be advertised +default-domain = example.com + +ipv4-network = 192.168.1.0 +ipv4-netmask = 255.255.255.0 +# Use the keywork local to advertize the local P-t-P address as DNS server +ipv4-dns = 192.168.1.1 + +# The NBNS server (if any) +#ipv4-nbns = 192.168.2.3 + +#ipv6-address = +#ipv6-mask = +#ipv6-dns = + +# Prior to leasing any IP from the pool ping it to verify that +# it is not in use by another (unrelated to this server) host. +ping-leases = false + +# Leave empty to assign the default MTU of the device +# mtu = + +no-route = 192.168.98.0/255.255.255.0 +route = default + +# +# The following options are for (experimental) AnyConnect client +# compatibility. They are only available if the server is built +# with --enable-anyconnect +# + +# Client profile xml. A sample file exists in doc/profile.xml. +# This file must be accessible from inside the worker's chroot. +# The profile is ignored by the openconnect client. +#user-profile = profile.xml + +# Unless set to false it is required for clients to present their +# certificate even if they are authenticating via a previously granted +# cookie. Legacy CISCO clients do not do that, and thus this option +# should be set for them. +#always-require-cert = false + diff --git a/tests/data/no-route-group.config b/tests/data/no-route-group.config new file mode 100644 index 00000000..60282fdb --- /dev/null +++ b/tests/data/no-route-group.config @@ -0,0 +1,192 @@ +# User authentication method. Could be set multiple times and in that case +# all should succeed. +# Options: certificate, pam. +#auth = "certificate" +auth = "plain[@SRCDIR@/data/test-group.passwd]" +#auth = "pam" + +isolate-workers = @ISOLATE_WORKERS@ + +# A banner to be displayed on clients +#banner = "Welcome" + +# Use listen-host to limit to specific IPs or to the IPs of a provided hostname. +#listen-host = [IP|HOSTNAME] + +use-dbus = no + +# Limit the number of clients. Unset or set to zero for unlimited. +#max-clients = 1024 +max-clients = 16 + +# Limit the number of client connections to one every X milliseconds +# (X is the provided value). Set to zero for no limit. +#rate-limit-ms = 100 + +# Limit the number of identical clients (i.e., users connecting multiple times) +# Unset or set to zero for unlimited. +max-same-clients = 2 + +# TCP and UDP port number +tcp-port = @PORT@ +udp-port = @PORT@ + +# Keepalive in seconds +keepalive = 32400 + +# Dead peer detection in seconds +dpd = 440 + +# MTU discovery (DPD must be enabled) +try-mtu-discovery = false + +# The key and the certificates of the server +# The key may be a file, or any URL supported by GnuTLS (e.g., +# tpmkey:uuid=xxxxxxx-xxxx-xxxx-xxxx-xxxxxxxx;storage=user +# or pkcs11:object=my-vpn-key;object-type=private) +# +# There may be multiple certificate and key pairs and each key +# should correspond to the preceding certificate. +server-cert = @SRCDIR@/certs/server-cert.pem +server-key = @SRCDIR@/certs/server-key.pem + +# Diffie-Hellman parameters. Only needed if you require support +# for the DHE ciphersuites (by default this server supports ECDHE). +# Can be generated using: +# certtool --generate-dh-params --outfile /path/to/dh.pem +#dh-params = /path/to/dh.pem + +# If you have a certificate from a CA that provides an OCSP +# service you may provide a fresh OCSP status response within +# the TLS handshake. That will prevent the client from connecting +# independently on the OCSP server. +# You can update this response periodically using: +# ocsptool --ask --load-cert=your_cert --load-issuer=your_ca --outfile response +# Make sure that you replace the following file in an atomic way. +#ocsp-response = /path/to/ocsp.der + +# In case PKCS #11 or TPM keys are used the PINs should be available +# in files. The srk-pin-file is applicable to TPM keys only (It's the storage +# root key). +#pin-file = /path/to/pin.txt +#srk-pin-file = /path/to/srkpin.txt + +# The name of the group that if selected it would allow to use +# the assigned by default group. +default-select-group = group1 + +# Instead of specifying manually all the allowed groups, you may instruct +# ocserv to scan all available groups and include the full list. That +# option is only functional on plain authentication. +#auto-select-group = true + +select-group = group1 +select-group = group3 + +# The Certificate Authority that will be used +# to verify clients if certificate authentication +# is set. +ca-cert = @SRCDIR@/certs/ca.pem + +# The object identifier that will be used to read the user ID in the client certificate. +# The object identifier should be part of the certificate's DN +# Useful OIDs are: +# CN = 2.5.4.3, UID = 0.9.2342.19200300.100.1.1 +cert-user-oid = 0.9.2342.19200300.100.1.1 + +# The object identifier that will be used to read the user group in the client +# certificate. The object identifier should be part of the certificate's DN +# Useful OIDs are: +# OU (organizational unit) = 2.5.4.11 +#cert-group-oid = 2.5.4.11 + +# A revocation list of ca-cert is set +#crl = /path/to/crl.pem + +# GnuTLS priority string +tls-priorities = "PERFORMANCE:%SERVER_PRECEDENCE:%COMPAT" + +# To enforce perfect forward secrecy (PFS) on the main channel. +#tls-priorities = "NORMAL:%SERVER_PRECEDENCE:%COMPAT:-RSA" + +# The time (in seconds) that a client is allowed to stay connected prior +# to authentication +auth-timeout = 40 + +# The time (in seconds) that a client is not allowed to reconnect after +# a failed authentication attempt. +#min-reauth-time = 2 + +# Script to call when a client connects and obtains an IP +# Parameters are passed on the environment. +# REASON, USERNAME, GROUPNAME, HOSTNAME (the hostname selected by client), +# DEVICE, IP_REAL (the real IP of the client), IP_LOCAL (the local IP +# in the P-t-P connection), IP_REMOTE (the VPN IP of the client). REASON +# may be "connect" or "disconnect". +#connect-script = /usr/bin/myscript +#disconnect-script = /usr/bin/myscript + +# UTMP +use-utmp = true + +# PID file +pid-file = ./ocserv.pid + +# The default server directory. Does not require any devices present. +#chroot-dir = /path/to/chroot + +# socket file used for IPC, will be appended with .PID +# It must be accessible within the chroot environment (if any) +socket-file = ./ocserv-socket + +# The user the worker processes will be run as. It should be +# unique (no other services run as this user). +run-as-user = @USERNAME@ +run-as-group = @GROUP@ + +# Network settings + +device = vpns + +# The default domain to be advertised +default-domain = example.com + +ipv4-network = 192.168.1.0 +ipv4-netmask = 255.255.255.0 +# Use the keywork local to advertize the local P-t-P address as DNS server +ipv4-dns = 192.168.1.1 + +# The NBNS server (if any) +#ipv4-nbns = 192.168.2.3 + +#ipv6-address = +#ipv6-mask = +#ipv6-dns = + +# Prior to leasing any IP from the pool ping it to verify that +# it is not in use by another (unrelated to this server) host. +ping-leases = false + +# Leave empty to assign the default MTU of the device +# mtu = + +route = default + +# +# The following options are for (experimental) AnyConnect client +# compatibility. They are only available if the server is built +# with --enable-anyconnect +# + +# Client profile xml. A sample file exists in doc/profile.xml. +# This file must be accessible from inside the worker's chroot. +# The profile is ignored by the openconnect client. +#user-profile = profile.xml + +# Unless set to false it is required for clients to present their +# certificate even if they are authenticating via a previously granted +# cookie. Legacy CISCO clients do not do that, and thus this option +# should be set for them. +#always-require-cert = false + +config-per-group = @SRCDIR@/data/group-config/ diff --git a/tests/no-route-default b/tests/no-route-default new file mode 100755 index 00000000..de713425 --- /dev/null +++ b/tests/no-route-default @@ -0,0 +1,86 @@ +#!/bin/bash +# +# Copyright (C) 2020 Nikos Mavrogiannopoulos +# +# This file is part of ocserv. +# +# ocserv is free software; you can redistribute it and/or modify it +# under the terms of the GNU General Public License as published by the +# Free Software Foundation; either version 2 of the License, or (at +# your option) any later version. +# +# ocserv is distributed in the hope that it will be useful, but +# WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +# General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with GnuTLS; if not, write to the Free Software Foundation, +# Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. + +SERV="${SERV:-../src/ocserv}" +srcdir=${srcdir:-.} +PORT=5743 +TMPFILE=$(mktemp) + +. `dirname $0`/common.sh + +echo "Testing whether exclude routes are being sent (with route=default)... " + +function finish { + set +e + echo " * Cleaning up..." + test -n "${CONFIG}" && rm -f ${CONFIG} >/dev/null 2>&1 + rm -f $TMPFILE 2>&1 + cleanup +} +trap finish EXIT + +update_config no-route-default.config +launch_simple_server -d 1 -f -c ${CONFIG} +PID=$! +wait_server $PID + +echo -n "Connecting to get routes... " +timeout 15s $OPENCONNECT -v localhost:$PORT --sslkey ${srcdir}/certs/user-key.pem -c ${srcdir}/certs/user-cert-testuser.pem --servercert=d66b507ae074d03b02eafca40d35f87dd81049d3 -s /bin/true ${TMPFILE} 2>&1 + +echo ok + +echo -n "Checking routes... " + +cat ${TMPFILE}|grep X-CSTP-Split-Exclude|grep 192.168.98.0 >/dev/null +if test $? != 0;then + cat ${TMPFILE}|grep X-CSTP-Split + echo "Temporary file contents are not correct" + exit 1 +fi + +echo ok + +kill $PID +sleep 5 + +echo "Testing whether exclude routes are being sent (with no route)... " +sed 's/route = default//g' -i ${CONFIG} +launch_simple_server -d 1 -f -c ${CONFIG} +PID=$! +wait_server $PID + +echo -n "Connecting to get routes... " +timeout 15s $OPENCONNECT -v localhost:$PORT --sslkey ${srcdir}/certs/user-key.pem -c ${srcdir}/certs/user-cert-testuser.pem --servercert=d66b507ae074d03b02eafca40d35f87dd81049d3 -s /bin/true ${TMPFILE} 2>&1 + +echo ok + +echo -n "Checking routes... " + +cat ${TMPFILE}|grep X-CSTP-Split-Exclude|grep 192.168.98.0 >/dev/null +if test $? != 0;then + cat ${TMPFILE}|grep X-CSTP-Split + echo "Temporary file contents are not correct" + exit 1 +fi + +echo ok + + +exit 0 diff --git a/tests/no-route-group b/tests/no-route-group new file mode 100755 index 00000000..2775aefc --- /dev/null +++ b/tests/no-route-group @@ -0,0 +1,86 @@ +#!/bin/bash +# +# Copyright (C) 2020 Nikos Mavrogiannopoulos +# +# This file is part of ocserv. +# +# ocserv is free software; you can redistribute it and/or modify it +# under the terms of the GNU General Public License as published by the +# Free Software Foundation; either version 2 of the License, or (at +# your option) any later version. +# +# ocserv is distributed in the hope that it will be useful, but +# WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +# General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with GnuTLS; if not, write to the Free Software Foundation, +# Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. + +SERV="${SERV:-../src/ocserv}" +srcdir=${srcdir:-.} +PORT=5744 +TMPFILE=$(mktemp) + +. `dirname $0`/common.sh + +echo "Testing whether group exclude routes are being sent (with route=default)... " + +function finish { + set +e + echo " * Cleaning up..." + test -n "${CONFIG}" && rm -f ${CONFIG} >/dev/null 2>&1 + rm -f $TMPFILE 2>&1 + cleanup +} +trap finish EXIT + +update_config no-route-group.config +launch_simple_server -d 1 -f -c ${CONFIG} +PID=$! +wait_server $PID + +echo -n "Connecting to get routes... " +echo "test" | timeout 15s $OPENCONNECT -v localhost:$PORT --authgroup group1 -u test --passwd-on-stdin --servercert=d66b507ae074d03b02eafca40d35f87dd81049d3 -s /bin/true >${TMPFILE} 2>&1 + +echo ok + +echo -n "Checking routes... " + +cat ${TMPFILE}|grep X-CSTP-Split-Exclude|grep 192.168.98.0 >/dev/null +if test $? != 0;then + cat ${TMPFILE}|grep X-CSTP-Split + echo "Temporary file contents are not correct" + exit 1 +fi + +echo ok + +kill $PID +sleep 5 + +echo "Testing whether group exclude routes are being sent (with no route)... " +sed 's/route = default//g' -i ${CONFIG} +launch_simple_server -d 1 -f -c ${CONFIG} +PID=$! +wait_server $PID + +echo -n "Connecting to get routes... " +echo test | timeout 15s $OPENCONNECT -v localhost:$PORT --authgroup group1 --passwd-on-stdin -u test --servercert=d66b507ae074d03b02eafca40d35f87dd81049d3 -s /bin/true >${TMPFILE} 2>&1 + +echo ok + +echo -n "Checking routes... " + +cat ${TMPFILE}|grep X-CSTP-Split-Exclude|grep 192.168.98.0 >/dev/null +if test $? != 0;then + cat ${TMPFILE}|grep X-CSTP-Split + echo "Temporary file contents are not correct" + exit 1 +fi + +echo ok + + +exit 0