From: Benjamin Peterson <benjamin@python.org>
Date: Thu, 19 Feb 2015 22:57:08 +0000 (-0500)
Subject: remove rc4 from the default client ciphers (closes #23481)
X-Git-Tag: v3.5.0a2~119^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=500af332f407d9a83a0232e4018e10cb49a76a79;p=python

remove rc4 from the default client ciphers (closes #23481)
---

diff --git a/Lib/ssl.py b/Lib/ssl.py
index 8c75f009c8..72115e4465 100644
--- a/Lib/ssl.py
+++ b/Lib/ssl.py
@@ -170,14 +170,12 @@ else:
 #   * Prefer any AES-GCM over any AES-CBC for better performance and security
 #   * Then Use HIGH cipher suites as a fallback
 #   * Then Use 3DES as fallback which is secure but slow
-#   * Finally use RC4 as a fallback which is problematic but needed for
-#     compatibility some times.
 #   * Disable NULL authentication, NULL encryption, and MD5 MACs for security
 #     reasons
 _DEFAULT_CIPHERS = (
     'ECDH+AESGCM:DH+AESGCM:ECDH+AES256:DH+AES256:ECDH+AES128:DH+AES:ECDH+HIGH:'
-    'DH+HIGH:ECDH+3DES:DH+3DES:RSA+AESGCM:RSA+AES:RSA+HIGH:RSA+3DES:ECDH+RC4:'
-    'DH+RC4:RSA+RC4:!aNULL:!eNULL:!MD5'
+    'DH+HIGH:ECDH+3DES:DH+3DES:RSA+AESGCM:RSA+AES:RSA+HIGH:RSA+3DES:!aNULL:'
+    '!eNULL:!MD5'
 )
 
 # Restricted and more secure ciphers for the server side
diff --git a/Misc/NEWS b/Misc/NEWS
index 3f50594e6a..5e695497b4 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -13,6 +13,8 @@ Core and Builtins
 Library
 -------
 
+- Issue #23481: Remove RC4 from the SSL module's default cipher list.
+
 - Issue #21548: Fix pydoc.synopsis() and pydoc.apropos() on modules with empty
   docstrings.