[Openvpn-devel] script-options.rst: Update ifconfig_* variables

Message ID 20240321161623.2794161-1-frank@lichtenheld.com
State Accepted
Headers show
Series [Openvpn-devel] script-options.rst: Update ifconfig_* variables | expand

Commit Message

Frank Lichtenheld March 21, 2024, 4:16 p.m. UTC
- Remove obsolete ifconfig_broadcast. Since this was
  removed in 2.5.0, do not add a removal note but just
  completely remove it.
- Add missing documentation of IPv6 variants for
  ifconfig_pool_* variables.

Github: #527
Change-Id: Ia8c8de6799f0291fc900628fbd06c8a414e741ca
Signed-off-by: Frank Lichtenheld <frank@lichtenheld.com>
---
 doc/man-sections/script-options.rst | 38 ++++++++++++++++++++---------
 1 file changed, 27 insertions(+), 11 deletions(-)

Comments

Gert Doering March 26, 2024, 1:23 p.m. UTC | #1
Acked-by: Gert Doering <gert@greenie.muc.de>

We're so bad at times at updating documentation...  verified that the
newly documented options exist and do what it says.  Confusing code...

Your patch has been applied to the master and release/2.6 branch (doc).

commit a94226cdc8ed037a6763675aa47e6c821983f174 (master)
commit ea0d9c70a44e3d871136f68bddb0befc299dd692 (release/2.6)
Author: Frank Lichtenheld
Date:   Thu Mar 21 17:16:23 2024 +0100

     script-options.rst: Update ifconfig_* variables

     Signed-off-by: Frank Lichtenheld <frank@lichtenheld.com>
     Acked-by: Gert Doering <gert@greenie.muc.de>
     Message-Id: <20240321161623.2794161-1-frank@lichtenheld.com>
     URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg28438.html
     Signed-off-by: Gert Doering <gert@greenie.muc.de>


--
kind regards,

Gert Doering

Patch

diff --git a/doc/man-sections/script-options.rst b/doc/man-sections/script-options.rst
index e05100a2..0d1f9aec 100644
--- a/doc/man-sections/script-options.rst
+++ b/doc/man-sections/script-options.rst
@@ -682,13 +682,6 @@  instances.
     recorded to this environmental variable sequence prior to ``--up``
     script execution.
 
-:code:`ifconfig_broadcast`
-    The broadcast address for the virtual ethernet segment which is derived
-    from the ``--ifconfig`` option when ``--dev tap`` is used. Set prior to
-    OpenVPN calling the :code:`ifconfig` or :code:`netsh` (windows version
-    of ifconfig) commands which normally occurs prior to ``--up`` script
-    execution.
-
 :code:`ifconfig_ipv6_local`
     The local VPN endpoint IPv6 address specified in the
     ``--ifconfig-ipv6`` option (first parameter). Set prior to OpenVPN
@@ -731,30 +724,53 @@  instances.
     occurs prior to ``--up`` script execution.
 
 :code:`ifconfig_pool_local_ip`
-    The local virtual IP address for the TUN/TAP tunnel taken from an
+    The local virtual IPv4 address for the TUN/TAP tunnel taken from an
     ``--ifconfig-push`` directive if specified, or otherwise from the
     ifconfig pool (controlled by the ``--ifconfig-pool`` config file
     directive). Only set for ``--dev tun`` tunnels. This option is set on
     the server prior to execution of the ``--client-connect`` and
     ``--client-disconnect`` scripts.
 
+:code:`ifconfig_pool_local_ip6`
+    The local virtual IPv6 address for the TUN/TAP tunnel taken from an
+    ``--ifconfig-ipv6-push`` directive if specified, or otherwise from the
+    ifconfig pool (controlled by the ``--ifconfig-ipv6-pool`` config file
+    directive). Only set for ``--dev tun`` tunnels. This option is set on
+    the server prior to execution of the ``--client-connect`` and
+    ``--client-disconnect`` scripts.
+
 :code:`ifconfig_pool_netmask`
-    The virtual IP netmask for the TUN/TAP tunnel taken from an
+    The virtual IPv4 netmask for the TUN/TAP tunnel taken from an
     ``--ifconfig-push`` directive if specified, or otherwise from the
     ifconfig pool (controlled by the ``--ifconfig-pool`` config file
     directive). Only set for ``--dev tap`` tunnels. This option is set on
     the server prior to execution of the ``--client-connect`` and
     ``--client-disconnect`` scripts.
 
+:code:`ifconfig_pool_ip6_netbits`
+    The virtual IPv6 prefix length for the TUN/TAP tunnel taken from an
+    ``--ifconfig-ipv6-push`` directive if specified, or otherwise from the
+    ifconfig pool (controlled by the ``--ifconfig-ipv6-pool`` config file
+    directive). Only set for ``--dev tap`` tunnels. This option is set on
+    the server prior to execution of the ``--client-connect`` and
+    ``--client-disconnect`` scripts.
+
 :code:`ifconfig_pool_remote_ip`
-    The remote virtual IP address for the TUN/TAP tunnel taken from an
+    The remote virtual IPv4 address for the TUN/TAP tunnel taken from an
     ``--ifconfig-push`` directive if specified, or otherwise from the
     ifconfig pool (controlled by the ``--ifconfig-pool`` config file
     directive). This option is set on the server prior to execution of the
     ``--client-connect`` and ``--client-disconnect`` scripts.
 
+:code:`ifconfig_pool_remote_ip6`
+    The remote virtual IPv6 address for the TUN/TAP tunnel taken from an
+    ``--ifconfig-ipv6-push`` directive if specified, or otherwise from the
+    ifconfig pool (controlled by the ``--ifconfig-ipv6-pool`` config file
+    directive). This option is set on the server prior to execution of the
+    ``--client-connect`` and ``--client-disconnect`` scripts.
+
 :code:`link_mtu`
-    No longer passed to scripts since OpenVPN 2.6.0.  Used to be the
+    *REMOVED* No longer passed to scripts since OpenVPN 2.6.0.  Used to be the
     maximum packet size (not including the IP header) of tunnel data in
     UDP tunnel transport mode.