diff options
| author | bwarsaw | 1999-02-27 22:19:51 +0000 |
|---|---|---|
| committer | bwarsaw | 1999-02-27 22:19:51 +0000 |
| commit | 0a919953f3aeb83d34438b52889876260af6802f (patch) | |
| tree | ef17e9886cd68e2d0df724bc4bf24bfdb3d66c68 | |
| parent | aadd11cc5b7dee0e3aac9f6f2313c192ec76c2fb (diff) | |
| download | mailman-0a919953f3aeb83d34438b52889876260af6802f.tar.gz mailman-0a919953f3aeb83d34438b52889876260af6802f.tar.zst mailman-0a919953f3aeb83d34438b52889876260af6802f.zip | |
| -rw-r--r-- | Mailman/MailCommandHandler.py | 24 |
1 files changed, 10 insertions, 14 deletions
diff --git a/Mailman/MailCommandHandler.py b/Mailman/MailCommandHandler.py index af891e01b..203297f4a 100644 --- a/Mailman/MailCommandHandler.py +++ b/Mailman/MailCommandHandler.py @@ -241,10 +241,18 @@ class MailCommandHandler: else: ShowSetUsage() return + try: + sender = self.FindUser(mail.GetSender()) + self.ConfirmUserPassword(sender, args[2]) + except Errors.MMNotAMemberError: + self.AddError("%s isn't subscribed to this list." + % mail.GetSender()) + return + except Errors.MMBadPasswordError: + self.AddError("You gave the wrong password.") + return if args[0] == 'digest': try: - sender = self.FindUser(mail.GetSender()) - self.ConfirmUserPassword(sender, args[2]) self.SetUserDigest(mail.GetSender(), value) self.AddToResponse("Succeeded.") except Errors.MMAlreadyDigested: @@ -258,16 +266,11 @@ class MailCommandHandler: self.AddError("List only accepts digest members.") except Errors.MMCantDigestError: self.AddError("List doesn't accept digest members.") - except Errors.MMNotAMemberError: - self.AddError("%s isn't subscribed to this list." - % mail.GetSender()) except Errors.MMListNotReady: self.AddError("List is not functional.") except Errors.MMNoSuchUserError: self.AddError("%s is not subscribed to this list." % mail.GetSender()) - except Errors.MMBadPasswordError: - self.AddError("You gave the wrong password.") except Errors.MMNeedApproval: self.AddApprovalMsg(cmd) except: @@ -278,15 +281,8 @@ class MailCommandHandler: self.AddError("%s" % sys.exc_type) elif option_info.has_key(args[0]): try: - sender = self.FindUser(mail.GetSender()) - if not sender: - self.AddError("You aren't subscribed.") - return - self.ConfirmUserPassword(sender, args[2]) self.SetUserOption(sender, option_info[args[0]], value) self.AddToResponse("Succeeded.") - except Errors.MMBadPasswordError: - self.AddError("You gave the wrong password.") except: self.AddError("An unknown Mailman error occured.") self.AddError("Please forward on your request to %s" % |
