-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Missing dependency on web-token/jwt-easy? #37783
Comments
Hi @hostep. Thank you for your report.
Join Magento Community Engineering Slack and ask your questions in #github channel. |
Hi @engcom-November. Thank you for working on this issue.
|
Hello @hostep, Thank you for report and collaboration! As you mentioned the Hence we can confirm the issue. Thanks. |
❌ Something went wrong. Cannot create Jira issue. |
❌ Cannot export the issue. This GitHub issue is already linked to Jira issue(s): https://jira.corp.adobe.com/browse/AC-9232 |
@magento I am working on this |
-Removed an unused dependency in the code
Preconditions and environment
2.4-develop
Steps to reproduce
magento2/app/code/Magento/JwtFrameworkAdapter/Model/AlgorithmProviderFactory.php
Line 11 in ca30c47
Jose\Easy
)Jose\Easy\AlgorithmProvider
) is nowhere to be found in the magento codebase after installing all composer dependenciesExpected result
Magento should use code that's included, it shouldn't use code that's not included as this will crash.
Actual result
Found referenced code that's not present in the codebase.
Additional information
I suspect this happened because:
web-token/jwt-framework
package was updated from version v2.2.11 to 3.1.2Jose\Easy\AlgorithmProvider
got removed fromweb-token/jwt-framework
Possible solution: add a requirement on
web-token/jwt-easy
, however, that package is considered abandoned, so not sure if that's the best way forward here?Also: there are probably too few automated tests for this
JwtFrameworkAdapter
magento module, otherwise this would have been catched earlier...Release note
No response
Triage and priority
The text was updated successfully, but these errors were encountered: