From: Gert Doering Date: Mon, 5 May 2025 14:22:16 +0000 (+0200) Subject: Add information-gathering about DNS resolvers configured to t_client.sh(.in) X-Git-Tag: v2.7_alpha1~23 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b4e31e57bc69819e6ea61ea1bc08ad199b2be272;p=thirdparty%2Fopenvpn.git Add information-gathering about DNS resolvers configured to t_client.sh(.in) With the patchsets to add DNS configuration on Unix+MacOS systems, and the addition of test stanzas to excercise and verify the OS specific "dns-updown" script, it becomes important to trace test failures ("did it not ping because the DNS was not installed, or did something else fail?") and also verify that DNS config is properly restored at the end of each test. Linux is probed with "resolvectl status", if available, and "cat resolv.conf" if not. MacOS uses scutil --dns. All other platforms use "cat resolv.conf" for now (because even if "a tool to maintain DNS config" is available, in the end resolv.conf is always where the final config lands). Include a bit of restructuring to handle linux iproute2 testing in the "Linux" branch, and make the control flow more amenable to having a second case / esac block. Change-Id: I9cae7314203424e4a604073c5445559260172477 Signed-off-by: Gert Doering Acked-by: Frank Lichtenheld Message-Id: <20250505142224.24935-1-gert@greenie.muc.de> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg31568.html Signed-off-by: Gert Doering --- diff --git a/tests/t_client.sh.in b/tests/t_client.sh.in index 1ccadebb3..7a271b6d9 100755 --- a/tests/t_client.sh.in +++ b/tests/t_client.sh.in @@ -166,53 +166,71 @@ fail() # this is higly system dependent... get_ifconfig_route() { - # linux / iproute2? (-> if configure got a path) - if [ -n "@IPROUTE@" ] - then - echo "-- linux iproute2 --" - @IPROUTE@ addr show | grep -v valid_lft - @IPROUTE@ route show - @IPROUTE@ -o -6 route show | grep -v ' cache' | sed -E -e 's/ expires [0-9]*sec//' -e 's/ (mtu|hoplimit|cwnd|ssthresh) [0-9]+//g' -e 's/ (rtt|rttvar) [0-9]+ms//g' - return - fi - - # try uname - case `uname -s` in + UNAME=`uname -s` + case $UNAME in Linux) - echo "-- linux / ifconfig --" - LANG=C @IFCONFIG@ -a |egrep "( addr:|encap:)" - LANG=C @NETSTAT@ -rn -4 -6 - return - ;; + # linux / iproute2? (-> if configure got a path) + if [ -n "@IPROUTE@" ] + then + echo "-- linux iproute2 --" + @IPROUTE@ addr show | grep -v valid_lft + @IPROUTE@ route show + @IPROUTE@ -o -6 route show | grep -v ' cache' | sed -E -e 's/ expires [0-9]*sec//' -e 's/ (mtu|hoplimit|cwnd|ssthresh) [0-9]+//g' -e 's/ (rtt|rttvar) [0-9]+ms//g' + else + echo "-- linux / ifconfig --" + LANG=C @IFCONFIG@ -a |egrep "( addr:|encap:)" + LANG=C @NETSTAT@ -rn -4 -6 + fi + ;; FreeBSD|NetBSD|Darwin) echo "-- FreeBSD/NetBSD/Darwin [MacOS X] --" @IFCONFIG@ -a | egrep "(flags=|inet)" @NETSTAT@ -rn | awk '$3 !~ /^UHL/ { print $1,$2,$3,$NF }' - return ;; OpenBSD) echo "-- OpenBSD --" @IFCONFIG@ -a | egrep "(flags=|inet)" | \ sed -e 's/pltime [0-9]*//' -e 's/vltime [0-9]*//' @NETSTAT@ -rn | awk '$3 !~ /^UHL/ { print $1,$2,$3,$NF }' - return ;; SunOS) echo "-- Solaris --" @IFCONFIG@ -a | egrep "(flags=|inet)" @NETSTAT@ -rn | awk '$3 !~ /^UHL/ { print $1,$2,$3,$6 }' - return ;; AIX) echo "-- AIX --" @IFCONFIG@ -a | egrep "(flags=|inet)" @NETSTAT@ -rn | awk '$3 !~ /^UHL/ { print $1,$2,$3,$6 }' - return ;; + *) + echo "get_ifconfig_route(): no idea how to get info on your OS (`uname -s`). FAIL." >&2 + exit 20 + ;; esac - echo "get_ifconfig_route(): no idea how to get info on your OS. FAIL." >&2 - exit 20 + # another round of per-platform information gathering, for DNS info + # for most of the platforms "cat /etc/resolv.conf" is good enough + # except Linux and MacOS + case $UNAME in + Linux) + if [ -x /usr/bin/resolvectl ] ; then + echo "-- linux resolvectl --" + resolvectl status + else + echo "-- linux resolv.conf --" + cat /etc/resolv.conf + fi + ;; + Darwin) + echo "-- MacOS scutil --dns" + scutil --dns + ;; + *) + echo "-- resolv.conf --" + cat /etc/resolv.conf + ;; + esac } # ----------------------------------------------------------