Repository navigation
fix(deps): symfony/http-foundation and symfony/yaml to 6.4.48, clearing two advisories - #3948
Merged
Merged
Conversation
…ng two advisories `quality / Security (composer)` is red on development. `composer audit` reports two advisories, both in packages-dev: - symfony/http-foundation v6.4.43, CVE-2026-108082: ReDoS in HeaderUtils::split(), reachable through the Cache-Control request header. Patched from 6.4.46. - symfony/yaml v6.4.40, CVE-2026-109413: unbounded recursion in the YAML inline lexer lets a single document crash the process. Patched from 6.4.45. It arrives through consolidation/robo (symfony/yaml ^6). Both move to v6.4.48, the newest 6.x, so nothing crosses a major boundary. composer.json is unchanged: the `^6.4.41` constraint on http-foundation already allowed a patched release, and yaml is a transitive requirement, so the lock alone was behind. The lock's content-hash is unchanged and only these two packages move. Assisted-by: ClaudeCode:claude-opus-5
rubenvdlinde
requested review from
WilcoLouwerse,
bbrands02 and
rjzondervan
as code owners
October 15, 2026 10:21
Contributor
Quality Report — ConductionNL/dossiq @
|
| Check | PHP | Vue | Security | License | Tests |
|---|---|---|---|---|---|
| lint | ✅ | ||||
| phpcs | ✅ | ||||
| phpmd | ✅ | ||||
| psalm | ✅ | ||||
| phpstan | ✅ | ||||
| phpmetrics | ✅ | ||||
| eslint | ✅ | ||||
| stylelint | ✅ | ||||
| build | ✅ | ||||
| check-manifest | ✅ | ||||
| check-vue3-compile | ✅ | ||||
| test-l10n | ✅ | ||||
| format | ✅ | ||||
| check-schema-l10n | ✅ | ||||
| check-l10n-js | ✅ | ||||
| check-debt-ratchet | ❌ | ||||
| check-types-e2e | ✅ | ||||
| composer | ✅ | ✅ 109/109 | |||
| npm | ✅ | ✅ 659/659 | |||
| app:check-code | ⏭️ | ||||
| info.xml | ✅ | ||||
| REUSE | ❌ | ||||
| lockfile sync | ✅ | ||||
| PHPUnit | ❌ 8657/8657 · 3 skipped (no test failed, the job failed for another reason — see its log) | ||||
| Newman | ⏭️ | ||||
| Playwright | ⏭️ deferred: E2E runs locally and on the promotion path only. This pull request targets development, so the suite is asked once per promotion into beta and main rather than once per push per open pull request. Run it locally with npx playwright test, or from the Actions tab on a branch with no open pull request into development. |
||||
| Hydra gates | ❌ |
Quality workflow — 2026-10-15 10:52 UTC
Download the full PDF report from the workflow artifacts.
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.
What is wrong
quality / Security (composer)is red ondevelopment.composer audit --format=plain --abandoned=report, the exact call the shared workflow makes, exits 1 with two advisories, both inpackages-dev:HeaderUtils::split()reachable through theCache-Controlrequest headersymfony/http-foundationis a directrequire-dev(^6.4.41).symfony/yamlis transitive:consolidation/robo4.0.6 requiressymfony/yaml ^6.What changes
Both move to v6.4.48, the newest 6.x release. Nothing crosses a major boundary.
composer.jsondoes not change. The^6.4.41constraint already allowed a patched release andsymfony/yamlhas no constraint here at all, so the lock alone was behind.composer update symfony/http-foundation symfony/yamlis enough:--with-dependenciesadditionally pullssymfony/polyfill-mbstringandsymfony/polyfill-php83forward, which nothing needs, so the narrower call is the one taken. The lock'scontent-hashis unchanged and the diff is 12 lines, all of them these two packages.Both packages are development-only. Nextcloud supplies
symfony/http-foundationto the app at runtime, andsymfony/yamlis used by one test (ZgwOpenApiConformanceTest) and by robo. So the exposure this clears is the local and CI toolchain, not a shipped install.Verification
Commands and exit codes, run with an isolated
HOMEandTMPDIRso the shared~/.pdependcache cannot answer for another clone.composer auditbefore: exit 1, "Found 2 security vulnerability advisories affecting 2 packages".composer audit --format=plain --abandoned=reportafter: exit 0, "No security vulnerability advisories found". The feed was reachable on both runs, which the before-run's two findings prove.composer installafter: exit 0, reproducible from the lock.composer validate: exit 0.vendor/bin/phpunit --no-coverage: exit 0, 8657 tests, 50850 assertions, 82 skipped, 1 warning. Same counts as before the bump.COMPOSER_PROCESS_TIMEOUT=0 composer check:strict: see the comment below for both runs, this branch against a cleanorigin/developmentin the same clone.No test accompanies this change: a lock bump has no behaviour to pin, and
composer auditis itself the check that was failing.Inherited
developmentis red on phpmd,check:debt-ratchetand several Hydra gates; each is owned by another lane in the same sweep, and none is touched here.Fleet
The constraint is copied across the fleet, and so is the problem. Locked versions in the local checkouts, read-only, not changed:
symfony/yamlsits below the patched 6.4.45 in opencatalogi, thematiq, launchpad, stackiq, larpinq, pipelinq, learniq, portaliq, decidiq, buildiq, planninq, hermiq and dossiq, andsymfony/http-foundationsits at v6.4.43 in pipelinq, shillinq, buildiq, hermiq and dossiq, with opencatalogi and integriq at v7.4.13 against a 7.4.19 fix. Those checkouts may trail their owndevelopment, so each app needs its owncomposer auditbefore anyone counts it affected.