Merge branch '4.4' into 5.4

* 4.4:
  [Mailer] Fix edge cases in STMP transports
  Bump Symfony version to 4.4.46
  Update VERSION for 4.4.45
  Update CONTRIBUTORS for 4.4.45
  Update CHANGELOG for 4.4.45

Author: fabpot

Tests/Transport/Smtp/SmtpTransportTest.php

@@ -13,6 +13,7 @@
 
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\Mailer\Envelope;
+use Symfony\Component\Mailer\Exception\LogicException;
 use Symfony\Component\Mailer\Exception\TransportException;
 use Symfony\Component\Mailer\Transport\Smtp\SmtpTransport;
 use Symfony\Component\Mailer\Transport\Smtp\Stream\AbstractStream;
@@ -133,6 +134,35 @@ public function testWriteEncodedRecipientAndSenderAddresses()
         $this->assertContains("RCPT TO:<[email protected]>\r\n", $stream->getCommands());
         $this->assertContains("RCPT TO:<[email protected]>\r\n", $stream->getCommands());
     }
+
+    public function testAssertResponseCodeNoCodes()
+    {
+        $this->expectException(LogicException::class);
+        $this->invokeAssertResponseCode('response', []);
+    }
+
+    public function testAssertResponseCodeWithEmptyResponse()
+    {
+        $this->expectException(TransportException::class);
+        $this->expectExceptionMessage('Expected response code "220" but got empty code.');
+        $this->invokeAssertResponseCode('', [220]);
+    }
+
+    public function testAssertResponseCodeWithNotValidCode()
+    {
+        $this->expectException(TransportException::class);
+        $this->expectExceptionMessage('Expected response code "220" but got code "550", with message "550 Access Denied".');
+        $this->expectExceptionCode(550);
+        $this->invokeAssertResponseCode('550 Access Denied', [220]);
+    }
+
+    private function invokeAssertResponseCode(string $response, array $codes): void
+    {
+        $transport = new SmtpTransport($this->getMockForAbstractClass(AbstractStream::class));
+        $m = new \ReflectionMethod($transport, 'assertResponseCode');
+        $m->setAccessible(true);
+        $m->invoke($transport, $response, $codes);
+    }
 }
 
 class DummyStream extends AbstractStream

Transport/Smtp/EsmtpTransport.php

@@ -100,7 +100,9 @@ public function addAuthenticator(AuthenticatorInterface $authenticator): void
 
     protected function doHeloCommand(): void
     {
-        $capabilities = $this->callHeloCommand();
+        if (!$capabilities = $this->callHeloCommand()) {
+            return;
+        }
 
         /** @var SocketStream $stream */
         $stream = $this->getStream();
@@ -129,6 +131,8 @@ private function callHeloCommand(): array
         } catch (TransportExceptionInterface $e) {
             try {
                 parent::doHeloCommand();
+
+                return [];
             } catch (TransportExceptionInterface $ex) {
                 if (!$ex->getCode()) {
                     throw $e;

Transport/Smtp/SmtpTransport.php

@@ -303,7 +303,8 @@ private function assertResponseCode(string $response, array $codes): void
         if (!$valid || !$response) {
             $codeStr = $code ? sprintf('code "%s"', $code) : 'empty code';
             $responseStr = $response ? sprintf(', with message "%s"', trim($response)) : '';
-            throw new TransportException(sprintf('Expected response code "%s" but got ', implode('/', $codes), $codeStr).$codeStr.$responseStr.'.', $code);
+
+            throw new TransportException(sprintf('Expected response code "%s" but got ', implode('/', $codes)).$codeStr.$responseStr.'.', $code ?: 0);
         }
     }