Merge branch '6.2' into 6.3

* 6.2:
  fix merge
  [DependencyInjection] Filter "container.excluded" services when using `findTaggedServiceIds()`
  [Cache] Removing null coalescing assignment operator on 5.4
  [Console] Add missing ZSH mention in DumpCompletionCommand help
  [Cache] Fix storing binary keys when using pgsql
  [FrameworkBundle] Add missing monolog channel tag for messenger services
  update documentation for telegram bridge notifier
  [FrameworkBundle] Improve documentation about translation:extract --sort option
  [VarDumper] Disable links for IntelliJ platform
  [Notifier] Add bridge documentation
  [HttpClient] Add hint about `timeout` and `max_duration` options
  [HttpFoundation] Use separate caches for IpUtils checkIp4 and checkIp6
  [FrameworkBundle] Fix wiring session.handler when handler_id is null
  [FrameworkBundle] Workflow - Fix LogicException about a wrong configuration of "enabled" node

Author: nicolas-grekas

ContainerBuilder.php

@@ -1298,7 +1298,7 @@ public function findTaggedServiceIds(string $name, bool $throwOnAbstract = false
         $this->usedTags[] = $name;
         $tags = [];
         foreach ($this->getDefinitions() as $id => $definition) {
-            if ($definition->hasTag($name)) {
+            if ($definition->hasTag($name) && !$definition->hasTag('container.excluded')) {
                 if ($throwOnAbstract && $definition->isAbstract()) {
                     throw new InvalidArgumentException(sprintf('The service "%s" tagged "%s" must not be abstract.', $id, $name));
                 }

Tests/ContainerBuilderTest.php

@@ -1053,6 +1053,11 @@ public function testfindTaggedServiceIds()
             ->addTag('bar', ['bar' => 'bar'])
             ->addTag('foo', ['foofoo' => 'foofoo'])
         ;
+        $builder
+            ->register('bar', 'Bar\FooClass')
+            ->addTag('foo')
+            ->addTag('container.excluded')
+        ;
         $this->assertEquals([
             'foo' => [
                 ['foo' => 'foo'],