Skip to content

Commit b402d7d

Browse files
nicolas-grekasnicolas-grekas
committed
[Runtime] a new component to decouple applications from global state
1 parent fc016dd commit b402d7d

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

42 files changed

+1423
-1
lines changed

‎.gitattributes

+1
Original file line numberDiff line numberDiff line change
@@ -3,3 +3,4 @@
33
/src/Symfony/Component/Mailer/Bridge export-ignore
44
/src/Symfony/Component/Messenger/Bridge export-ignore
55
/src/Symfony/Component/Notifier/Bridge export-ignore
6+
/src/Symfony/Component/Runtime export-ignore

‎.github/patch-types.php

+1
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@
3030
case false !== strpos($file, '/src/Symfony/Component/ErrorHandler/Tests/Fixtures/'):
3131
case false !== strpos($file, '/src/Symfony/Component/PropertyInfo/Tests/Fixtures/Dummy.php'):
3232
case false !== strpos($file, '/src/Symfony/Component/PropertyInfo/Tests/Fixtures/ParentDummy.php'):
33+
case false !== strpos($file, '/src/Symfony/Component/Runtime/Internal/ComposerPlugin.php'):
3334
case false !== strpos($file, '/src/Symfony/Component/Serializer/Tests/Normalizer/Features/ObjectOuter.php'):
3435
case false !== strpos($file, '/src/Symfony/Component/VarDumper/Tests/Fixtures/NotLoadableClass.php'):
3536
continue 2;

‎composer.json

+6-1
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,8 @@
5252
"symfony/polyfill-mbstring": "~1.0",
5353
"symfony/polyfill-php73": "^1.11",
5454
"symfony/polyfill-php80": "^1.15",
55-
"symfony/polyfill-uuid": "^1.15"
55+
"symfony/polyfill-uuid": "^1.15",
56+
"symfony/runtime": "self.version"
5657
},
5758
"replace": {
5859
"symfony/asset": "self.version",
@@ -193,6 +194,10 @@
193194
"symfony/contracts": "2.4.x-dev"
194195
}
195196
}
197+
},
198+
{
199+
"type": "path",
200+
"url": "src/Symfony/Component/Runtime"
196201
}
197202
],
198203
"minimum-stability": "dev"

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

+2
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@
3838
use Symfony\Component\HttpKernel\KernelEvents;
3939
use Symfony\Component\HttpKernel\KernelInterface;
4040
use Symfony\Component\HttpKernel\UriSigner;
41+
use Symfony\Component\Runtime\SymfonyRuntime;
4142
use Symfony\Component\String\LazyString;
4243
use Symfony\Component\String\Slugger\AsciiSlugger;
4344
use Symfony\Component\String\Slugger\SluggerInterface;
@@ -78,6 +79,7 @@ class_exists(WorkflowEvents::class) ? WorkflowEvents::ALIASES : []
7879
service('argument_resolver'),
7980
])
8081
->tag('container.hot_path')
82+
->tag('container.preload', ['class' => SymfonyRuntime::class])
8183
->alias(HttpKernelInterface::class, 'http_kernel')
8284

8385
->set('request_stack', RequestStack::class)

‎src/Symfony/Component/Runtime/.gitattributes

+4
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
/Tests export-ignore
2+
/phpunit.xml.dist export-ignore
3+
/.gitattributes export-ignore
4+
/.gitignore export-ignore

‎src/Symfony/Component/Runtime/.gitignore

+3
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
vendor/
2+
composer.lock
3+
phpunit.xml

‎src/Symfony/Component/Runtime/CHANGELOG.md

+7
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
CHANGELOG
2+
=========
3+
4+
5.3.0
5+
-----
6+
7+
* Add the component

‎src/Symfony/Component/Runtime/GenericRuntime.php

