8000 bug #53061 [Mailer] [Mailjet] Disable tls for mailjet as it should us… · symfony/symfony@dfa5e5d · GitHub
[go: up one dir, main page]
Skip to content

Commit dfa5e5d

Browse files
nicolas-grekasnicolas-grekas
committed
bug #53061 [Mailer] [Mailjet] Disable tls for mailjet as it should use STARTTLS (RFreij)
This PR was merged into the 6.4 branch. Discussion ---------- [Mailer] [Mailjet] Disable tls for mailjet as it should use STARTTLS | Q | A | ------------- | --- | Branch? | 6.4 <!-- see below --> | Bug fix? | yes | New feature? | no <!-- please update src/**/CHANGELOG.md files --> | Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Issues | <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead --> | License | MIT #52560 updates the default mailjet port and references the same change for mailgun. Similar to #51042 and with the same error message, the mailjet connection doesn't work now. The PR above already has a reference, but for completion sake: Ref #34846 (comment) for more explanation Commits ------- 709d56f fix(mailjet-smtp): disable tls for mailjet as it should use starttls instead
2 parents 6c702cf + 709d56f commit dfa5e5d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

‎src/Symfony/Component/Mailer/Bridge/Mailjet/Transport/MailjetSmtpTransport.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ class MailjetSmtpTransport extends EsmtpTransport
1919
{
2020
public function __construct(string $username, #[\SensitiveParameter] string $password, EventDispatcherInterface $dispatcher = null, LoggerInterface $logger = null)
2121
{
22-
parent::__construct('in-v3.mailjet.com', 587, true, $dispatcher, $logger);
22+
parent::__construct('in-v3.mailjet.com', 587, false, $dispatcher, $logger);
2323

2424
$this->setUsername($username);
2525
$this->setPassword($password);

0 commit comments

Comments
 (0)
0