gate_news: Handling missing groups
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
GNU Mailman |
Fix Released
|
Medium
|
Mark Sapiro |
Bug Description
At the moment, if a non-existing newsgroup is chosen
for mirroring, we break out of the loop:
for listname in Utils.list_names():
[...]
try:
conn, first, last = open_newsgroup(
except (socket.error, nntplib.NNTPError):
break
[...]
I believe that, at least for some NNTPErrors, we should
do a "continue" instead.
My naive patch would be this:
try:
conn, first, last = open_newsgroup(
except nntplib.NNTPError:
except socket.error:
break
But if people have good reason to break on
othernNTPErrors, we might want to do a more finegrained
check.
[http://
Changed in mailman: | |
milestone: | none → 2.1.12 |
status: | Fix Committed → Fix Released |
I encountered the same problem and applied the same fix as proposed by Nis (and added a log message in the case of an NNTPError). It runs fine, i.e., the offending newsgroup is reported in the log file, but other groups are still processed correctly. Can the attached patch be applied to the next release, or do you see any problems with this?
Kind regards,
Markus