[issue1186900] nntplib shouldn't raise generic EOFError

2014-08-21 Thread Martin Panter
Martin Panter added the comment: NNTPConnectError does still seem a slightly awkward name. I would go for NNTPConnectionError instead, but I’m also happy to put my bikeshed paint away let this patch be applied as is :) Handling of NNTPTemporaryError with a code of 400 is similar to handling

[issue1186900] nntplib shouldn't raise generic EOFError

2014-08-21 Thread Martin Panter
Martin Panter added the comment: Some more points: * I suggest adding something like this to the documentation: exception nntplib.NNTPConnect[ion]Error Exception raised when the server unexpectedly shuts down the connection. * The tests should use BytesIO rather than StringIO. Other than

[issue1186900] nntplib shouldn't raise generic EOFError

2014-08-21 Thread Martin Panter
Martin Panter added the comment: PPS: Documentation should probably have the “New in version 3.5” tag as well -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1186900 ___

[issue1186900] nntplib shouldn't raise generic EOFError

2014-08-21 Thread Lita Cho
Lita Cho added the comment: Thank yo so much, Martin! I will incorporate these changes and add them soon! -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1186900 ___

[issue1186900] nntplib shouldn't raise generic EOFError

2014-08-04 Thread Antoine Pitrou
Antoine Pitrou added the comment: Indeed, I find the description rather vague and potentially misleading. Connection establishment would usually refer to the TCP connection, but an EOFError is actually a high-level, logical error (it has nothing to do with networking per se: it's probably

[issue1186900] nntplib shouldn't raise generic EOFError

2014-08-04 Thread Lita Cho
Lita Cho added the comment: I'ved changed the comment to say Connection closed unexpectedly. -- Added file: http://bugs.python.org/file36264/nntplib_error_v2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1186900

[issue1186900] nntplib shouldn't raise generic EOFError

2014-08-03 Thread Martin Panter
Martin Panter added the comment: I could be wrong, but isn’t this error raised when expecting a response from any command, not just during “connection establishment”? Perhaps change the docstring to say something like “Connection closed unexpectedly” instead. -- nosy: +vadmium

[issue1186900] nntplib shouldn't raise generic EOFError

2014-07-13 Thread Jessica McKellar
Changes by Jessica McKellar jesst...@mit.edu: -- keywords: +needs review stage: test needed - patch review ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1186900 ___

[issue1186900] nntplib shouldn't raise generic EOFError

2014-07-13 Thread Berker Peksag
Berker Peksag added the comment: diff -r 8f85262fbe8a Lib/nntplib.py --- a/Lib/nntplib.pySun Jul 06 02:24:24 2014 -0400 +++ b/Lib/nntplib.pyThu Jul 10 16:10:38 2014 -0700 @@ -122,6 +122,9 @@ Error in response data pass +class NNTPConnectError(NNTPError): +Error during

[issue1186900] nntplib shouldn't raise generic EOFError

2014-07-13 Thread Ezio Melotti
Ezio Melotti added the comment: Wouldn't this be backward incompatible? Even if the EOFError that is raised is not documented explicitly, people might be catching it, and switching to a new exception would break their programs. Maybe NNTPConnectError should inherit from EOFError too?

[issue1186900] nntplib shouldn't raise generic EOFError

2014-07-13 Thread Lita Cho
Lita Cho added the comment: That's a good point. I can add that so the NNTPConnectError can inherit the EOFError -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1186900 ___

[issue1186900] nntplib shouldn't raise generic EOFError

2014-07-13 Thread Lita Cho
Lita Cho added the comment: Here is an updated patch. -- Added file: http://bugs.python.org/file35941/nntplib_error.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1186900 ___

[issue1186900] nntplib shouldn't raise generic EOFError

2014-07-10 Thread Lita Cho
Lita Cho added the comment: I am going to fix it so that it raises the NNTPConnectionError rather than update the documentation. -- nosy: +Lita.Cho, jesstess ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1186900

[issue1186900] nntplib shouldn't raise generic EOFError

2014-07-10 Thread Lita Cho
Lita Cho added the comment: I have a fix and added some test coverage in order to make sure the NNTFConnectError was being called. However, in the test case, I am monkey patching. If there is a way to do this with mock, I would appreciate the feedback. -- keywords: +patch Added file:

[issue1186900] nntplib shouldn't raise generic EOFError

2014-02-03 Thread Mark Lawrence
Changes by Mark Lawrence breamore...@yahoo.co.uk: -- nosy: -BreamoreBoy ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1186900 ___ ___

[issue1186900] nntplib shouldn't raise generic EOFError

2013-03-07 Thread Ankur Ankan
Changes by Ankur Ankan ankuran...@gmail.com: -- nosy: +Ankur.Ankan ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1186900 ___ ___ Python-bugs-list

[issue1186900] nntplib shouldn't raise generic EOFError

2010-08-21 Thread Mark Lawrence
Mark Lawrence breamore...@yahoo.co.uk added the comment: The OP would accept a documentation change if the code's not going to be changed. -- nosy: +BreamoreBoy versions: +Python 3.2 -Python 2.7 ___ Python tracker rep...@bugs.python.org

[issue1186900] nntplib shouldn't raise generic EOFError

2009-02-15 Thread Daniel Diniz
Changes by Daniel Diniz aja...@gmail.com: -- stage: - test needed versions: +Python 2.7 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1186900 ___