-
Notifications
You must be signed in to change notification settings - Fork 253
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
refs Fix from Deivison Arthur for formKey issues and switch between fixes #846
Conversation
This PR adds 2 different ways to bypass the formKey issue. The first and default way is to pipe the first page visit for new sessions and the second one is an implementation of Inovarti_FixAddToCartMage18. This can be managed from System - Configuration - Turpentine - Varnish Options - General Options - Use VCL fix. |
From the official documentation there is another way to solve this issue. CSRF can be completely disabled now by going to System - Advanced - System - CRSF protection - Add secret key to URL - No. This was the official fix by Magento to known issues betwen Magento and Varnish We don't recommend completely disabling the CSRF even when it represents a low risk for the frontend. While using this in conjunction with Turpetine, go to System - Configuration - Turpentine - Varnish Options - General Options - Use VCL fix - No. |
@miguelbalparda Thanks, seems a better solution by giving everyone an option instead of just disabling session generation |
refs Fix from Deivison Arthur for formKey issues and switch between fixes
got this error when i tried to install inovarti fix, can you advise. it also blocked my admin backend. There has been an error processing your request Mage registry key "_singleton/fixaddtocartmage18/observer" already exists Trace: |
@jasonv77 you should ask the module provider about that one. Anyway I think there is no need to install anything to solve this issue since we already have 3 possibles fixes. |
i had turned off csrf to fix my issue but as you advised was not recommended so i tried the other one and i got that error |
The code you are referring to is not in our repository. It seems you installed the fix and then you tried to use it in conjunction with Turpentine @jasonv77. I think it is better to ask this in the repository where you found the code you are using. |
k |
I have some problems with cart and checkout and Use VCL fix = no solved these problems. Is dangerous to keep Use VCL fix = no? (because you write "Please note that the observer method will disable CRSF protection.") |
Added fix for formKey from https://github.com/deivisonarthur/Inovarti_FixAddToCartMage18/ by Deivison Arthur.