Skip to contentSkip to conversation

doc: The CRL processing is not a deprecated feature

PATCH In review David Sommerseth 3 participants 3 messages Jun 28, 2017 – Jun 29, 2017

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 doc: The CRL processing is not a deprecated feature

The note related to the CRL processing was somehow put into the deprecated section. This is quite confusing.

Since this is a fairly important change, and there have been a noticable amount of supports questions related to OpenVPN not starting due to CRL errors, I put this into the "New features" section labelled as an improvement. Otherwise I fear this would drown in the list of "User-visible Changes" later on.

---
 Changes.rst | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)
Changes.rst +7 -6
index 9db0a451..0b2b04dd 100644
@@ -44,6 +44,13 @@ ECDH key exchange
     The TLS control channel now supports for elliptic curve diffie-hellmann
     key exchange (ECDH).
 
+Improved Certificate Revocation List (CRL) processing
+    CRLs are now handled by the crypto library (OpenSSL or mbed TLS), instead
+    of inside OpenVPN itself.  The crypto library implementations are more
+    strict than the OpenVPN implementation was.  This might reject peer
+    certificates that would previously be accepted.  If this occurs, OpenVPN
+    will log the crypto library's error description.
+
 Dualstack round-robin DNS client connect
     Instead of only using the first address of each ``--remote`` OpenVPN
     will now try all addresses (IPv6 and IPv4) of a ``--remote`` entry.
@@ -160,12 +167,6 @@ Deprecated features
   will then use ``--key-method 2`` by default.  Note that this requires changing
   the option in both the client and server side configs.
 
-- CRLs are now handled by the crypto library (OpenSSL or mbed TLS), instead of
-  inside OpenVPN itself.  The crypto library implementations are more strict
-  than the OpenVPN implementation was.  This might reject peer certificates
-  that would previously be accepted.  If this occurs, OpenVPN will log the
-  crypto library's error description.
-
 - ``--tls-remote`` is removed in 2.4, as indicated in the 2.3 man-pages.  Similar
   functionality is provided via ``--verify-x509-name``, which does the same job in
   a better way.
Signature · 1 line
2.11.0
#1 lore raw
Quoted 37 lines · David Sommerseth

The note related to the CRL processing was somehow put into the deprecated section. This is quite confusing.

Since this is a fairly important change, and there have been a noticable amount of supports questions related to OpenVPN not starting due to CRL errors, I put this into the "New features" section labelled as an improvement. Otherwise I fear this would drown in the list of "User-visible Changes" later on.

---
 Changes.rst | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)
Changes.rst +7 -0
index 9db0a451..0b2b04dd 100644
@@ -44,6 +44,13 @@ ECDH key exchange
     The TLS control channel now supports for elliptic curve diffie-hellmann
     key exchange (ECDH).
 
+Improved Certificate Revocation List (CRL) processing
+    CRLs are now handled by the crypto library (OpenSSL or mbed TLS), instead
+    of inside OpenVPN itself.  The crypto library implementations are more
+    strict than the OpenVPN implementation was.  This might reject peer
+    certificates that would previously be accepted.  If this occurs, OpenVPN
+    will log the crypto library's error description.
+
 Dualstack round-robin DNS client connect
     Instead of only using the first address of each ``--remote`` OpenVPN
     will now try all addresses (IPv6 and IPv4) of a ``--remote`` entry.
@@ -160,12 +167,6 @@ Deprecated features
   will then use ``--key-method 2`` by default.  Note that this requires changing
   the option in both the client and server side configs.

Changes.rst Deprecated features · line 172

Quoted 10 lines
Changes.rst +0 -6
 
-- CRLs are now handled by the crypto library (OpenSSL or mbed TLS), instead of
-  inside OpenVPN itself.  The crypto library implementations are more strict
-  than the OpenVPN implementation was.  This might reject peer certificates
-  that would previously be accepted.  If this occurs, OpenVPN will log the
-  crypto library's error description.
-
 - ``--tls-remote`` is removed in 2.4, as indicated in the 2.3 man-pages.  Similar
   functionality is provided via ``--verify-x509-name``, which does the same job in
   a better way.

ACK

-Steffan

#2 lore raw
In reply to David Sommerseth

Your patch has been applied to the master and release/2.4 branch.

commit f9ebfe1b5a011e55fb87a5026b1897c8ffb8f75e (master) commit 67e3f0280937757ce716211829c82c3795d21ad1 (release/2.4)

Author: David Sommerseth
Date:   Wed Jun 28 21:15:38 2017 +0200
     doc: The CRL processing is not a deprecated feature

     Signed-off-by: David Sommerseth <[email protected]...>
     Acked-by: Steffan Karger <[email protected]...>
     Message-Id: <[email protected]...>
     URL: https://www.mail-archive.com/[email protected]/msg14985.html
     Signed-off-by: Gert Doering <[email protected]...>

--
kind regards,

Gert Doering