Skip to content

Commit

Permalink
Merge pull request #2226 from magento-thunder/MAGETWO-85983
Browse files Browse the repository at this point in the history
Fixed issues:
- MAGETWO-85983: [Backport for 2.1.x] Admin login always takes >30 sec when using Redis on 2.2.1 #12385
  • Loading branch information
Yevhen Miroshnychenko authored Mar 16, 2018
2 parents e719669 + 86f25a5 commit 92325c4
Showing 1 changed file with 2 additions and 11 deletions.
13 changes: 2 additions & 11 deletions lib/internal/Magento/Framework/Session/SessionManager.php
Original file line number Diff line number Diff line change
Expand Up @@ -503,18 +503,9 @@ public function regenerateId()
if (headers_sent()) {
return $this;
}
//@see http://php.net/manual/en/function.session-regenerate-id.php#53480 workaround

if ($this->isSessionExists()) {
$oldSessionId = session_id();
session_regenerate_id();
$newSessionId = session_id();
session_id($oldSessionId);
session_destroy();

$oldSession = $_SESSION;
session_id($newSessionId);
session_start();
$_SESSION = $oldSession;
session_regenerate_id(true);
} else {
session_start();
}
Expand Down

1 comment on commit 92325c4

@danielmartinezd
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,
Do we know when this code will be released to production?
Thanks

Please sign in to comment.