8000 Deprecat service "session" · symfony/symfony@dd1ce2c · GitHub
[go: up one dir, main page]
Skip to content

Commit dd1ce2c

Browse files
jderussejderusse
committed
Deprecat service "session"
1 parent fdf9a43 commit dd1ce2c

File tree

14 files changed

+238
-45
lines changed

14 files changed

+238
-45
lines changed

‎src/Symfony/Bundle/FrameworkBundle/Controller/AbstractController.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -199,11 +199,11 @@ protected function file($file, string $fileName = null, string $disposition = Re
199199
*/
200200
protected function addFlash(string $type, $message): void
201201
{
202-
if (!$this->container->has('session')) {
202+
if (null === $session = $this->container->get('request_stack')->getSession()) {
203203
throw new \LogicException('You can not use the addFlash method if sessions are disabled. Enable them in "config/packages/framework.yaml".');
204204
}
205205

206-
$this->container->get('session')->getFlashBag()->add($type, $message);
206+
$session->getFlashBag()->add($type, $message);
207207
}
208208

209209
/**

‎src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/SessionPass.php‎

Lines changed: 52 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -22,29 +22,66 @@ class SessionPass implements CompilerPassInterface
2222
{
2323
public function process(ContainerBuilder $container)
2424
{
25-
if (!$container->hasDefinition('session')) {
25+
if (!$container->has('session.storage')) {
2626
return;
2727
}
2828

29-
$bags = [
30-
'session.flash_bag' => $container->hasDefinition('session.flash_bag') ? $container->getDefinition('session.flash_bag') : null,
31-
'session.attribute_bag' => $container->hasDefinition('session.attribute_bag') ? $container->getDefinition('session.attribute_bag') : null,
32-
];
29+
// if the session is not override, use SessionFactoryListener instead of SessionListener
30+
if (!$this->isOverridden($container) && !$this->isInjected($container)) {
31+
$container->removeDefinition('session_listener');
32+
$container->getDefinition( B271 'session')->setArgument(3, null);
3333

34-
foreach ($container->getDefinition('session')->getArguments() as $v) {
35-
if (!$v instanceof Reference || !isset($bags[$bag = (string) $v]) || !\is_array($factory = $bags[$bag]->getFactory())) {
36-
continue;
37-
}
34+
$locator = $container->getDefinition('session_factory_listener')->getArgument(0);
35+
$locator->setValues($locator->getValues() + [
36+
'session.flash_bag' => $container->hasDefinition('session.flash_bag') ? $container->getDefinition('session.flash_bag') : null,
37+
'session.attribute_bag' => $container->hasDefinition('session.attribute_bag') ? $container->getDefinition('session.attribute_bag') : null,
38+
]);
39+
} else {
40+
$container->removeDefinition('session_factory_listener');
41+
$definition = $container->getDefinition('session');
3842

39-
if ([0, 1] !== array_keys($factory) || !$factory[0] instanceof Reference || 'session' !== (string) $factory[0]) {
40-
continue;
41-
}
43+
$bags = [
44+
'session.flash_bag' => $container->hasDefinition('session.flash_bag') ? $container->getDefinition('session.flash_bag') : null,
45+
'session.attribute_bag' => $container->hasDefinition('session.attribute_bag') ? $container->getDefinition('session.attribute_bag') : null,
46+
];
47+
48+
foreach ($definition->getArguments() as $v) {
49+
if (!$v instanceof Reference || !isset($bags[$bag = (string) $v]) || !\is_array($factory = $bags[$bag]->getFactory())) {
50+
continue;
51+
}
4252

43-
if ('get'.ucfirst(substr($bag, 8, -4)).'Bag' !== $factory[1]) {
44-
continue;
53+
if ([0, 1] !== array_keys($factory) || !$factory[0] instanceof Reference || 'session' !== (string) $factory[0]) {
54+
continue;
55+
}
56+
57+
if ('get'.ucfirst(substr($bag, 8, -4)).'Bag' !== $factory[1]) {
58+
continue;
59+
}
60+
61+
$bags[$bag]->setFactory(null);
4562
}
63+
}
64+
}
65+
66+
private function isInjected(ContainerBuilder $container): bool
67+
{
68+
return false;
69+
}
4670

47-
$bags[$bag]->setFactory(null);
71+
private function isOverridden(ContainerBuilder $container): bool
72+
{
73+
$sessionDefinition = $container->getDefinition('session');
74+
if (!$sessionDefinition->isPrivate()) {
75+
return true;
4876
}
77+
78+
$args = $sessionDefinition->getArguments();
79+
// if the session is not override, use SessionFactoryListener instead of SessionListener
80+
return (
81+
$args[0] instanceof Reference && 'session.storage' === (string) $args[0]
82+
&& null === $args[1]
83+
&& null === $args[2]
84+
&& is_array($args[3]) && $args[3][1] === 'onSessionUsage' && $args[3][0] instanceof Reference && 'session_listener' === (string) $args[3][0]
85+
);
4986
}
5087
}

‎src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1005,6 +1005,11 @@ private function registerSessionConfiguration(array $config, ContainerBuilder $c
10051005
'session_storage' => new Reference('session.storage', ContainerInterface::IGNORE_ON_INVALID_REFERENCE),
10061006
'request_stack' => new Reference('request_stack'),
10071007
]);
1008+
1009+
$locator = $container->getDefinition('session_factory_listener')->getArgument(0);
1010+
$locator->setValues($locator->getValues() + [
1011+
'request_stack' => new Reference('request_stack'),
1012+
]);
10081013
}
10091014

10101015
$container->setParameter('session.storage.options', $options);

‎src/Symfony/Bundle/FrameworkBundle/FrameworkBundle.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ public function build(ContainerBuilder $container)
159159
$container->addCompilerPass(new RegisterReverseContainerPass(true));
160160
$container->addCompilerPass(new RegisterReverseContainerPass(false), PassConfig::TYPE_AFTER_REMOVING);
161161
$container->addCompilerPass(new RemoveUnusedSessionMarshallingHandlerPass());
162-
$container->addCompilerPass(new SessionPass());
162+
$container->addCompilerPass(new SessionPass(), PassConfig::TYPE_BEFORE_OPTIMIZATION, -1000);
163163

164164
if ($container->getParameter('kernel.debug')) {
165165
$container->addCompilerPass(new AddDebugLogProcessorPass(), PassConfig::TYPE_BEFORE_OPTIMIZATION, 2);

‎src/Symfony/Bundle/FrameworkBundle/KernelBrowser.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@ public function loginUser($user, string $firewallContext = 'main'): self
122122

123123
$token = new TestBrowserToken($user->getRoles(), $user);
124124
$token->setAuthenticated(true);
125-
$session = $this->getContainer()->get('session');
125+
$session = $this->getContainer()->get('request_stack')->getSession();
126126
$session->set('_security_'.$firewallContext, serialize($token));
127127
$session->save();
128128

‎src/Symfony/Bundle/FrameworkBundle/Resources/config/security_csrf.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
->alias(TokenGeneratorInterface::class, 'security.csrf.token_generator')
2828

2929
->set('security.csrf.token_storage', SessionTokenStorage::class)
30-
->args([service('session')])
30+
->args([service('request_stack')])
3131

3232
->alias(TokenStorageInterface::class, 'security.csrf.token_storage')
3333

‎src/Symfony/Bundle/FrameworkBundle/Resources/config/session.php‎

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
use Symfony\Component\HttpFoundation\Session\Storage\NativeSessionStorage;
2828
use Symfony\Component\HttpFoundation\Session\Storage\PhpBridgeSessionStorage;
2929
use Symfony\Component\HttpFoundation\Session\Storage\SessionStorageInterface;
30+
use Symfony\Component\HttpKernel\EventListener\FactorySessionListener;
3031
use Symfony\Component\HttpKernel\EventListener\SessionListener;
3132

3233
return static function (ContainerConfigurator $container) {
@@ -41,7 +42,9 @@
4142
null, // FlashBagInterface
4243
[service('session_listener'), 'onSessionUsage'],
4344
])
45+
->deprecate('symfony/framework-bundle', '5.3', 'The "%service_id%" service is deprecated, use "$requestStack->getSession()" instead.')
4446
->alias(SessionInterface::class, 'session')
47+
->deprecate('symfony/framework-bundle', '5.3', 'The "%alias_id%" alias is deprecated, use "$requestStack->getSession()" instead.')
4548
->alias(SessionStorageInterface::class, 'session.storage')
4649
->alias(\SessionHandlerInterface::class, 'session.handler')
4750

@@ -91,6 +94,18 @@
9194
->factory([SessionHandlerFactory::class, 'createHandler'])
9295
->args([abstract_arg('A string or a connection object')])
9396

97+
->set('session_factory_listener', FactorySessionListener::class)
98+
->args([
99+
service_locator([
100+
'session.storage' => service('session.storage')->ignoreOnInvalid(),
101+
'logger' => service('logger')->ignoreOnInvalid(),
102+
'session_collector' => service('data_collector.request.session_collector')->ignoreOnInvalid(),
103+
'session.flash_bag' => service('session.flash_bag')->ignoreOnInvalid(),
104+
'session.attribute_bag' => service('session.attribute_bag')->ignoreOnInvalid(),
105+
]),
106+
param('kernel.debug'),
107+
])
108+
->tag('kernel.event_subscriber')
94109
->set('session_listener', SessionListener::class)
95110
->args([
96111
service_locator([

‎src/Symfony/Bundle/SecurityBundle/DependencyInjection/Compiler/RegisterTokenUsageTrackingPass.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ public function process(ContainerBuilder $container)
4141
TokenStorageInterface::class => new BoundArgument(new Reference('security.untracked_token_storage'), false),
4242
]);
4343

44-
if (!$container->has('session')) {
44+
if (!$container->has('session.storage')) {
4545
$container->setAlias('security.token_storage', 'security.untracked_token_storage')->setPublic(true);
4646
$container->getDefinition('security.untracked_token_storage')->addTag('kernel.reset', ['method' => 'reset']);
4747
} elseif ($container->hasDefinition('security.context_listener')) {

‎src/Symfony/Bundle/SecurityBundle/Resources/config/security.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@
7474
->args([
7575
service('security.untracked_token_storage'),
7676
service_locator([
77-
'session' => service('session'),
77+
'request_stack' => service('request_stack'),
7878
]),
7979
])
8080
->tag('kernel.reset', ['method' => 'disableUsageTracking'])

‎src/Symfony/Component/HttpFoundation/RequestStack.php‎

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,8 @@
1111

1212
namespace Symfony\Component\HttpFoundation;
1313

14+
use Symfony\Component\HttpFoundation\Session\SessionInterface;
15+
1416
/**
1517
* Request stack that controls the lifecycle of requests.
1618
*
@@ -100,4 +102,18 @@ public function getParentRequest()
100102

101103
return $this->requests[$pos];
102104
}
105+
106+
/**
107+
* Gets the current session.
108+
*
109+
* @return SessionInterface|null
110+
*/
111+
public function getSession(): ?SessionInterface
112+
{
113+
if (null === $request = end($this->requests) ?: null) {
114+
return null;
115+
}
116+
117+
return $request->hasSession() ? $request->getSession() : null;
118+
}
103119
}

0 commit comments

Comments
 (0)
0