------------------------------------------------------------
revno: 1881
fixes bug: https://launchpad.net/bugs/1951769
committer: Mark Sapiro <m...@msapiro.net>
branch nick: 2.1
timestamp: Tue 2021-11-23 19:38:19 -0800
message:
  Prior commit was incomplete.
modified:
  Mailman/Cgi/options.py


--
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/Cgi/options.py'
--- Mailman/Cgi/options.py	2021-11-22 04:31:11 +0000
+++ Mailman/Cgi/options.py	2021-11-24 03:38:19 +0000
@@ -166,10 +166,11 @@
         return
     # Sanity check the user, but only give the "no such member" error when
     # using public rosters, otherwise, we'll leak membership information.
-    if not mlist.isMember(user) and mlist.private_roster == 0:
-        doc.addError(_('No such member: %(safeuser)s.'))
-        loginpage(mlist, doc, None, language)
-        print doc.Format()
+    if not mlist.isMember(user):
+        if mlist.private_roster == 0:
+            doc.addError(_('No such member: %(safeuser)s.'))
+            loginpage(mlist, doc, None, language)
+            print doc.Format()
         return
 
     # Avoid cross-site scripting attacks

_______________________________________________
Mailman-checkins mailing list -- mailman-checkins@python.org
To unsubscribe send an email to mailman-checkins-le...@python.org
https://mail.python.org/mailman3/lists/mailman-checkins.python.org/
Member address: arch...@jab.org

Reply via email to