Skip to content

chore: resolve open dependabot security alerts#139

Merged
JamieSinn merged 4 commits into
mainfrom
chore/dependabot-alerts
Jul 23, 2026
Merged

chore: resolve open dependabot security alerts#139
JamieSinn merged 4 commits into
mainfrom
chore/dependabot-alerts

Conversation

@jonathannorris

Copy link
Copy Markdown
Member

Summary

Resolved 20 open Dependabot security alerts by bumping vulnerable dependencies across the SDK and example apps.

Dependabot Alerts Resolved

Alert Package Severity Fix
#37, #34, #33, #32, #31, #28 guzzlehttp/guzzle medium Bumped to 7.13.1 in main + all example lockfiles
#36, #29 guzzlehttp/guzzle medium Bumped to 7.13.1 (HTTPS-Proxy downgrade)
#35, #30, #27, #26 guzzlehttp/psr7 medium Bumped to 2.12.3 (CRLF injection in HTTP Start-Line)
#25, #23, #21, #20 guzzlehttp/psr7 medium Bumped to 2.12.3 (CRLF injection via URI Host)
#24, #22, #19, #18 guzzlehttp/psr7 medium Bumped to 2.12.3 (Host confusion via Authority reinterpretation)

Also bumped phpunit/phpunit 10.5.45 to 10.5.63 to resolve CVE-2026-24765 (unsafe deserialization in PHPT code coverage handling, high severity).

All 22 tests pass locally.

- guzzlehttp/guzzle 7.9.2 -> 7.13.1 (medium, alerts #28, #29, #31, #32, #33, #34, #36, #37)
- guzzlehttp/psr7 2.7.0 -> 2.12.3 (medium, alerts #18, #19, #20, #21, #22, #23, #24, #25, #26, #27, #30, #35)
- phpunit/phpunit 10.5.45 -> 10.5.63 (high, CVE-2026-24765)
Copilot AI review requested due to automatic review settings July 2, 2026 13:39
@jonathannorris
jonathannorris requested a review from a team as a code owner July 2, 2026 13:39
@jonathannorris
jonathannorris enabled auto-merge (squash) July 2, 2026 13:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates Composer dependencies in the PHP Server SDK and example apps to address open Dependabot security alerts by bumping vulnerable packages (notably guzzlehttp/guzzle, guzzlehttp/psr7, and phpunit/phpunit) and regenerating lockfiles.

Changes:

  • Bumped guzzlehttp/guzzle to 7.13.1 and guzzlehttp/psr7 to 2.12.3 in lockfiles.
  • Bumped phpunit/phpunit (and related dev dependencies) in the root lockfile.
  • Updated example app lockfiles to align with the patched dependency set.

Reviewed changes

Copilot reviewed 2 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
composer.json Updates dependency constraints for Guzzle/Psr7 and PHPUnit.
composer.lock Pins updated dependency graph (including Guzzle/Psr7 and PHPUnit transitive updates).
examples/hello-world/composer.lock Updates example lockfile dependency graph to patched versions.
examples/hello-world-http-proxy/composer.lock Updates example lockfile dependency graph to patched versions.
examples/hello-world-udp-proxy/composer.lock Updates example lockfile dependency graph to patched versions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread composer.json Outdated
@jonathannorris
jonathannorris requested a review from JamieSinn July 20, 2026 19:53
Comment thread .github/workflows/test.yml Outdated
- guzzlehttp/guzzle ^7.12.1 -> ^7.15.1 (medium severity, alerts #37-57)
- guzzlehttp/psr7 also updated to 2.13.0 as part of dependency resolution
- Updated lockfiles for root and all three example apps
- Bumped test.yml php_version from 8.3 to 8.4 (PHP 8.3 active support ended)
@JamieSinn

Copy link
Copy Markdown
Member

Test harness will likely need a version bump too for PHP

@jonathannorris

Copy link
Copy Markdown
Member Author

The test harness Dockerfile at proxies/php/Dockerfile currently installs php81 via Alpine packages. The harness CI runs have been failing consistently since at least August 2025 with a Jest ESM configuration error (SyntaxError: Cannot use import statement outside a module), which is unrelated to the PHP version. I'll file a separate issue in the test-harness repo to track both the Jest config fix and the PHP version bump.

@JamieSinn
JamieSinn disabled auto-merge July 23, 2026 14:46
@JamieSinn
JamieSinn merged commit 14dd961 into main Jul 23, 2026
4 of 5 checks passed
@JamieSinn
JamieSinn deleted the chore/dependabot-alerts branch July 23, 2026 14:46
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