Commitd52b6c9

committed

[SecurityBundle] Don't register in memory users as services

1 parent40ced3a commitd52b6c9Copy full SHA for d52b6c9

File tree

-14

lines changed

-14

lines changed

Lines changed: 4 additions & 9 deletions

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,6 @@`
`14`	`14`	`useSymfony\Component\Config\Definition\Builder\NodeDefinition;`
`15`	`15`	`useSymfony\Component\DependencyInjection\DefinitionDecorator;`
`16`	`16`	`useSymfony\Component\DependencyInjection\ContainerBuilder;`
`17`		`-useSymfony\Component\DependencyInjection\Reference;`
`18`	`17`
`19`	`18`	`/**`
`20`	`19`	`* InMemoryFactory creates services for the memory provider.`
`@@ -27,17 +26,13 @@ class InMemoryFactory implements UserProviderFactoryInterface`
`27`	`26`	`publicfunctioncreate(ContainerBuilder$container,$id,$config)`
`28`	`27`	`{`
`29`	`28`	`$definition =$container->setDefinition($id,newDefinitionDecorator('security.user.provider.in_memory'));`
	`29`	`+$users =array();`
`30`	`30`
`31`	`31`	`foreach ($config['users']as$username =>$user) {`
`32`		`-$userId =$id.'_'.$username;`
`33`		`-`
`34`		`-$container`
`35`		`- ->setDefinition($userId,newDefinitionDecorator('security.user.provider.in_memory.user'))`
`36`		`- ->setArguments(array($username, (string)$user['password'],$user['roles']))`
`37`		`- ;`
`38`		`-`
`39`		`-$definition->addMethodCall('createUser',array(newReference($userId)));`
	`32`	`+$users[$username] =array('password' => (string)$user['password'],'roles' =>$user['roles']);`
`40`	`33`	`}`
	`34`	`+`
	`35`	`+$definition->addArgument($users);`
`41`	`36`	`}`
`42`	`37`
`43`	`38`	`publicfunctiongetKey()`

Lines changed: 0 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -160,7 +160,6 @@`
`160`	`160`
`161`	`161`	`<!-- Provisioning-->`
`162`	`162`	`<serviceid="security.user.provider.in_memory"class="%security.user.provider.in_memory.class%"abstract="true"public="false" />`
`163`		`- <serviceid="security.user.provider.in_memory.user"class="%security.user.provider.in_memory.user.class%"abstract="true"public="false" />`
`164`	`163`
`165`	`164`	`<serviceid="security.user.provider.chain"class="%security.user.provider.chain.class%"abstract="true"public="false" />`
`166`	`165`

Lines changed: 0 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -44,12 +44,8 @@ public function testUserProviders()`
`44`	`44`
`45`	`45`	`$expectedProviders =array(`
`46`	`46`	`'security.user.provider.concrete.default',`
`47`		`-'security.user.provider.concrete.default_foo',`
`48`	`47`	`'security.user.provider.concrete.digest',`
`49`		`-'security.user.provider.concrete.digest_foo',`
`50`	`48`	`'security.user.provider.concrete.basic',`
`51`		`-'security.user.provider.concrete.basic_foo',`
`52`		`-'security.user.provider.concrete.basic_bar',`
`53`	`49`	`'security.user.provider.concrete.service',`
`54`	`50`	`'security.user.provider.concrete.chain',`
`55`	`51`	`);`

Comments

(0)