Skip to content

Mention region parameter for Mailgun mailer too #13154

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

Merged
merged 1 commit into from
Feb 19, 2020
Merged

Mention region parameter for Mailgun mailer too #13154

merged 1 commit into from
Feb 19, 2020

Conversation

pyrech
Copy link
Contributor

@pyrech pyrech commented Feb 16, 2020

Just got a hard time to get SMTP working with european servers from Mailgun. It was unclear how to configure the EU host (ie. should I change the @default host? Should I add another config?). I think region parameter deserves at least a small mention in the mailer documentation.

I also proposed to add this parameter by default in mailgun mailer's recipe: symfony/recipes#742

Copy link
Contributor

@HeahDude HeahDude left a comment

Choose a reason for hiding this comment

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

Thank you

@OskarStark
Copy link
Contributor

Thank you @pyrech.

OskarStark added a commit that referenced this pull request Feb 19, 2020
This PR was merged into the 4.4 branch.

Discussion
----------

Mention region parameter for Mailgun mailer too

Just got a hard time to get SMTP working with european servers from Mailgun. It was unclear how to configure the EU host (ie. should I change the `@default` host? Should I add another config?). I think `region` parameter deserves at least a small mention in the mailer documentation.

I also proposed to add this parameter by default in mailgun mailer's recipe: symfony/recipes#742

Commits
-------

72dddcc Mention region parameter for Mailgun mailer too
@OskarStark OskarStark merged commit 72dddcc into symfony:4.4 Feb 19, 2020
@pyrech pyrech deleted the patch-2 branch April 9, 2021 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants