Skip to content

fix(deps): php_codesniffer 3.13.6 — CVE-2026-67434 OS command injection - #237

Merged
rubenvdlinde merged 1 commit into
developmentfrom
chore/phpcs-cve-2026-67434
Aug 6, 2026
Merged

fix(deps): php_codesniffer 3.13.6 — CVE-2026-67434 OS command injection#237
rubenvdlinde merged 1 commit into
developmentfrom
chore/phpcs-cve-2026-67434

Conversation

@rubenvdlinde

Copy link
Copy Markdown
Contributor

New advisory, fleet-wide

squizlabs/php_codesniffer — CVE-2026-67434 / GHSA-hmqg-cxww-wqhq — OS command injection. Reported 2026-08-05. Affected: <3.13.6 | >=4.0.0,<4.0.2.

All 16 Conduction repos checked were on 3.13.5. This is currently failing the Security (composer) gate on every one of them.

The advisory landed mid-session: composer audit --locked returned zero advisories for this repo about an hour before it returned this one. The audit DB is live, so a lockfile that was clean this morning is not clean now.

Change

composer.lock only — squizlabs/php_codesniffer 3.13.5 → 3.13.6. The composer.json constraint already allowed it.

Verification

  • composer audit --locked"No security vulnerability advisories found" (was 1 advisory).
  • vendor/bin/phpcs --version3.13.6.
  • composer phpcsrc=0.
  • Positive control: a deliberately non-conforming file under lib/ made phpcs exit 2 with 13 findings, then was deleted. A green check that cannot fail is not evidence. git status confirmed clean afterwards.

OS command injection in PHP_CodeSniffer, GHSA-hmqg-cxww-wqhq, reported
2026-08-05. Affected: <3.13.6 | >=4.0.0,<4.0.2. This repo was on 3.13.5.

All 16 repos checked across the fleet are on 3.13.5 and equally affected. The
advisory is live in the audit DB, so this repo's Security (composer) gate is
failing until this lands.

Verified
- composer audit --locked: 'No security vulnerability advisories found' (was 1).
- vendor/bin/phpcs --version -> 3.13.6.
- composer phpcs: rc=0.
- Positive control: a deliberately non-conforming file under lib/ made phpcs
  exit 2 with 13 findings, so the green above is a real pass and not a checker
  that no-ops. Probe removed; only composer.lock is modified.
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/nldesign @ 96bee77

Check PHP Vue Security License Tests
lint
phpcs
phpmd
psalm
phpstan
phpmetrics
eslint
stylelint
build
check-manifest
test-l10n
composer ✅ 100/100
npm ✅ 2/2
PHPUnit
Newman
Playwright
Hydra gates

Quality workflow — 2026-08-06 12:10 UTC

Download the full PDF report from the workflow artifacts.

@rubenvdlinde
rubenvdlinde merged commit 0ccad44 into development Aug 6, 2026
29 of 32 checks passed
@rubenvdlinde
rubenvdlinde deleted the chore/phpcs-cve-2026-67434 branch August 6, 2026 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant