Skip to content

Commit 1b1ed67

Browse files
author
Robin Chalas
committed
Merge branch '3.3' into 3.4
* 3.3: [SecurityBundle] Don't register in memory users as services
2 parents 5f75d43 + 00621d4 commit 1b1ed67

File tree

2 files changed

+4
-13
lines changed

2 files changed

+4
-13
lines changed

src/Symfony/Bundle/SecurityBundle/DependencyInjection/Security/UserProvider/InMemoryFactory.php

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
use Symfony\Component\DependencyInjection\ChildDefinition;
1616
use Symfony\Component\DependencyInjection\ContainerBuilder;
1717
use Symfony\Component\DependencyInjection\Parameter;
18-
use Symfony\Component\DependencyInjection\Reference;
1918

2019
/**
2120
* InMemoryFactory creates services for the memory provider.
@@ -29,17 +28,13 @@ public function create(ContainerBuilder $container, $id, $config)
2928
{
3029
$definition = $container->setDefinition($id, new ChildDefinition('security.user.provider.in_memory'));
3130
$defaultPassword = new Parameter('container.build_id');
31+
$users = array();
3232

3333
foreach ($config['users'] as $username => $user) {
34-
$userId = $id.'_'.$username;
35-
36-
$container
37-
->setDefinition($userId, new ChildDefinition('security.user.provider.in_memory.user'))
38-
->setArguments(array($username, null !== $user['password'] ? (string) $user['password'] : $defaultPassword, $user['roles']))
39-
;
40-
41-
$definition->addMethodCall('createUser', array(new Reference($userId)));
34+
$users[$username] = array('password' => (string) $user['password'], 'roles' => $user['roles']);
4235
}
36+
37+
$definition->addArgument($users);
4338
}
4439

4540
public function getKey()

src/Symfony/Bundle/SecurityBundle/Tests/DependencyInjection/CompleteConfigurationTest.php

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -58,12 +58,8 @@ public function testUserProviders()
5858

5959
$expectedProviders = array(
6060
'security.user.provider.concrete.default',
61-
'security.user.provider.concrete.default_foo',
6261
'security.user.provider.concrete.digest',
63-
'security.user.provider.concrete.digest_foo',
6462
'security.user.provider.concrete.basic',
65-
'security.user.provider.concrete.basic_foo',
66-
'security.user.provider.concrete.basic_bar',
6763
'security.user.provider.concrete.service',
6864
'security.user.provider.concrete.chain',
6965
);

0 commit comments

Comments
 (0)