symfony · ogizanagi · Sep 15, 2023 · stof · Sep 15, 2023 · ogizanagi
@@ -435,6 +435,10 @@ private function addPasswordHashersSection(ArrayNodeDefinition $rootNode): void
                         ->canBeUnset()
                         ->performNoDeepMerging()
                         ->beforeNormalization()->ifString()->then(fn ($v) => ['algorithm' => $v])->end()
+                        ->validate()
+                            ->ifTrue(fn ($v) => isset($v['migrate_from'], $v['id']) && 0 !== \count($v['migrate_from']))
+                            ->thenInvalid('You cannot use "migrate_from" when using a custom service id.')
+                        ->end()
                         ->children()
                             ->scalarNode('algorithm')
                                 ->cannotBeEmpty()

@@ -191,4 +191,27 @@ public function testFirewalls()
         $configuration = new MainConfiguration(['stub' => $factory], []);
         $configuration->getConfigTreeBuilder();
     }
+
+    public function testHasherThrowsOnServiceIdWithMigrate()
+    {
+        $config = [
+            'password_hashers' => [
+                'legacy' => 'bcrypt',
+                'custom' => [
+                    'id' => 'app.custom_hasher',
+                    'migrate_from' => 'legacy',
+                ],
+            ],
+        ];
+
+        $config = array_merge(static::$minimalConfig, $config);
+
+        $processor = new Processor();
+        $configuration = new MainConfiguration([], []);
+
+        $this->expectException(InvalidConfigurationException::class);
+        $this->expectExceptionMessage('Invalid configuration for path "security.password_hashers.custom": You cannot use "migrate_from" when using a custom service id');
+
+        $processor->processConfiguration($configuration, [$config]);
+    }
 }