-
Notifications
You must be signed in to change notification settings - Fork 144
Feature/update php jwt use php8 #246
base: 3.x
Are you sure you want to change the base?
Feature/update php jwt use php8 #246
Conversation
Updated codebase in order to still use multiple keys. |
@tuupola are you looking for maintainers? Seems like this PR solves a bunch of the compatibility issues. |
Actually I am pondering giving away maintainership of some of the PHP projects including |
Hi @tuupola I would be interested in joining as a maintainer on this project to help move it forward. Please let me know if that's something you'd like. |
In case I can help with something else, let me know. Also, if there is more to implement at this PR, I'd be glad to do it as well. |
@tuupola any reason this hasn't been moved forward? Is there any help needed here to push this forward? Happy to lend some assistance if helpful. |
Such a shame to see this project die :'( |
Open support for PHP versions above 8.0 and include php-jwt ^6.0.
Some refactors were made in order to use some features in the future as named parameters, typed properties and so on.
Since using multiple keys are not usual, I personally preferred to remove support for this feature, as it usually leads to difficulty in regard of maintaining keys secure and less prone to misleading behaviour.