diff options
| author | bwarsaw | 2002-04-19 17:07:29 +0000 |
|---|---|---|
| committer | bwarsaw | 2002-04-19 17:07:29 +0000 |
| commit | 3e60c08b4da9a7971a47dda85fc97d97087ed001 (patch) | |
| tree | dc044db47e3f9837b244f6116b04b394d44297a6 | |
| parent | f8b701c6615c12f399bdbbbaec77f5559285a4bc (diff) | |
| download | mailman-3e60c08b4da9a7971a47dda85fc97d97087ed001.tar.gz mailman-3e60c08b4da9a7971a47dda85fc97d97087ed001.tar.zst mailman-3e60c08b4da9a7971a47dda85fc97d97087ed001.zip | |
ProcessConfirmation(): When checking the subscribe_policy, be sure to
compare it to 2 and 3 (approval & approval+confirm). It's possible
the policy was changed in the middle of the confirmation process.
Reported by Dale Newfield.
| -rw-r--r-- | Mailman/MailList.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/Mailman/MailList.py b/Mailman/MailList.py index 70c4aba8e..3924a511b 100644 --- a/Mailman/MailList.py +++ b/Mailman/MailList.py @@ -1033,7 +1033,10 @@ class MailList(MailCommandHandler, HTMLFormatter, Deliverer, ListAdmin, lang = userdesc.language except ValueError: raise Errors.MMBadConfirmation, 'bad subscr data %s' % (data,) - if self.subscribe_policy == 3: # confirm + approve + # We check for both 2 (approval required) and 3 (confirm + + # approval) because the policy could have been changed in the + # middle of the confirmation dance. + if self.subscribe_policy in (2, 3): self.HoldSubscription(addr, fullname, password, digest, lang) name = self.real_name raise Errors.MMNeedApproval, _( |