fix for issue #7 #10

Merged
merged 1 commit into from Jan 22, 2012

Conversation

Projects
None yet
2 participants
Member

marcinkuzminski commented Jan 22, 2012

This fixes issue #7, bcc still will get delivered since bcc addresses are appended to the mail recipients list

Owner

mcdonc commented Jan 22, 2012

I'll be lazy and ask this question.

Will bcc work when I apply this patch? Or does this patch just make it impossible to use a bcc header?

Member

marcinkuzminski commented Jan 22, 2012

I wrote it in the comment: "This fixes issue #7, bcc still will get delivered since bcc addresses are appended to the mail recipients list"

THis patch removes BCC header, and BCC recipients still are delivered due to this:

    @property
    def send_to(self):
        return set(self.recipients) | set(self.bcc or ()) | set(self.cc or ())

@mcdonc mcdonc merged commit 7b3dcaa into Pylons:master Jan 22, 2012

Owner

mcdonc commented Jan 22, 2012

Great, thanks. Applied. I'll make a release soon.

Owner

mcdonc commented Jan 22, 2012

0.6 released

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment