Skip to content
New issue

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

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

Already on GitHub? # to your account

[5.0] STMP restart for Swift_Mailer #8222

Closed
wants to merge 2 commits into from
Closed

[5.0] STMP restart for Swift_Mailer #8222

wants to merge 2 commits into from

Conversation

YOzaz
Copy link

@YOzaz YOzaz commented Mar 30, 2015

Second try.

@YOzaz YOzaz changed the title STMP restart for Swift_Mailer [5.0] STMP restart for Swift_Mailer Mar 30, 2015
{
$transport = $this->getSwiftMailerTransport();

if ( ! isset($transport))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One line this please. Also, isset is wrong here.

@taylorotwell
Copy link
Member

I don't want to force this on everyone. Not sure if it will break some existing applications using SMTP.

@lukepolo
Copy link
Contributor

Can we at least document this somewhere? It would save a lot of people time researching this only to see that Laravel does not retry the connection if your in the daemon queue (knowing that it makes sense) but seems to be a frequent occurrence.

@YOzaz
Copy link
Author

YOzaz commented Mar 30, 2016

@YOzaz
Copy link
Author

YOzaz commented Apr 7, 2016

@lukepolo af8eb1f

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

Successfully merging this pull request may close these issues.

4 participants