Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Related to issue #965150: mailkey exclusion does not work at all.
Needs to be larger, in order to allow logintoboggan mailkeys. I suggest a minimum #maxlength of 512, or, ideally, it should be a multi-line text area.
I assume that newlines in the data (if we use a textarea) won't cause problems since we're splitting the config variable using preg_split in queue_mail.inc: $ignore_list = preg_split("/[\s,'\.\"]+/", variable_get('queue_mail_keys', ''));
Comment | File | Size | Author |
---|---|---|---|
#7 | queue_mail-issue-1072232.patch | 662 bytes | Steven Jones |
#5 | queue_mail-issue-1072232.patch | 679 bytes | Steven Jones |
#2 | queue_mail-issue-1072232.patch2_.txt | 679 bytes | mcurry |
#1 | queue_mail-issue-1072232.patch | 1.46 KB | mcurry |
Comments
Comment #1
mcurry CreditAttribution: mcurry commentedI've attached a patch against the 6.x-1.x-dev archive (http://ftp.drupal.org/files/projects/queue_mail-6.x-1.x-dev.tar.gz)
Please review and comment.
Comment #2
mcurry CreditAttribution: mcurry commentedPlease ignore the previous patch, it was incorrect. Here's the correct one, applied against the 6.x-1.x-dev tarball:
Comment #3
mattcasey CreditAttribution: mattcasey commentedSeems to work for me, thanks
Comment #4
mattcasey CreditAttribution: mattcasey commentedComment #5
Steven Jones CreditAttribution: Steven Jones commentedHere's the patch from #2 for the testbot to look at.
Comment #7
Steven Jones CreditAttribution: Steven Jones commentedLet's try that again.
Comment #8
Steven Jones CreditAttribution: Steven Jones commentedThanks so much, committed to 6.x-1.x.