Repository navigation
Require Symfony ExpressionLanguage 8.1.9 or newer - #16
Merged
brianvarskonst merged 1 commit intoOct 15, 2026
Merged
brianvarskonst merged 1 commit into
brianvarskonst merged 1 commit into
Conversation
brianvarskonst
approved these changes
Oct 15, 2026
brianvarskonst
left a comment
Member
There was a problem hiding this comment.
Reviewed exact head bdf149a. No blocking findings; independent review agrees.
The ^8.1.9 security floor matches Symfony's complete fix for CVE-2026-109449. Manifest, installation example, and changelog are synchronized; runtime code and public APIs are unchanged.
Validation:
- All three required GitHub checks passed: workflows / actionlint, qa / Discover Composer targets, qa / QA (.).
- Fresh isolated PHP 8.5.9 validation: composer validate --strict, dependency resolution selecting ExpressionLanguage 8.1.9, composer audit --locked, and composer qa passed. Local PHPUnit: 79 tests / 375 assertions, seven notices, one skipped test. Hosted QA: 79 tests / 391 assertions, seven notices, no skipped test.
- Normal expression evaluation, unary nesting rejection, binary tree-depth rejection, compiled service-container expressions, and dumped runtime-container expressions passed additional smoke checks.
- Independent theme-starter canary resolved the 93-package graph using this exact Kernel metadata, upgraded ExpressionLanguage 8.1.6 to 8.1.9, retained the other private package metadata, and passed composer audit --locked. Dependency resolution only; no WordPress boot or deployment.
Consumers must update their lockfiles after a release containing this change.
Advisory: https://symfony.com/blog/cve-2026-109449-unbounded-nesting-in-the-expression-parser-lets-untrusted-expressions-crash-the-process
CI: https://github.com/SymPress/kernel/actions/runs/38771718447
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The current
symfony/expression-language: ^8.0requirement permits Symfony 8 releases affected by CVE-2026-109449. Symfony identifies 8.1.9 as the complete fix; the earlier 8.1.6 fix missed some expression-tree shapes.Require
^8.1.9and synchronize the installation example and changelog. The existingcontainer.expression_languageservice and supported service-expression configuration remain available. Other Symfony requirements are unchanged.Validation
composer validate --strictandcomposer audit --locked: passed with ExpressionLanguage 8.1.9.composer qa: passed code style, static analysis and 79 tests / 375 assertions. PHPUnit reports seven notices and one skipped test. Ran on a temporary Linux-filesystem copy because the DATA mount does not provide the private directory permissions required by cache tests.SyntaxError. Independent review also verified a compiled/dumped service-container parameter expression.This addresses the dependency version policy; it does not claim that this Kernel exposes untrusted expressions. Consumers still need to update their lockfiles after a release containing this change.