More bullet-proof expression evaluation

Wirone · Wirone · commit f22ebf917eda · 2025-03-14T15:21:13.000+01:00
In some scenarios evaluating expression can lead to TypeError (unresolved DI params being strings, not expected integers etc).
diff --git a/src/Symfony/Component/DependencyInjection/Compiler/CheckTypeDeclarationsPass.php b/src/Symfony/Component/DependencyInjection/Compiler/CheckTypeDeclarationsPass.php
@@ -228,7 +228,7 @@ private function checkType(Definition $checkedDefinition, mixed $value, \Reflect
         } elseif ($value instanceof Expression) {
             try {
                 $value = $this->getExpressionLanguage()->evaluate($value, ['container' => $this->container]);
-            } catch (\Exception) {
+            } catch (\Throwable $e) {
                 // If a service from the expression cannot be fetched from the container, we skip the validation.
                 return;
             }
@@ -243,7 +243,7 @@ private function checkType(Definition $checkedDefinition, mixed $value, \Reflect
                 if ('' === preg_replace('/'.$envPlaceholderUniquePrefix.'_\w+_[a-f0-9]{32}/U', '', $value, -1, $c) && 1 === $c) {
                     try {
                         $value = $this->container->resolveEnvPlaceholders($value, true);
-                    } catch (\Exception) {
+                    } catch (\Throwable) {
                         // If an env placeholder cannot be resolved, we skip the validation.
                         return;
                     }
