Cheryl Sabella added the comment:
This issue has been in pending status for over 18 months following Christian's
question to the OP, so closing as out of date.
--
nosy: +csabella
resolution: -> out of date
stage: patch review -> resolved
status: pending -> closed
Christian Heimes added the comment:
The bug is 8 years old and hasn't seen activity for three years. Is SMTP over
SSL still broken for you?
--
nosy: +christian.heimes
status: open -> pending
___
Python tracker
Torsten Bronger added the comment:
For five Ubuntu releases now, I apply this patch. In contrast to Catalin's
statement, it is not solved for me unless the upstream changes of two years ago
haven't yet found their way into Ubuntu. I'm currently using Python 2.7.4 on
Ubuntu 13.04. That
Antoine Pitrou added the comment:
When you say I apply this patch, you mean
smtplib_05_shutdown_socket_v2.patch, right?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4470
___
Antoine Pitrou added the comment:
In any case, I'm growing wary of bugfix regressions right now, so I would only
apply the patch on the default branch.
--
versions: +Python 3.4 -Python 2.6, Python 2.7, Python 3.1, Python 3.2
___
Python tracker
Torsten Bronger added the comment:
Sorry, after having had another look at it, I realised that I have a different
SSMTP issue now, non-Python-related. So for me, Ubuntu 13.04 indeed solves my
old issue.
--
___
Python tracker rep...@bugs.python.org
R. David Murray added the comment:
Lorenzo, any chance you could supply a unit test that fails without
smtplib_05_shutdown_socket.diff applied?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4470
Lorenzo M. Catucci added the comment:
On Sun, 19 May 2013, R. David Murray wrote:
RDM
RDM R. David Murray added the comment:
RDM
RDM Lorenzo, any chance you could supply a unit test that fails without
RDM smtplib_05_shutdown_socket.diff applied?
RDM
It would really be a functional test,
Lorenzo M. Catucci lore...@sancho.ccd.uniroma2.it added the comment:
I'd still prefer if smtplib_05_shutdown_socket_v2.patch could get in,
^^
since, this way the REMOTE socket close will be unconditionally correct,
instead of
Changes by Lorenzo M. Catucci lore...@sancho.ccd.uniroma2.it:
Added file: http://bugs.python.org/file22401/v2_01_fix_lmtp_init
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4470
___
Changes by Lorenzo M. Catucci lore...@sancho.ccd.uniroma2.it:
Added file: http://bugs.python.org/file22402/v2_02_mock_socket_shutdown
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4470
___
Changes by Lorenzo M. Catucci lore...@sancho.ccd.uniroma2.it:
Added file: http://bugs.python.org/file22403/v2_03_shutdown_socket_on_close
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4470
___
Changes by Lorenzo M. Catucci lore...@sancho.ccd.uniroma2.it:
Removed file: http://bugs.python.org/file22403/v2_03_shutdown_socket_on_close
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4470
___
Changes by Lorenzo M. Catucci lore...@sancho.ccd.uniroma2.it:
Removed file: http://bugs.python.org/file22402/v2_02_mock_socket_shutdown
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4470
___
Lorenzo M. Catucci lore...@sancho.ccd.uniroma2.it added the comment:
Just finished testing both 2.7 and default branches' socket close behaviour,
and it seems 05 is not strictly needed.
I'd still prefer if smtplib_05_shutdown_socket_v2.patch since, this way the
REMOTE socket close will be
R. David Murray rdmur...@bitdance.com added the comment:
Torsten, can you provide a clear, failing unittest for this?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4470
___
Torsten Bronger bron...@physik.rwth-aachen.de added the comment:
No, I don't know how to do that. All I can provide is a minimal version of my
code that triggers the above mentioned traceback. It is:
import smtplib
s = smtplib.SMTP_SSL(relay-auth.rwth-aachen.de)
s.login(***, ***)
Torsten Bronger bron...@physik.rwth-aachen.de added the comment:
Sorry, it must be:
import smtplib
s = smtplib.SMTP_SSL(relay-auth.rwth-aachen.de)
s.login(***, ***)
s.sendmail(bron...@physik.rwth-aachen.de, [bronger.ran...@googlemail.com],
Hello)
(A bracket was missing.)
--
R. David Murray rdmur...@bitdance.com added the comment:
According to your traceback you should be seeing the error in the first line
(the creation of the SMTP_SSL object). If I run that line at the python prompt
of python2.7.1, I get your connection failure. If I run it using 2.7 tip (or
Lorenzo M. Catucci lore...@sancho.ccd.uniroma2.it added the comment:
On Fri, 17 Jun 2011, R. David Murray wrote:
RDM
RDM R. David Murray rdmur...@bitdance.com added the comment:
RDM
RDM According to your traceback you should be seeing the error in the
RDM first line (the creation of the
Catalin Iacob iacobcata...@gmail.com added the comment:
Most of the problems in this issue were solved already so it could almost be
closed:
* patch 1 was addressed in #11927
* patch 2 was addressed in #4066
* patches 3 and 4 were addressed in #11893
Torsten's problem was addressed by
Torsten Bronger bron...@physik.rwth-aachen.de added the comment:
My Python version is Python 2.7.1+ and the package is called python2.7
2.7.1-5ubuntu2 (Ubuntu Natty).
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4470
Torsten Bronger bron...@physik.rwth-aachen.de added the comment:
I still have to apply Catucci's patch (or a modification of) after every Ubuntu
installation or upgrade. Otherwise, I get
...
File /usr/lib/python2.7/smtplib.py, line 752, in __init__
SMTP.__init__(self, host, port,
Changes by Antoine Pitrou pit...@free.fr:
--
versions: +Python 2.7, Python 3.1, Python 3.2
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4470
___
R. David Murray rdmur...@bitdance.com added the comment:
With the closure of 4066 all the tests in the test patch pass, so I'm
lowering the piority of this ticket.
I haven't reviewed the other patches, but the tests in the test patch
appear to be doing tests in the setup method, which doesn't
Changes by Daniel Diniz aja...@gmail.com:
--
dependencies: +smtplib SMTP_SSL._get_socket doesn't return a value
priority: - high
stage: - patch review
type: - behavior
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4470
Changes by Lorenzo M. Catucci lore...@sancho.ccd.uniroma2.it:
Added file: http://bugs.python.org/file12964/test_smtpnet.py
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4470
___
Changes by Lorenzo M. Catucci [EMAIL PROTECTED]:
Removed file: http://bugs.python.org/file12168/smtplib.py.patch
___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4470
___
Changes by Lorenzo M. Catucci [EMAIL PROTECTED]:
Added file: http://bugs.python.org/file12197/smtplib_01_default_port.diff
___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4470
___
Changes by Lorenzo M. Catucci [EMAIL PROTECTED]:
Added file: http://bugs.python.org/file12198/smtplib_02_fix_ssl.diff
___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4470
___
Changes by Lorenzo M. Catucci [EMAIL PROTECTED]:
Added file: http://bugs.python.org/file12199/smtplib_03_use_makefile.diff
___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4470
___
Changes by Lorenzo M. Catucci [EMAIL PROTECTED]:
Added file: http://bugs.python.org/file12200/smtplib_04_remove_fakefile.diff
___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4470
___
Lorenzo M. Catucci [EMAIL PROTECTED] added the comment:
I've reworked the patch into a series, like haypo requested for
poplib and imaplib.
Added file: http://bugs.python.org/file12201/smtplib_05_shutdown_socket.diff
___
Python tracker [EMAIL PROTECTED]
Changes by Giampaolo Rodola' [EMAIL PROTECTED]:
--
nosy: +giampaolo.rodola
___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4470
___
___
Python-bugs-list
New submission from Lorenzo M. Catucci [EMAIL PROTECTED]:
The enclosed patch does three things:
1. enables SMTP_SSL working: the _get_socket method was setting
self.sock instead of returning the socket to the caller, which
did reset self.sock to None
2. replace home-grown SSLFakeFile()
35 matches
Mail list logo