+151
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,151 @@
1+
<?php
2+
3+
/*
4+
* This file is part of the Symfony package.
5+
*
6+
* (c) Fabien Potencier <fabien@symfony.com>
7+
*
8+
* For the full copyright and license information, please view the LICENSE
9+
* file that was distributed with this source code.
10+
*/
11+
12+
namespace Symfony\Component\Runtime;
13+
14+
use Symfony\Component\Runtime\Internal\BasicErrorHandler;
15+
use Symfony\Component\Runtime\Resolver\ClosureResolver;
16+
use Symfony\Component\Runtime\Resolver\DebugClosureResolver;
17+
use Symfony\Component\Runtime\Runner\ClosureRunner;
18+
19+
// Help opcache.preload discover always-needed symbols
20+
class_exists(ClosureResolver::class);
21+
22+
/**
23+
* A runtime to do bare-metal PHP without using superglobals.
24+
*
25+
* One option named "debug" is supported; it toggles displaying errors.
26+
*
27+
* The app-callable can declare arguments among either:
28+
* - "array $context" to get a local array similar to $_SERVER;
29+
* - "array $argv" to get the command line arguments when running on the CLI;
30+
* - "array $request" to get a local array with keys "query", "body", "files" and
31+
* "session", which map to $_GET, $_POST, $FILES and &$_SESSION respectively.
32+
*
33+
* It should return a Closure():int|string|null or an instance of RunnerInterface.
34+
*
35+
* In debug mode, the runtime registers a strict error handler
36+
* that throws exceptions when a PHP warning/notice is raised.
37+
*
38+
* @author Nicolas Grekas <p@tchwork.com>
39+
*
40+
* @experimental in 5.3
41+
*/
42+
class GenericRuntime implements RuntimeInterface
43+
{
44+
private $debug;
45+
46+
public function __construct(array $options = [])
47+
{
48+
if ($this->debug = $options['debug'] ?? true) {
49+
$errorHandler = new BasicErrorHandler($this->debug);
50+
set_error_handler($errorHandler);
51+
}
52+
}
53+
54+
/**
55+
* {@inheritdoc}
56+
*/
57+
public function getResolver(callable $callable): ResolverInterface
58+
{
59+
if (!$callable instanceof \Closure) {
60+
$callable = \Closure::fromCallable($callable);
61+
}
62+
63+
$function = new \ReflectionFunction($callable);
64+
$parameters = $function->getParameters();
65+
66+
$arguments = function () use ($parameters) {
67+
$arguments = [];
68+
69+
try {
70+
foreach ($parameters as $parameter) {
71+
$type = $parameter->getType();
72+
$arguments[] = $this->getArgument($parameter, $type instanceof \ReflectionNamedType ? $type->getName() : null);
73+
}
74+
} catch (\InvalidArgumentException $e) {
75+
if (!$parameter->isOptional()) {
76+
throw $e;
77+
}
78+
}
79+
80+
return $arguments;
81+
};
82+
83+
if ($this->debug) {
84+
return new DebugClosureResolver($callable, $arguments);
85+
}
86+
87+
return new ClosureResolver($callable, $arguments);
88+
}
89+
90+
/**
91+
* {@inheritdoc}
92+
*/
93+
public function getRunner(object $application): RunnerInterface
94+
{
95+
if ($application instanceof RunnerInterface) {
96+
return $application;
97+
}
98+
99+
if (!\is_callable($application)) {
100+
throw new \LogicException(sprintf('"%s" doesn\'t know how to handle apps of type "%s".', get_debug_type($this), get_debug_type($application)));
101+
}
102+
103+
if (!$application instanceof \Closure) {
104+
$application = \Closure::fromCallable($application);
105+
}
106+
107+
if ($this->debug && ($r = new \ReflectionFunction($application)) && $r->getNumberOfRequiredParameters()) {
108+
throw new \ArgumentCountError(sprintf('Zero argument should be required by the runner callable, but at least one is in "%s" on line "%d.', $r->getFileName(), $r->getStartLine()));
109+
}
110+
111+
return new ClosureRunner($application);
112+
}
113+
114+
/**
115+
* @return mixed
116+
*/
117+
protected function getArgument(\ReflectionParameter $parameter, ?string $type)
118+
{
119+
if ('array' === $type) {
120+
switch ($parameter->name) {
121+
case 'context':
122+
$context = $_SERVER;
123+
124+
if ($_ENV && !isset($_SERVER['PATH']) && !isset($_SERVER['Path'])) {
125+
$context += $_ENV;
126+
}
127+
128+
return $context;
129+
130+
case 'argv':
131+
return $_SERVER['argv'] ?? [];
132+
133+
case 'request':
134+
return [
135+
'query' => $_GET,
136+
'body' => $_POST,
137+
'files' => $_FILES,
138+
'session' => &$_SESSION,
139+
];
140+
}
141+
}
142+
143+
if (RuntimeInterface::class === $type) {
144+
return $this;
145+
}
146+
147+
$r = $parameter->getDeclaringFunction();
148+
149+
throw new \InvalidArgumentException(sprintf('Cannot resolve argument "%s $%s" in "%s" on line "%d": "%s" supports only arguments "array $context", "array $argv" and "array $request".', $type, $parameter->name, $r->getFileName(), $r->getStartLine(), get_debug_type($this)));
150+
}
151+
}

‎src/Symfony/Component/Runtime/Internal/BasicErrorHandler.php

+53
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,53 @@
1+
<?php
2+
3+
/*
4+
* This file is part of the Symfony package.
5+
*
6+
* (c) Fabien Potencier <fabien@symfony.com>
7+
*
8+
* For the full copyright and license information, please view the LICENSE
9+
* file that was distributed with this source code.
10+
*/
11+
12+
namespace Symfony\Component\Runtime\Internal;
13+
14+
/**
15+
* @author Nicolas Grekas <p@tchwork.com>
16+
*
17+
* @internal
18+
*/
19+
class BasicErrorHandler
20+
{
21+
public function __construct(bool $debug)
22+
{
23+
error_reporting(-1);
24+
25+
if (!\in_array(\PHP_SAPI, ['cli', 'phpdbg'], true)) {
26+
ini_set('display_errors', $debug);
27+
} elseif (!filter_var(ini_get('log_errors'), \FILTER_VALIDATE_BOOLEAN) || ini_get('error_log')) {
28+
// CLI - display errors only if they're not already logged to STDERR
29+
ini_set('display_errors', 1);
30+
}
31+
32+
if (0 <= ini_get('zend.assertions')) {
33+
ini_set('zend.assertions', 1);
34+
ini_set('assert.active', $debug);
35+
ini_set('assert.bail', 0);
36+
ini_set('assert.warning', 0);
37+
ini_set('assert.exception', 1);
38+
}
39+
}
40+
41+
public function __invoke(int $type, string $message, string $file, int $line): bool
42+
{
43+
if ((\E_DEPRECATED | \E_USER_DEPRECATED) & $type) {
44+
return true;
45+
}
46+
47+
if ((error_reporting() | \E_ERROR | \E_RECOVERABLE_ERROR | \E_PARSE | \E_CORE_ERROR | \E_COMPILE_ERROR | \E_USER_ERROR) & $type) {
48+
throw new \ErrorException($message, 0, $type, $file, $line);
49+
}
50+
51+
return false;
52+
}
53+
}

0 commit comments

Comments
 (0)