Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

transport not passed to Email::Sender::Simple->sendmail properly #1

Closed
bigpresh opened this issue Mar 21, 2017 · 3 comments
Closed
Assignees

Comments

@bigpresh
Copy link

From trying to help someone on the beginners@perl.org mailing list having a problem with Email::Mailer ignoring a transport:
http://www.mail-archive.com/beginners%40perl.org/msg119395.html

... I think he found a bug in Email::Mailer.

Email::Mailer sends the email with:

    # send the email with Email::Sender::Simple
    sendmail( $email_mime, $mail->{transport} );

but looking at Email::Sender::Simple->send_email(), :

    sub send_email {
        my ($class, $email, $arg) = @_;
 
        my $transport = $class->default_transport;
 
        if ($arg->{transport}) {
            ...

So, Email::Sender::Simple->send_email() is expecting the second parameter passed to it to be a hashref containing a 'transport' key, but it's getting passed an Email::Sender::Transport::SMTP object
instead, and ignoring it.

@gryphonshafer gryphonshafer self-assigned this Mar 21, 2017
@gryphonshafer
Copy link
Owner

Yes, looks like a bug. Should be trivial to fix, but it'll take me a few minutes to spin up an environment. I should have a new version released shortly thereafter.

@gryphonshafer
Copy link
Owner

OK, this should be resolved now under v1.05, just released. Should show up on CPAN in a few hours.

@bigpresh
Copy link
Author

Fantastically quick response :) Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants