code.ossystems Code Review - openembedded-core.git/blob

   1 From 46c719ec4f79d6830c55ab7f5a03d826eabd0bd5 Mon Sep 17 00:00:00 2001
   2 From: Christian Heimes <christian@python.org>
   3 Date: Thu, 7 Sep 2017 20:23:52 -0700
   4 Subject: [PATCH 2/4] bpo-29136: Add TLS 1.3 cipher suites and OP_NO_TLSv1_3
   5  (GH-1363) (#3444)
   6
   7 * bpo-29136: Add TLS 1.3 support
   8
   9 TLS 1.3 introduces a new, distinct set of cipher suites. The TLS 1.3
  10 cipher suites don't overlap with cipher suites from TLS 1.2 and earlier.
  11 Since Python sets its own set of permitted ciphers, TLS 1.3 handshake
  12 will fail as soon as OpenSSL 1.1.1 is released. Let's enable the common
  13 AES-GCM and ChaCha20 suites.
  14
  15 Additionally the flag OP_NO_TLSv1_3 is added. It defaults to 0 (no op) with
  16 OpenSSL prior to 1.1.1. This allows applications to opt-out from TLS 1.3
  17 now.
  18
  19 Signed-off-by: Christian Heimes <christian@python.org>.
  20 (cherry picked from commit cb5b68abdeb1b1d56c581d5b4d647018703d61e3)
  21
  22 Upstream-Status: Backport
  23 [https://github.com/python/cpython/commit/cb5b68abdeb1b1d56c581d5b4d647018703d61e3]
  24
  25 Signed-off-by: Anuj Mittal <anuj.mittal@intel.com>
  26 ---
  27  Doc/library/ssl.rst                           | 21 ++++++++++++++
  28  Lib/ssl.py                                    | 14 +++++++++
  29  Lib/test/test_ssl.py                          | 29 ++++++++++++++++++-
  30  .../2017-09-04-16-39-49.bpo-29136.vSn1oR.rst  |  1 +
  31  Modules/_ssl.c                                | 13 +++++++++
  32  5 files changed, 77 insertions(+), 1 deletion(-)
  33  create mode 100644 Misc/NEWS.d/next/Library/2017-09-04-16-39-49.bpo-29136.vSn1oR.rst
  34
  35 diff --git a/Doc/library/ssl.rst b/Doc/library/ssl.rst
  36 index 14f2d68217..29c5e94cf6 100644
  37 --- a/Doc/library/ssl.rst
  38 +++ b/Doc/library/ssl.rst
  39 @@ -285,6 +285,11 @@ purposes.
  40
  41       3DES was dropped from the default cipher string.
  42
  43 +   .. versionchanged:: 3.7
  44 +
  45 +     TLS 1.3 cipher suites TLS_AES_128_GCM_SHA256, TLS_AES_256_GCM_SHA384,
  46 +     and TLS_CHACHA20_POLY1305_SHA256 were added to the default cipher string.
  47 +
  48
  49  Random generation
  50  ^^^^^^^^^^^^^^^^^
  51 @@ -719,6 +724,16 @@ Constants
  52
  53     .. versionadded:: 3.4
  54
  55 +.. data:: OP_NO_TLSv1_3
  56 +
  57 +   Prevents a TLSv1.3 connection. This option is only applicable in conjunction
  58 +   with :const:`PROTOCOL_TLS`. It prevents the peers from choosing TLSv1.3 as
  59 +   the protocol version. TLS 1.3 is available with OpenSSL 1.1.1 or later.
  60 +   When Python has been compiled against an older version of OpenSSL, the
  61 +   flag defaults to *0*.
  62 +
  63 +   .. versionadded:: 3.7
  64 +
  65  .. data:: OP_CIPHER_SERVER_PREFERENCE
  66
  67     Use the server's cipher ordering preference, rather than the client's.
  68 @@ -783,6 +798,12 @@ Constants
  69
  70     .. versionadded:: 3.3
  71
  72 +.. data:: HAS_TLSv1_3
  73 +
  74 +   Whether the OpenSSL library has built-in support for the TLS 1.3 protocol.
  75 +
  76 +   .. versionadded:: 3.7
  77 +
  78  .. data:: CHANNEL_BINDING_TYPES
  79
  80     List of supported TLS channel binding types.  Strings in this list
  81 diff --git a/Lib/ssl.py b/Lib/ssl.py
  82 index 4d302a78fa..ac2c0cbaf3 100644
  83 --- a/Lib/ssl.py
  84 +++ b/Lib/ssl.py
  85 @@ -122,6 +122,14 @@ _import_symbols('OP_')
  86  _import_symbols('ALERT_DESCRIPTION_')
  87  _import_symbols('SSL_ERROR_')
  88  _import_symbols('VERIFY_')
  89 +from _ssl import HAS_SNI, HAS_ECDH, HAS_NPN, HAS_ALPN, HAS_TLSv1_3
  90 +from _ssl import _OPENSSL_API_VERSION
  91 +
  92 +
  93 +_IntEnum._convert(
  94 +    '_SSLMethod', __name__,
  95 +    lambda name: name.startswith('PROTOCOL_') and name != 'PROTOCOL_SSLv23',
  96 +    source=_ssl)
  97
  98  from _ssl import HAS_SNI, HAS_ECDH, HAS_NPN, HAS_ALPN
  99
 100 @@ -162,6 +170,7 @@ else:
 101  # (OpenSSL's default setting is 'DEFAULT:!aNULL:!eNULL')
 102  # Enable a better set of ciphers by default
 103  # This list has been explicitly chosen to:
 104 +#   * TLS 1.3 ChaCha20 and AES-GCM cipher suites
 105  #   * Prefer cipher suites that offer perfect forward secrecy (DHE/ECDHE)
 106  #   * Prefer ECDHE over DHE for better performance
 107  #   * Prefer AEAD over CBC for better performance and security
 108 @@ -173,6 +182,8 @@ else:
 109  #   * Disable NULL authentication, NULL encryption, 3DES and MD5 MACs
 110  #     for security reasons
 111  _DEFAULT_CIPHERS = (
 112 +    'TLS13-AES-256-GCM-SHA384:TLS13-CHACHA20-POLY1305-SHA256:'
 113 +    'TLS13-AES-128-GCM-SHA256:'
 114      'ECDH+AESGCM:ECDH+CHACHA20:DH+AESGCM:DH+CHACHA20:ECDH+AES256:DH+AES256:'
 115      'ECDH+AES128:DH+AES:ECDH+HIGH:DH+HIGH:RSA+AESGCM:RSA+AES:RSA+HIGH:'
 116      '!aNULL:!eNULL:!MD5:!3DES'
 117 @@ -180,6 +191,7 @@ _DEFAULT_CIPHERS = (
 118
 119  # Restricted and more secure ciphers for the server side
 120  # This list has been explicitly chosen to:
 121 +#   * TLS 1.3 ChaCha20 and AES-GCM cipher suites
 122  #   * Prefer cipher suites that offer perfect forward secrecy (DHE/ECDHE)
 123  #   * Prefer ECDHE over DHE for better performance
 124  #   * Prefer AEAD over CBC for better performance and security
 125 @@ -190,6 +202,8 @@ _DEFAULT_CIPHERS = (
 126  #   * Disable NULL authentication, NULL encryption, MD5 MACs, DSS, RC4, and
 127  #     3DES for security reasons
 128  _RESTRICTED_SERVER_CIPHERS = (
 129 +    'TLS13-AES-256-GCM-SHA384:TLS13-CHACHA20-POLY1305-SHA256:'
 130 +    'TLS13-AES-128-GCM-SHA256:'
 131      'ECDH+AESGCM:ECDH+CHACHA20:DH+AESGCM:DH+CHACHA20:ECDH+AES256:DH+AES256:'
 132      'ECDH+AES128:DH+AES:ECDH+HIGH:DH+HIGH:RSA+AESGCM:RSA+AES:RSA+HIGH:'
 133      '!aNULL:!eNULL:!MD5:!DSS:!RC4:!3DES'
 134 diff --git a/Lib/test/test_ssl.py b/Lib/test/test_ssl.py
 135 index f91af7bd05..1acc12ec2d 100644
 136 --- a/Lib/test/test_ssl.py
 137 +++ b/Lib/test/test_ssl.py
 138 @@ -150,6 +150,13 @@ class BasicSocketTests(unittest.TestCase):
 139              ssl.OP_NO_COMPRESSION
 140          self.assertIn(ssl.HAS_SNI, {True, False})
 141          self.assertIn(ssl.HAS_ECDH, {True, False})
 142 +        ssl.OP_NO_SSLv2
 143 +        ssl.OP_NO_SSLv3
 144 +        ssl.OP_NO_TLSv1
 145 +        ssl.OP_NO_TLSv1_3
 146 +    if ssl.OPENSSL_VERSION_INFO >= (1, 0, 1):
 147 +            ssl.OP_NO_TLSv1_1
 148 +            ssl.OP_NO_TLSv1_2
 149
 150      def test_str_for_enums(self):
 151          # Make sure that the PROTOCOL_* constants have enum-like string
 152 @@ -3028,12 +3035,33 @@ else:
 153                      self.assertEqual(s.version(), 'TLSv1')
 154                  self.assertIs(s.version(), None)
 155
 156 +        @unittest.skipUnless(ssl.HAS_TLSv1_3,
 157 +                             "test requires TLSv1.3 enabled OpenSSL")
 158 +        def test_tls1_3(self):
 159 +            context = ssl.SSLContext(ssl.PROTOCOL_TLS)
 160 +            context.load_cert_chain(CERTFILE)
 161 +            # disable all but TLS 1.3
 162 +            context.options |= (
 163 +                ssl.OP_NO_TLSv1 | ssl.OP_NO_TLSv1_1 | ssl.OP_NO_TLSv1_2
 164 +            )
 165 +            with ThreadedEchoServer(context=context) as server:
 166 +                with context.wrap_socket(socket.socket()) as s:
 167 +                    s.connect((HOST, server.port))
 168 +                    self.assertIn(s.cipher()[0], [
 169 +                        'TLS13-AES-256-GCM-SHA384',
 170 +                        'TLS13-CHACHA20-POLY1305-SHA256',
 171 +                        'TLS13-AES-128-GCM-SHA256',
 172 +                    ])
 173 +
 174          @unittest.skipUnless(ssl.HAS_ECDH, "test requires ECDH-enabled OpenSSL")
 175          def test_default_ecdh_curve(self):
 176              # Issue #21015: elliptic curve-based Diffie Hellman key exchange
 177              # should be enabled by default on SSL contexts.
 178              context = ssl.SSLContext(ssl.PROTOCOL_SSLv23)
 179              context.load_cert_chain(CERTFILE)
 180 +            # TLSv1.3 defaults to PFS key agreement and no longer has KEA in
 181 +            # cipher name.
 182 +            context.options |= ssl.OP_NO_TLSv1_3
 183              # Prior to OpenSSL 1.0.0, ECDH ciphers have to be enabled
 184              # explicitly using the 'ECCdraft' cipher alias.  Otherwise,
 185              # our default cipher list should prefer ECDH-based ciphers
 186 @@ -3394,7 +3422,6 @@ else:
 187                          s.sendfile(file)
 188                          self.assertEqual(s.recv(1024), TEST_DATA)
 189
 190 -
 191  def test_main(verbose=False):
 192      if support.verbose:
 193          import warnings
 194 diff --git a/Misc/NEWS.d/next/Library/2017-09-04-16-39-49.bpo-29136.vSn1oR.rst b/Misc/NEWS.d/next/Library/2017-09-04-16-39-49.bpo-29136.vSn1oR.rst
 195 new file mode 100644
 196 index 0000000000..e76997ef83
 197 --- /dev/null
 198 +++ b/Misc/NEWS.d/next/Library/2017-09-04-16-39-49.bpo-29136.vSn1oR.rst
 199 @@ -0,0 +1 @@
 200 +Add TLS 1.3 cipher suites and OP_NO_TLSv1_3.
 201 diff --git a/Modules/_ssl.c b/Modules/_ssl.c
 202 index 0d5c121d2c..c71d89607c 100644
 203 --- a/Modules/_ssl.c
 204 +++ b/Modules/_ssl.c
 205 @@ -4842,6 +4842,11 @@ PyInit__ssl(void)
 206  #if HAVE_TLSv1_2
 207      PyModule_AddIntConstant(m, "OP_NO_TLSv1_1", SSL_OP_NO_TLSv1_1);
 208      PyModule_AddIntConstant(m, "OP_NO_TLSv1_2", SSL_OP_NO_TLSv1_2);
 209 +#endif
 210 +#ifdef SSL_OP_NO_TLSv1_3
 211 +    PyModule_AddIntConstant(m, "OP_NO_TLSv1_3", SSL_OP_NO_TLSv1_3);
 212 +#else
 213 +    PyModule_AddIntConstant(m, "OP_NO_TLSv1_3", 0);
 214  #endif
 215      PyModule_AddIntConstant(m, "OP_CIPHER_SERVER_PREFERENCE",
 216                              SSL_OP_CIPHER_SERVER_PREFERENCE);
 217 @@ -4890,6 +4895,14 @@ PyInit__ssl(void)
 218      Py_INCREF(r);
 219      PyModule_AddObject(m, "HAS_ALPN", r);
 220
 221 +#if defined(TLS1_3_VERSION) && !defined(OPENSSL_NO_TLS1_3)
 222 +    r = Py_True;
 223 +#else
 224 +    r = Py_False;
 225 +#endif
 226 +    Py_INCREF(r);
 227 +    PyModule_AddObject(m, "HAS_TLSv1_3", r);
 228 +
 229      /* Mappings for error codes */
 230      err_codes_to_names = PyDict_New();
 231      err_names_to_codes = PyDict_New();
 232 --
 233 2.17.1
 234