Skip to content

Commit dfa5e5d

Browse files
bug symfony#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 symfony#52560 updates the default mailjet port and references the same change for mailgun. Similar to symfony#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 symfony#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

Lines changed: 1 addition & 1 deletion
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)