Skip to content

Update firebase/php-jwt to ^7.0 (security fix)#456

Open
lizkenyon wants to merge 4 commits intomainfrom
update-firebase-jwt
Open

Update firebase/php-jwt to ^7.0 (security fix)#456
lizkenyon wants to merge 4 commits intomainfrom
update-firebase-jwt

Conversation

@lizkenyon
Copy link
Contributor

Summary

  • Updates firebase/php-jwt from ^5.2 || ^6.2 to ^7.0 to address security vulnerability GHSA-2x45-7fc3-mxwq (CVE-2025-45769, CVSS 7.3 — weak encryption)
  • Updates test secret keys to meet v7's minimum 32-byte HMAC key requirement and recalculates all dependent HMAC fixtures
  • No production code changes required — only the dependency constraint and test fixtures are affected

Context

firebase/php-jwt v7 enforces minimum key sizes for HMAC algorithms (32 bytes for HS256). Shopify API secrets are always >= 32 characters (legacy format: 32 hex chars, new format: shpss_ + 32 chars = 38 chars), so no real users are affected by this enforcement. The test suite used short mock secrets ('steffi', 'rocky') which needed updating.

Closes #454

Test plan

  • All 221 tests pass (1 skipped: APCu extension test, pre-existing)
  • Linter passes clean
  • CI passes

🤖 Generated with Claude Code

lizkenyon and others added 4 commits February 27, 2026 13:24
Addresses GHSA-2x45-7fc3-mxwq (CVE-2025-45769, CVSS 7.3) by upgrading
firebase/php-jwt from ^5.2 || ^6.2 to ^7.0. The v7 release enforces
minimum HMAC key sizes (32 bytes for HS256), which required updating
test secret keys and recalculating dependent HMAC fixtures.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The previous lockfile was generated on PHP 8.4, which resolved
brick/math to 0.14.8 (requires PHP ^8.2). This broke the library's
declared PHP ^8.1 support. Regenerating on PHP 8.1 resolves
brick/math to 0.13.1, which supports PHP ^8.1.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Improves readability by replacing inline hex strings with
TEST_API_SECRET and TEST_API_SECRET_ALT constants in BaseTestCase.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

Package affected by PHP-JWT vulnerability

1 participant