Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Mailer] Allow create custom transports and register DSN for it #31385

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

Closed
Koc opened this issue May 5, 2019 · 2 comments
Closed

[Mailer] Allow create custom transports and register DSN for it #31385

Koc opened this issue May 5, 2019 · 2 comments

Comments

@Koc
Copy link
Contributor

Koc commented May 5, 2019

For the now there is no extension point for registering custom DSN for adding own Mailer Transport. Would be nice allow create custom transports like already done in Messenger.

@javiereguiluz
Copy link
Member

We can't provide official integrations for all mailer services, only for the most popular/used ones ... so I guess this is something we should implement to let the community create their own mailer integrations. Thanks.

@Koc
Copy link
Contributor Author

Koc commented Jun 7, 2019

See #31946 with some ideas about implementation

Koc added a commit to Koc/symfony that referenced this issue Jun 11, 2019
Koc added a commit to Koc/symfony that referenced this issue Jun 29, 2019
Koc added a commit to Koc/symfony that referenced this issue Jun 30, 2019
Koc added a commit to Koc/symfony that referenced this issue Jul 4, 2019
Koc added a commit to Koc/symfony that referenced this issue Jul 14, 2019
Koc added a commit to Koc/symfony that referenced this issue Jul 14, 2019
Koc added a commit to Koc/symfony that referenced this issue Jul 14, 2019
Koc added a commit to Koc/symfony that referenced this issue Jul 16, 2019
@fabpot fabpot closed this as completed Jul 17, 2019
fabpot added a commit that referenced this issue Jul 17, 2019
…stom transports (Koc)

This PR was merged into the 4.4 branch.

Discussion
----------

[Mailer] Extract transport factory and allow create custom transports

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes, failure unrelated (master hasn't this PR)
| Fixed tickets | #31385, #32523
| License       | MIT
| Doc PR        | TBD

Alternative approach to allow create custom transports and register DSN for it. Replaces #31931, #31935 . Similar to already existent TansportFactory from Messenger.

TODO:
 - [x] Update changelog
 - [x] Add more tests for factories
 - [x] Add test for configuration + DI extension

Commits
-------

5b9cded Add transport factories (closes #31385, closes #32523)
nicolas-grekas added a commit that referenced this issue Jul 17, 2019
* 4.4:
  [WebProfilerBundle] Remove unneeded information in the routing panel
  Add transport factories (closes #31385, closes #32523)
  [Lock] remove uusage of the StoreInterface
  [HttpClient] make toStream() throw by default
  [Mailer] added XML configuration for the mailer envelope
  added missing test
  [Mailer] Allow register mailer configuration in xml format
  [VarDumper] Allow to configure VarDumperTestTrait casters & flags
  Improve errors when trying to find a writable property
  [Lock] add aliases for LockFactory
  fixed CS
  fix some deprecations and add upgrade instructions
  fix typo
  Added Nl translations
  fixed CS
  [FrameworkBundle] Fix descriptor of routes described as callable array
  [Debug][DebugClassLoader] Include found files instead of requiring them
  [HttpKernel] fix tests
  Adding missing event_dispatcher wiring for messenger.middleware.send_message
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants