Problem/Motivation
Bump Symfony versions for https://symfony.com/blog/cve-2026-48736-iputils-private-subnets-omits-ip... and https://symfony.com/blog/cve-2026-48784-urlgenerator-encoding-skips-ever...
Steps to reproduce
Proposed resolution
Remaining tasks
User interface changes
Introduced terminology
API changes
Data model changes
Release notes snippet
Issue fork drupal-3592421
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 3592421-update-symfony-versions-10.5.x
changes, plain diff MR !15905
- 3592421-update-symfony-versions-10.6.x
changes, plain diff MR !15906
- 3592421-update-symfony-versions-11.2.x
changes, plain diff MR !15903
- 3592421-update-symfony-versions-11.3.x
changes, plain diff MR !15902
- 3592421-update-symfony-versions-11.4.x
changes, plain diff MR !15901
- 3592421-update-symfony-versions-11.x
changes, plain diff MR !15900
- 3592421-update-symfony-versions
changes, plain diff MR !15899
Comments
Comment #8
dwwLooks like a sea of green MRs. Will try to review once I get to my desk.
Comment #9
godotislateI have MRs up for main, 11.x, 11.4.x, 11.3.x, 11.2.x, and 10.5.x.
10.6.x is on hold for the commit from #3592382: Update Twig to v3.27.0.
Comment #10
znerol commentedNote, updating symfony is not enough to fix CVE-2026-48784, Drupal has its own
UrlGenerator#3592341: Fix dot-segment encoding for chained "../" and "./" in generated URLs.Comment #11
dwwI tested locally on all of the following branches:
In each case, a fresh
git pull; composer install; composer auditresulted in:After checking out the corresponding branch from the fork repo, another
composer install; composer auditgives:2 things prevent me from RTBC'ing:
Copy/paste failed myself. Everything is cool. The polyfills are gone fromcomposer/Metapackage/PinnedDevDependencies/composer.jsonis only updated in the MR for 10.5.x. However, on all the 11.* branches, that file is still pinning"symfony/polyfill-php84": "~v1.37.0",but we bump that version to"v1.38.1"in 10.5.x. Presumably we need that everywhere.PinnedDevDependenciesin the 11.* branches.Comment #13
dwwArgh, I copy/paste/failed myself. Edited previous comment to clarify. Only awaiting 10.6.x MR, then this is RTBC.
Comment #14
godotislate10.6.x MR up https://git.drupalcode.org/project/drupal/-/merge_requests/15906
Comment #15
dwwDid the same test on
10.6.x. Same 2 vulnerabilities before. After checking out3592421-update-symfony-versions-10.6.xand runningcomposer install,composer auditis happy. MR diff looks good. Given the bot is happy everywhere else, and the changes are functionally equivalent, the only way I see the pipeline failing is a random fail. Being bold, and moving to RTBC.Thanks!
-Derek
Comment #16
dwwBumping to major (at least) and tagging for Security since the change is to resolve CVEs.
Comment #17
znerol commentedIt looks like the issue from last week #3591179: Update main and 11.x to latest Twig and Symfony versions changed
core/composer.jsonand various othercomposer.jsonfiles in addition to the one incomposer/Metapackage/CoreRecommended. I used this command to compile a CSV containing a list of symfony dependencies defined in variouscomposer.jsonfiles.Not super efficient, but it does what it should. Using the spreadsheet I can see that
symfony/http-kernelis at a different patch level incomposer/Metapackage/CoreRecommended/composer.jsonvs.core/composer.json:Comment #18
longwave@znerol that's fine. We only bump the minimum dependency when there is a security release. So 8.0.12 is still secure, but we have locked to 8.0.13 in the lockfile so ~8.0.13 is in core recommended, because we didn't test with 8.0.12 explicitly.
Comment #19
godotislateSorry about this, I ran the wrong composer commands originally.
Documenting for posterity:
I did
COMPOSER_ROOT_VERSION={branch} composer update symfony/*Spoke to longwave and correct commands are
MRs updated.
Comment #20
godotislateComment #21
dwwRe-reviewed and retested everywhere. Smaller diff now that we're only updating those 2 symfony packages, not every Symfony thing with a new release. All looks good to me. Ship it. 😅
Thanks!
-Derek
Comment #22
znerol commentedThanks @godotislate, tested all MRs and looked at the diffs/stats. I think its all okay now.
Comment #37
catchCommitted/pushed to all branches, thanks!
The 11.x MR cherry-picked to 11.4.x so I did that one as a cherry-pick.