Skip Menu |

This queue is for tickets about the HTML-Mail CPAN distribution.

Report information
The Basics
Id: 47298
Status: open
Priority: 0/
Queue: HTML-Mail

People
Owner: plank [...] cpan.org
Requestors: dagolden [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)



Subject: Tests should not send email
As per the subject, the tests should be attempting to send actual emails by default. If users are behind a firewall that blocks outgoing mail, this can make it look like their system is attempting to spam an outside address.
On Tue Jun 23 18:19:31 2009, DAGOLDEN wrote: Show quoted text
> As per the subject, the tests should be attempting to send actual
emails Show quoted text
> by default.
Should or shouldn't? The subject and body are contradictory. Have you read the README file? I think it covers your request (whether it is should or shouldn't).
On Tue Jun 23 21:59:51 2009, PLANK wrote: Show quoted text
> On Tue Jun 23 18:19:31 2009, DAGOLDEN wrote:
> > As per the subject, the tests should be attempting to send actual
> emails
> > by default.
> > Should or shouldn't? The subject and body are contradictory. > > Have you read the README file? > I think it covers your request (whether it is should or shouldn't).
Sorry -- typo in the body. I think it shouldn't by default, but then let the README give instructions for how to do tests that send email. The rationale is that README's often don't get read because most of them are boilerplate and sending an email from a machine behind a firewall that blocks outbound port 25 can look like a virus infected machine sending spam. Perhaps a better alternative would be to prompt with ExtUtils::MakeMaker::prompt() -- that way it doesn't get lost in a README: my ($send_email, $email_to); $send_email = prompt("Should I attempt to send email during testing? (yes/no)", "no"); if ( $send_email =~ /^y/i ) { $email_to = prompt("Where should I send the test email?", "plank@cpan.org"); } if ( $email_to ) { # write stuff out } Regards, David