]> granicus.if.org Git - python/commitdiff
Fix Issue #4066: smtplib.SMTP_SSL._get_socket now correctly returns
authorR. David Murray <rdmurray@bitdance.com>
Sat, 23 May 2009 00:48:58 +0000 (00:48 +0000)
committerR. David Murray <rdmurray@bitdance.com>
Sat, 23 May 2009 00:48:58 +0000 (00:48 +0000)
the socket.  Patch by Farhan Ahmad, test by Marcin Bachry.

Lib/smtplib.py
Lib/test/test_smtpnet.py [new file with mode: 0644]
Misc/ACKS
Misc/NEWS

index 69d7f71d96ab1f1a11865c7bac3b7d6057b09b61..9300474e1d9e587ebe93edd83a34a24d0dc8e4b1 100755 (executable)
@@ -754,9 +754,10 @@ if _have_ssl:
 
         def _get_socket(self, host, port, timeout):
             if self.debuglevel > 0: print>>stderr, 'connect:', (host, port)
-            self.sock = socket.create_connection((host, port), timeout)
-            self.sock = ssl.wrap_socket(self.sock, self.keyfile, self.certfile)
-            self.file = SSLFakeFile(self.sock)
+            new_socket = socket.create_connection((host, port), timeout)
+            new_socket = ssl.wrap_socket(new_socket, self.keyfile, self.certfile)
+            self.file = SSLFakeFile(new_socket)
+            return new_socket
 
     __all__.append("SMTP_SSL")
 
diff --git a/Lib/test/test_smtpnet.py b/Lib/test/test_smtpnet.py
new file mode 100644 (file)
index 0000000..c38eeee
--- /dev/null
@@ -0,0 +1,26 @@
+#!/usr/bin/env python
+
+import unittest
+from test import test_support
+import smtplib
+
+test_support.requires(
+    "network",
+    "use of network resource is not enabled and "
+    "test requires Internet access for communication with smtp.gmail.com:465",
+    )
+
+class SmtpSSLTest(unittest.TestCase):
+    testServer = 'smtp.gmail.com'
+    remotePort = 465
+
+    def test_connect(self):
+        server = smtplib.SMTP_SSL(self.testServer, self.remotePort)
+        server.ehlo()
+        server.quit()
+
+def test_main():
+    test_support.run_unittest(SmtpSSLTest)
+
+if __name__ == "__main__":
+    test_main()
index abcad74cb86cf0179c63b25273c05ed0391dcdfd..ccb782774ea64c84e971f04d3ab11b8aeb1a1e38 100644 (file)
--- a/Misc/ACKS
+++ b/Misc/ACKS
@@ -12,6 +12,7 @@ PS: In the standard Python distribution, this file is encoded in Latin-1.
 
 David Abrahams
 Jim Ahlstrom
+Farhan Ahmad
 Jyrki Alakuijala
 Billy G. Allie
 Kevin Altis
@@ -29,6 +30,7 @@ Jan-Hein B"uhrman
 Donovan Baarda
 Attila Babo
 Alfonso Baciero
+Marcin Bachry
 Dwayne Bailey
 Stig Bakken
 Greg Ball
index db2d60d3ec515236aee48b79cb64460282d086d5..358d6eddc936ff2f56c77a57dc1e399eeff68bf3 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -302,6 +302,9 @@ Core and Builtins
 Library
 -------
 
+- Issue #4066: smtplib.SMTP_SSL._get_socket now correctly returns the socket.
+  Patch by Farhan Ahmad, test by Marcin Bachry.
+
 - Issue #6062: In distutils, fixed the package option of build_ext. Feedback 
   and tests on pywin32 by Tim Golden.