diff mbox series

[Openvpn-devel,v2] Document comp-lzo no and compress being incompatible

Message ID 20200811110248.3396-1-arne@rfc2549.org
State Accepted
Headers show
Series [Openvpn-devel,v2] Document comp-lzo no and compress being incompatible | expand

Commit Message

Arne Schwabe Aug. 11, 2020, 1:02 a.m. UTC 
Most of the new compress but not v2 version do use swap operation. For
'compress lzo' the swap option is not used for backwards compatibility.
For lz4 the swap option is also not a problem since there is no version
without swap. Unfortunately, compress introduced a second stub format
with swap, contrary to the one in 'comp-lzo no' that does not use swap.

Document this weirdness to let not others fall into this trap.

Patch V2: redo patch for rst man pages
---
 doc/man-sections/protocol-options.rst | 3 +++
 1 file changed, 3 insertions(+)

Comments

Gert Doering Aug. 11, 2020, 1:08 a.m. UTC | #1 
Your patch has been applied to the master branch.

commit e02616d82105aeb77f8f811c94fd595ca1a96eae
Author: Arne Schwabe
Date:   Tue Aug 11 13:02:48 2020 +0200

     Document comp-lzo no and compress being incompatible

     Acked-by: Gert Doering <gert@greenie.muc.de>
     Message-Id: <20200811110248.3396-1-arne@rfc2549.org>
     URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg20708.html
     Signed-off-by: Gert Doering <gert@greenie.muc.de>


--
kind regards,

Gert Doering
diff mbox series

Patch

diff --git a/doc/man-sections/protocol-options.rst b/doc/man-sections/protocol-options.rst
index 240d0edf..82e4b2bd 100644
--- a/doc/man-sections/protocol-options.rst
+++ b/doc/man-sections/protocol-options.rst
@@ -103,6 +103,9 @@  configured in a compatible way between both the local and remote side.
   ``lzo`` and ``lz4`` compression support via *IV_* variables to the
   server.
 
+  Note: the :code:`stub` (or empty) option is NOT compatible with the older
+  option ``--comp-lzo no``.
+
   ***Security Considerations***
 
   Compression and encryption is a tricky combination. If an attacker knows