------------------------------------------------------------
revno: 1586
fixes bug: https://launchpad.net/bugs/1512866
committer: Mark Sapiro <m...@msapiro.net>
branch nick: 2.1
timestamp: Fri 2015-11-06 09:14:29 -0800
message:
  Posts gated to usenet will no longer have other than the target group
  in the Newsgroups: header.
modified:
  Mailman/Queue/NewsRunner.py
  NEWS


--
lp:mailman/2.1
https://code.launchpad.net/~mailman-coders/mailman/2.1

Your team Mailman Checkins is subscribed to branch lp:mailman/2.1.
To unsubscribe from this branch go to 
https://code.launchpad.net/~mailman-coders/mailman/2.1/+edit-subscription
=== modified file 'Mailman/Queue/NewsRunner.py'
--- Mailman/Queue/NewsRunner.py	2005-08-27 01:40:17 +0000
+++ Mailman/Queue/NewsRunner.py	2015-11-06 17:14:29 +0000
@@ -1,4 +1,4 @@
-# Copyright (C) 2000-2005 by the Free Software Foundation, Inc.
+# Copyright (C) 2000-2015 by the Free Software Foundation, Inc.
 #
 # This program is free software; you can redistribute it and/or
 # modify it under the terms of the GNU General Public License
@@ -111,20 +111,12 @@
         del msg['subject']
         msg['subject'] = stripped_subject
     # Add the appropriate Newsgroups: header
-    ngheader = msg['newsgroups']
-    if ngheader is not None:
-        # See if the Newsgroups: header already contains our linked_newsgroup.
-        # If so, don't add it again.  If not, append our linked_newsgroup to
-        # the end of the header list
-        ngroups = [s.strip() for s in ngheader.split(',')]
-        if mlist.linked_newsgroup not in ngroups:
-            ngroups.append(mlist.linked_newsgroup)
-            # Subtitute our new header for the old one.
-            del msg['newsgroups']
-            msg['Newsgroups'] = COMMASPACE.join(ngroups)
-    else:
-        # Newsgroups: isn't in the message
-        msg['Newsgroups'] = mlist.linked_newsgroup
+    if msg['newsgroups'] is not None:
+        # This message is gated from our list to it's associated usnet group.
+        # If it has a Newsgroups: header mentioning other groups, it's not
+        # up to us to post it to those groups.
+        del msg['newsgroups']
+    msg['Newsgroups'] = mlist.linked_newsgroup
     # Note: We need to be sure two messages aren't ever sent to the same list
     # in the same process, since message ids need to be unique.  Further, if
     # messages are crossposted to two Usenet-gated mailing lists, they each

=== modified file 'NEWS'
--- NEWS	2015-10-18 02:30:06 +0000
+++ NEWS	2015-11-06 17:14:29 +0000
@@ -22,6 +22,9 @@
 
   Bug fixes and other patches
 
+    - Posts gated to usenet will no longer have other than the target group
+      in the Newsgroups: header.  (LP: #1512866)
+
     - Invalid regexps in *_these_nonmembers, subscribe_auto_approval and
       ban_list are now logged.  (LP: #1507241)
 

_______________________________________________
Mailman-checkins mailing list
Mailman-checkins@python.org
Unsubscribe: 
https://mail.python.org/mailman/options/mailman-checkins/archive%40jab.org

Reply via email to