Skip to content
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

Magento 2.1.9 + PHP 7.0 memcache session storage = slow login/checkout #12742

Closed
AlexisEvo opened this issue Dec 15, 2017 · 2 comments
Closed
Labels
Component: Framework/Cache Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line Issue: Clear Description Gate 2 Passed. Manual verification of the issue description passed Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release

Comments

@AlexisEvo
Copy link

Preconditions

  1. Magento version 2.1.9 (or newer)
  2. PHP 7.0 (or newer)
  3. Unofficial pecl-memcache extension (from https://github.com/websupport-sk/pecl-memcache/ -- there is no official release of memcache for PHP 7.0)

Steps to reproduce

  1. Install Magento 2.1.9+ on PHP 7.0 with the memcache (not memcached) extension
  2. Configure Magento to use memcache (not memcached) for session storage
  3. Attempt to login (frontend or backend) or checkout

Expected result

  1. Page should load within 1-2 seconds

Actual result

  1. Magento stalls for 15 seconds (memcache.lock_timeout) when trying to regenerate the session ID in Magento::Framework::Session->regenerateId();

This was broken in 2.1.9 due to changes to the regenerate session ID code in Magento. You may understandably want to blame the memcache extension for this (as the code works in file storage, and with the memcached (with a D) extension). However, since Magento 2.2.0+ requires PHP 7.0, I feel that Magento should refuse to use the 'memcache' extension or throw an error/warning about this bug until it is fixed upstream.

@magento-engcom-team magento-engcom-team added the Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed label Dec 15, 2017
@engcom-backlog-nickolas engcom-backlog-nickolas added Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Component: Framework/Cache Issue: Clear Description Gate 2 Passed. Manual verification of the issue description passed Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line labels Aug 10, 2018
@engcom-backlog-nickolas

Hello @AlexisEvo, thank you for your report.
We've acknowledged the issue and added to our backlog.

@engcom-backlog-nickolas engcom-backlog-nickolas removed their assignment Aug 10, 2018
@AlexisEvo
Copy link
Author

This issue was fixed in 2.2.3 I believe, related to #12385 #12362 #14973. Closing out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Framework/Cache Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line Issue: Clear Description Gate 2 Passed. Manual verification of the issue description passed Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release
Projects
None yet
Development

No branches or pull requests

3 participants