Skip to content

fix: bump phpunit/phpunit to 9.6.33 (Snyk, dev-only) - #15

Closed
pritesh-jain-cstk wants to merge 2 commits into
fix/ci-codeql-v3-license-2026from
fix/snyk-phpunit-9.6.33
Closed

pritesh-jain-cstk wants to merge 2 commits into
fix/ci-codeql-v3-license-2026from
fix/snyk-phpunit-9.6.33

Conversation

@pritesh-jain-cstk

@pritesh-jain-cstk pritesh-jain-cstk commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Finding

SNYK-PHP-PHPUNITPHPUNIT-15123584: phpunit/phpunit 9.5.20, high, Deserialization of Untrusted Data. Fixed in 9.6.33 (same 9.x major).

Change

image-preset-builder/php/composer.lock only (composer update phpunit/phpunit:9.6.33 -w -m, resolved against a temporary platform.php 7.3.0 pin that is not committed). composer.json is unchanged (^9.5 already allows 9.6.33).

  • phpunit/phpunit 9.5.20 -> 9.6.33
  • Its own dependency tree moved with it: php-code-coverage 9.2.32, sebastian/* patch/minor bumps, doctrine/instantiator 1.4.1 -> 1.5.0, nikic/php-parser 4.13.2 -> 4.19.5, myclabs/deep-copy 1.11.0 -> 1.13.4, phar-io/manifest 2.0.4, theseer/tokenizer 1.3.1. phpunit 9.6.33 requires newer versions of several of these.
  • Removed from the lock because phpunit 9.6 no longer needs them: phpspec/prophecy, phpdocumentor/*, webmozart/assert, symfony/polyfill-ctype.
  • No production dependency changed (jakeasmith/http_build_url untouched).
  • Minimum PHP for installing the dev toolchain from this lock is unchanged (7.3), verified.

Verification

  • Snyk (dev, composer.lock): before 1 issue / 43 vulnerable paths; after 0.
  • Prod scan 0, Snyk Code 0 (from baseline; no source changed).
  • composer test: OK (16 tests, 16 assertions), run on PHP 8.5.12 only (7.3 not available locally). Existing "implicitly nullable" deprecation notices in src/ImageTransform.php are unrelated and untouched.

Notes

  • Dev-only dependency. This repo is public, so CAB is needed to merge.
  • Open Dependabot branch for phpunit 12.5.22 exists and is not touched by this PR.

🤖 Generated with Claude Code

…23584)

Dev-only dependency in image-preset-builder/php. composer.json unchanged.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@pritesh-jain-cstk
pritesh-jain-cstk requested a review from a team as a code owner October 30, 2026 11:41
@snyk-io

snyk-io Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues
✅ Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@github-actions

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

@cs-raj cs-raj closed this Oct 30, 2026
@cs-raj cs-raj reopened this Oct 30, 2026
@github-actions

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

cs-raj
cs-raj previously approved these changes Oct 30, 2026
…loor

Previous lock was resolved on PHP 8.5 and pulled doctrine/instantiator 2.1.0
(php ^8.4). Re-resolved from main's lock with a temporary platform pin
(not committed); composer.json unchanged.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

@pritesh-jain-cstk
pritesh-jain-cstk changed the base branch from main to fix/ci-codeql-v3-license-2026 October 30, 2026 19:45
@pritesh-jain-cstk
pritesh-jain-cstk removed this pull request from stack #17 October 30, 2026 19:47
@pritesh-jain-cstk

Copy link
Copy Markdown
Contributor Author

Superseded by #16, which includes these phpunit commits plus the CI fixes.

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.

3 participants