Skip to content

Commit

Permalink
minor #32894 [Mailer] Change the order of authenticators (fabpot)
Browse files Browse the repository at this point in the history
This PR was merged into the 4.3 branch.

Discussion
----------

[Mailer] Change the order of authenticators

| Q             | A
| ------------- | ---
| Branch?       | 4.3
| Bug fix?      | no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

This PR uses the same order of SMTP authenticators as Swiftmailer. It should not change anything, but it does not hurt while I'm hunting for #32148 :)

Commits
-------

0800e90 [Mailer] fixed the order of authenticators
  • Loading branch information
fabpot committed Aug 3, 2019
2 parents 8f1d9d2 + 0800e90 commit 8e3161b
Showing 1 changed file with 2 additions and 2 deletions.
Expand Up @@ -38,10 +38,10 @@ public function __construct(string $host = 'localhost', int $port = 25, string $
parent::__construct(null, $dispatcher, $logger);

$this->authenticators = [
new Auth\PlainAuthenticator(),
new Auth\CramMd5Authenticator(),
new Auth\LoginAuthenticator(),
new Auth\PlainAuthenticator(),
new Auth\XOAuth2Authenticator(),
new Auth\CramMd5Authenticator(),
];

/** @var SocketStream $stream */
Expand Down

0 comments on commit 8e3161b

Please sign in to comment.