-
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
Critical Klarna issue on Magento 2.4.3 - request logging blocks checkout #35065
Comments
Hi @bartoszkubicki. Thank you for your report.
Make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, Add a comment to the issue:
For more details, review the Magento Contributor Assistant documentation. Add a comment to assign the issue: To learn more about issue processing workflow, refer to the Code Contributions.
🕙 You can find the schedule on the Magento Community Calendar page. 📞 The triage of issues happens in the queue order. If you want to speed up the delivery of your contribution, join the Community Contributions Triage session to discuss the appropriate ticket. ✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel |
We just had this issue. The second point that I think should be raised is why it causes all the checkout to fail? We have other payment providers as well and due to this issue, we had no orders coming in. Regards. |
@oliverde8 It is caused by poor Klarna error handling. We have multiple fixes on native Klarna module, it throws exception here an there, just because some quote is not matching its conditions, instead of detaching itself from payment methods list. |
Hi @engcom-Delta. Thank you for working on this issue.
|
Hi @bartoszkubicki , |
As stated in the klarna documentation, you need to have 32767 Klarna orders in the database before this issue appears. You also need to have the parameter, "Logging" Enabled for all these orders. So this issue can not be reproduced on a clean Magento install (unless you pass 32767 orders or fill up the table manually). It's a ticking time bomb if the website is using Klarna. Regards. |
Hi @oliverde8 , Many thanks for quick turn around, Would like to clarify once again , do i need to have 32767 Klarna orders in the DB ? or are you referring issue # 32767? .Kindly confirm. |
Hello, Yes to the DB. You need 32767 entries in the klarna_logs table. The klarna_logs table is using tinyint so the number of logs it can store is quite limited. That's the main problem. By default the cleanup process keeps 90 days of logs in that table, I doubt we had 32K orders in 3 month (default config). But we didn't check so it might be correct. The patch Klarna provides on their website contains changes (that I find strange) to the cleanup process as well. As a workaround, We simply disabled the logs and emptied that table and didn't apply that patch. |
I confirm what @oliverde8 has written. Anyway it is described in Klarna article I've posted in initial post. |
Hi @engcom-Hotel. Thank you for working on this issue.
|
@magento give me 2.4.3-p1 instance |
Hi @engcom-Hotel. Thank you for your request. I'm working on Magento instance for you. |
Hi @engcom-Hotel, unfortunately there is no ability to deploy Magento instance at the moment. Please try again later. |
Hello @bartoszkubicki, We have tried to reproduce this issue in latest stable version of Magento i.e. 2.4.3-p1. We have also gone through with the link provided in the main description. The issue seems reproducible by looking into the code base as well as database structure. Please have a look into the below screenshot of the The Thanks |
✅ Jira issue https://jira.corp.magento.com/browse/AC-2801 is successfully created for this GitHub issue. |
✅ Confirmed by @engcom-Hotel. Thank you for verifying the issue. |
Preconditions (*)
Steps to reproduce (*)
Expected result (*)
Actual result (*)
Please provide Severity assessment for the Issue as Reporter. This information will help during Confirmation and Issue triage processes.
The text was updated successfully, but these errors were encountered: