Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[Mailer] Implement additional mailer transport options#37432
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Uh oh!
There was an error while loading.Please reload this page.
9efb0e4 to0db6030Comparefritzmg commentedAug 27, 2020
OskarStark commentedAug 27, 2020
|
fritzmg commentedAug 27, 2020
Hm, what do you mean by that? It i used in the example. It is missing from the proposed documentation, that is true. |
0db6030 to665d1cdComparefabpot commentedAug 27, 2020
Thank you@fritzmg. |
Description-----------| Q | A| -----------------| ---| Fixed issues |Fixes#2164| Docs PR or issue | -This PR adds support for the `native://default` mailer transport, which uses the PHP `sendmail_path` configuration. Some notes:* This feature is available in Symfony `5.2.0` and up. Since the `symfony/mailer` dependency is independent from other Symfony packages, it can be allowed to be installed, while everything else stays at Symfony `4.4.*`.* Since the actual transport factory for the native transport is however defined by the `symfony/framework-bundle`, an additional compiler pass is necessary, to add said transport factory to the container, if supported and not already present. This can be removed once the core requires at least Symfony `5.2.*`.* Changing the `symfony/mailer` dependency to include `5.2.*` also allows you to use [additional options](symfony/symfony#37432) for your transport (like a local domain for an SMTP relay or a custom sendmail command).It would be important to include this in the upcoming Contao 4.11 version, as otherwise there exists no possibility of using a custom sendmail command and thus you are stuck with the hardcoded one.Commits-------5ea20a7 support native transport53836ae use class_exists3bbd552 update ContaoCoreBundleTestb084330 check for native mailer support in PluginTest too4df690a update AddNativeTransportFactoryPassTest for prefer-lowest72e72d2 Apply suggestions from code reviewCo-authored-by: Leo Feyer <github@contao.org>ad36b75 merge with mastere98b617 removed superfluous function1969b8e fix merge error9fbdaf8 CS49187f0 Merge branch 'master' into use-native-mailer
Description-----------| Q | A| -----------------| ---| Fixed issues | Fixes #2164| Docs PR or issue | -This PR adds support for the `native://default` mailer transport, which uses the PHP `sendmail_path` configuration. Some notes:* This feature is available in Symfony `5.2.0` and up. Since the `symfony/mailer` dependency is independent from other Symfony packages, it can be allowed to be installed, while everything else stays at Symfony `4.4.*`.* Since the actual transport factory for the native transport is however defined by the `symfony/framework-bundle`, an additional compiler pass is necessary, to add said transport factory to the container, if supported and not already present. This can be removed once the core requires at least Symfony `5.2.*`.* Changing the `symfony/mailer` dependency to include `5.2.*` also allows you to use [additional options](symfony/symfony#37432) for your transport (like a local domain for an SMTP relay or a custom sendmail command).It would be important to include this in the upcoming Contao 4.11 version, as otherwise there exists no possibility of using a custom sendmail command and thus you are stuck with the hardcoded one.Commits-------5ea20a71 support native transport53836ae6 use class_exists3bbd552f update ContaoCoreBundleTestb084330c check for native mailer support in PluginTest too4df690a8 update AddNativeTransportFactoryPassTest for prefer-lowest72e72d27 Apply suggestions from code reviewCo-authored-by: Leo Feyer <github@contao.org>ad36b75c merge with mastere98b617f removed superfluous function1969b8e1 fix merge error9fbdaf8e CS49187f05 Merge branch 'master' into use-native-mailer
Description-----------| Q | A| -----------------| ---| Fixed issues | Fixes #2164| Docs PR or issue | -This PR adds support for the `native://default` mailer transport, which uses the PHP `sendmail_path` configuration. Some notes:* This feature is available in Symfony `5.2.0` and up. Since the `symfony/mailer` dependency is independent from other Symfony packages, it can be allowed to be installed, while everything else stays at Symfony `4.4.*`.* Since the actual transport factory for the native transport is however defined by the `symfony/framework-bundle`, an additional compiler pass is necessary, to add said transport factory to the container, if supported and not already present. This can be removed once the core requires at least Symfony `5.2.*`.* Changing the `symfony/mailer` dependency to include `5.2.*` also allows you to use [additional options](symfony/symfony#37432) for your transport (like a local domain for an SMTP relay or a custom sendmail command).It would be important to include this in the upcoming Contao 4.11 version, as otherwise there exists no possibility of using a custom sendmail command and thus you are stuck with the hardcoded one.Commits-------5ea20a71 support native transport53836ae6 use class_exists3bbd552f update ContaoCoreBundleTestb084330c check for native mailer support in PluginTest too4df690a8 update AddNativeTransportFactoryPassTest for prefer-lowest72e72d27 Apply suggestions from code reviewCo-authored-by: Leo Feyer <github@contao.org>ad36b75c merge with mastere98b617f removed superfluous function1969b8e1 fix merge error9fbdaf8e CS49187f05 Merge branch 'master' into use-native-mailer
…ao#2554)Description-----------| Q | A| -----------------| ---| Fixed issues |Fixescontao#2164| Docs PR or issue | -This PR adds support for the `native://default` mailer transport, which uses the PHP `sendmail_path` configuration. Some notes:* This feature is available in Symfony `5.2.0` and up. Since the `symfony/mailer` dependency is independent from other Symfony packages, it can be allowed to be installed, while everything else stays at Symfony `4.4.*`.* Since the actual transport factory for the native transport is however defined by the `symfony/framework-bundle`, an additional compiler pass is necessary, to add said transport factory to the container, if supported and not already present. This can be removed once the core requires at least Symfony `5.2.*`.* Changing the `symfony/mailer` dependency to include `5.2.*` also allows you to use [additional options](symfony/symfony#37432) for your transport (like a local domain for an SMTP relay or a custom sendmail command).It would be important to include this in the upcoming Contao 4.11 version, as otherwise there exists no possibility of using a custom sendmail command and thus you are stuck with the hardcoded one.Commits-------5ea20a7 support native transport53836ae use class_exists3bbd552 update ContaoCoreBundleTestb084330 check for native mailer support in PluginTest too4df690a update AddNativeTransportFactoryPassTest for prefer-lowest72e72d2 Apply suggestions from code reviewCo-authored-by: Leo Feyer <github@contao.org>ad36b75 merge with mastere98b617 removed superfluous function1969b8e fix merge error9fbdaf8 CS49187f0 Merge branch 'master' into use-native-mailer
This PR was squashed before being merged into the 5.2 branch.Discussion----------[Mailer] Document new Mailer transport optionsThis documents the additional options for the Mailer component introduced insymfony/symfony#37432Commits-------f713212 [Mailer] Document new Mailer transport options
Uh oh!
There was an error while loading.Please reload this page.
This implements additional transport configuration options mentioned in#37300. It also adds a
commandoption to be able to define the command used by thesendmailtransport.Examples: