Skip to content

Pin patched PHP and Node test runtimes - #65

Merged
kvz merged 1 commit into
mainfrom
runtime-security
Oct 21, 2026
Merged

kvz merged 1 commit into
mainfrom
runtime-security

Conversation

@kvz

@kvz kvz commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

Node 20 CI tooling and the floating PHP/Node Docker inputs predate the October 21 security fixes. Pin CI to Node 22.23.4 and PHP 8.2.35, and the development image to PHP 8.3.36 plus the official Node 22.23.4 runtime. Copy Node/npm from its pinned image so apt repository lag cannot select an older runtime.

These are test/development runtimes. The published PHP >=7.4 support contract and the existing PHP 8.1 compatibility lane are preserved. Node 22 matches the Docker tooling's existing major line.

Validation:

  • All six CI matrix jobs pass, including locked/lowest/highest dependency sets on PHP 8.2.35 and the retained PHP 8.1 compatibility lane.
  • Existing unit suite with Node signature parity: 30 tests / 117 assertions pass (workstation PHP 8.5.10 and verified Node 26.12.1).
  • PHP CodeSniffer, YAML syntax and git diff checks pass.
  • The exact PHP Docker tag is now published. The complete amd64 image builds with --pull and reports PHP 8.3.36 and Node 22.23.4. PHP CodeSniffer and all 30 unit/parity tests / 117 assertions also pass inside this image using tracked source and dummy signing credentials. Native ARM image execution was not repeated locally.
  • Council review completed; its initial image-availability finding is resolved by the published tag and successful full build and in-image tests.

References: Node security release, PHP releases.

🤖 Claimed by kvz on framework1/api2-clone-5, framework1/php-sdk, framework1/clawfleet since 2026-10-21 · resume: codex resume 01a15e32-a05f-7153-89c2-0b1273795a49

@kvz kvz self-assigned this Oct 21, 2026
@github-actions

Copy link
Copy Markdown

Coverage report for commit: a9a00e6
File: ./build/logs/clover.xml

Summary - Lines: 90.55% | Methods: 65.22%
FilesLinesMethodsBranches
lib/transloadit
   CurlRequest.php91.67%66.67%100.00%
   CurlResponse.php100.00%100.00%100.00%
   Transloadit.php86.67%55.56%100.00%
   TransloaditRequest.php92.98%57.14%100.00%
   TransloaditResponse.php100.00%100.00%100.00%

🤖 comment via lucassabreu/comment-coverage-clover

@kvz
kvz requested a review from tim-kos October 21, 2026 16:47
@kvz
kvz marked this pull request as ready for review October 21, 2026 16:47
@kvz
kvz merged commit b96021d into main Oct 21, 2026
6 checks passed
@kvz
kvz deleted the runtime-security branch October 21, 2026 19:37
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