From f63f8d81d718d779dd6cdb55c66bf3def0fe9041 Mon Sep 17 00:00:00 2001 From: Mark Sapiro Date: Tue, 23 Nov 2021 19:38:19 -0800 Subject: Prior commit was incomplete. --- Mailman/Cgi/options.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) (limited to 'Mailman') diff --git a/Mailman/Cgi/options.py b/Mailman/Cgi/options.py index af6e3add..c5c6425a 100644 --- a/Mailman/Cgi/options.py +++ b/Mailman/Cgi/options.py @@ -166,10 +166,11 @@ def main(): 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 -- cgit v1.2.3