Skip to content

Fix PHP version constraint to match php-jwt v7 requirements#66

Merged
maztch merged 1 commit intoilovepdf:developfrom
xeno-suter:feature/fix-php-version-constraint
Apr 20, 2026
Merged

Fix PHP version constraint to match php-jwt v7 requirements#66
maztch merged 1 commit intoilovepdf:developfrom
xeno-suter:feature/fix-php-version-constraint

Conversation

@xeno-suter
Copy link
Copy Markdown

Summary

Follow-up to #64firebase/php-jwt was upgraded to ^7.0, but the PHP version constraint was not updated.

Problem

firebase/php-jwt v7 requires PHP >=8.0, but composer.json still declares "php": ">=7.3". This means composer install on PHP 7.x will fail with an unresolvable dependency, which is confusing for users.

See: https://github.com/firebase/php-jwt/blob/main/composer.json

Changes

  • composer.json: "php": ">=7.3""php": ">=8.0"
  • readme.md: PHP requirement updated from 7.4 to 8.0

Why this is safe

  • PHP 7.3 EOL: December 2021
  • PHP 7.4 EOL: November 2022
  • No code changes needed — JWT::encode() API is identical in v7
  • firebase/php-jwt v6.x was affected by CVE-2025-45769 (see: GHSA-2x45-7fc3-mxwq)

Follow-up to ilovepdf#64 - firebase/php-jwt was upgraded to ^7.0 which requires PHP >=8.0, but the PHP constraint was not updated.
@D3strukt0r
Copy link
Copy Markdown

Seems my work colleague forgot to notify you, so I might as well do it for him: @maztch

@mirkoRegold
Copy link
Copy Markdown

In JWT version 6 there is a weak encryption, UP for this PR

@D3strukt0r
Copy link
Copy Markdown

i'd also be happy for there to be a release soon, there is no point if this stays in develop forever

@federicoRegold
Copy link
Copy Markdown

UP

1 similar comment
@timnijssen
Copy link
Copy Markdown

UP

@maztch maztch merged commit 8b45fbc into ilovepdf:develop Apr 20, 2026
@MirkoFarina
Copy link
Copy Markdown

@maztch, when will the new version with this fix be released?

maztch added a commit that referenced this pull request Apr 20, 2026
* Add support for php-jwt v7 (#64)
* Fix PHP version constraint to match php-jwt v7 requirements (#66)
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.

7 participants