Merge branch '6.4' into 7.0

* 6.4:
  [DebugBundle] Remove obsolete VarDumper feature detection
  Remove obsolete DBAL 2 compat code
  Fix for TraceableAuthenticator debug when no Auth
  [TwigBridge] Remove obsolete Workflow feature detection
  [Console] Remove Process::fromShellCommandline() detection
  [TwigBridge] Change return type of Symfony\Bridge\Twig\AppVariable::getSession()
  [mailer] reset $code variable each iteration for prevent wrong codes to be used
  bug [mailer] fix EsmtpTransport variable $code definition
  UrlHelper: minor code style fix
  [Serializer] Looking for DiscriminatorMap on interfaces when the current object also extends from a class
  [DoctrineBridge][Security] Rename loadUserByUsername tests to loadUserByIdentifier

Author: fabpot

Authenticator/Debug/TraceableAuthenticator.php

@@ -54,7 +54,7 @@ static function (BadgeInterface $badge): array {
                         'resolved' => $badge->isResolved(),
                     ];
                 },
-                $this->passport->getBadges(),
+                $this->passport?->getBadges() ?? [],
             ),
         ];
     }

Tests/Authenticator/Debug/TraceableAuthenticatorTest.php

@@ -36,4 +36,14 @@ public function testGetInfo()
         $this->assertSame($passport, $traceable->authenticate($request));
         $this->assertSame($passport, $traceable->getInfo()['passport']);
     }
+
+    public function testGetInfoWithoutAuth()
+    {
+        $authenticator = $this->createMock(AuthenticatorInterface::class);
+
+        $traceable = new TraceableAuthenticator($authenticator);
+        $this->assertNull($traceable->getInfo()['passport']);
+        $this->assertIsArray($traceable->getInfo()['badges']);
+        $this->assertSame([], $traceable->getInfo()['badges']);
+    }
 }