Code review comment for lp:~ralfjung-e/mailman/2.1
Revision history for this message
| Ralf Jung (ralfjung-e) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
Ah good catch, I forgot to test this without CAPTCHAS set.
However, the variable is used later to compute the hash for the form secret. Removing it conditionally will mean we have to mirror the same logic on the other side in subscribe.py. So for now I changed it to use the empty string instead. I don't know if this will automatically get included in this MR?