Skip to content

Update to PHP8.3 and lcobucci/jwt 5#42

Open
heiglandreas wants to merge 2 commits into
steverhoades:masterfrom
heiglandreas:prepareForPhp8.3
Open

Update to PHP8.3 and lcobucci/jwt 5#42
heiglandreas wants to merge 2 commits into
steverhoades:masterfrom
heiglandreas:prepareForPhp8.3

Conversation

@heiglandreas
Copy link
Copy Markdown

This PR provides changes to be able to use the library with PHP8.3 as well as with lcobucci/jwt 5.

The changes were rather small. It consists of 2 things:

  • Update to PHPUnit 10 to avoid the prophecy requirement that is stuck to PHP8.2. The changes required are mainly moving from instance data-providers to static data-providers along with some minor adaptions.
  • Update the code to use lcobucci/jwt 5. The only change necessary was to replace using the configuration object to parse the token. The simple way of creating a configuration object doesn ot work any more, so I instead created the parser that is used in the configurartions "parse" method and use that directly.

THis prepares the whole testsuite to be usable with PHP8.3
THis allows to use the latest version of the JWT-lib. Some changes where
necessary due to the previous way not being supported anymore but as
that was anyhow only used to parse the token we can also use the default
way of parsing the token without needing a whole configuration object in
the first place
@JurRutten
Copy link
Copy Markdown

+1
Any updates on this pr?

@MichielDilissen
Copy link
Copy Markdown

+1

1 similar comment
@ToonFra
Copy link
Copy Markdown

ToonFra commented Nov 7, 2025

+1

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.

4 participants