-
Notifications
You must be signed in to change notification settings - Fork 455
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Apple - Broken since 5.5.0 #1015
Comments
Can you PR a fix? Otherwise I will revert the previous PR |
Please check my pull request with bugfix, for me it works. Lcobucci\JWT\Configuration::forUnsecuredSigner() - Fixed using Configuration::forSymmetricSigner() And bonus, I added the option to enter a certificate instead of a secret key. (In separate PR) |
@hamrak can you please split this into two PRs? Bugfix for issue first please and then you can do the other changes. In the meantime i have reverted. |
lcobucci/jwt has removed a number of functions:
See https://lcobucci-jwt.readthedocs.io/en/stable/upgrading/#removal-of-none-algorithm - however 5.5.0 still makes use of
Lcobucci\JWT\Configuration::forUnsecuredSigner()
hereProviders/src/Apple/Provider.php
Line 148 in e5cbb4b
@hamrak @atymic
The text was updated successfully, but these errors were encountered: