-
Notifications
You must be signed in to change notification settings - Fork 32
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
Support phpseclib V3 #132
Comments
@grmeyer-hw-dev Can we maybe bring the gree/jose old not-maintaned package to inside hyperwallet/sdk so we can correct the namespaces? I think besides replacing the whole JWT package that would be an easier and quicker solution for now. If I do the change would you merge it? I just need to make sure you're ok with it, we really need phpseclib v3 |
I saw your Initial PR, and I tried to update too but it requires more change upgrade to phpseclib V3.
|
But we can replace it for sure, Do you know any other option for |
Hmm I actually raised an MR and all the checks passed. If you don't want to do that I guess because I brought the package inside the hyperwallet SDK, you could try using firebase/php-jwt |
My PR #133 |
Hey there,
We could really use some help getting phpseclib V3 support up and running—it's crucial for our application. I've tried updating the version in composer.json, but we're hitting a roadblock with the 'gree/jose' package, which hasn't been maintained in 8 years and remains a key dependency, even in HW SDK v4.
Do you have any plans to replace this package with a more maintained one like lcobucci/jwt? That would be a game-changer for us.
The text was updated successfully, but these errors were encountered: