aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private/Session/CryptoSessionData.php
diff options
context:
space:
mode:
Diffstat (limited to 'lib/private/Session/CryptoSessionData.php')
-rw-r--r--lib/private/Session/CryptoSessionData.php17
1 files changed, 17 insertions, 0 deletions
diff --git a/lib/private/Session/CryptoSessionData.php b/lib/private/Session/CryptoSessionData.php
index 2e3bd46da5b..b01887e39e2 100644
--- a/lib/private/Session/CryptoSessionData.php
+++ b/lib/private/Session/CryptoSessionData.php
@@ -97,8 +97,17 @@ class CryptoSessionData implements \ArrayAccess, ISession {
* @param mixed $value
*/
public function set(string $key, $value) {
+ if ($this->get($key) === $value) {
+ // Do not write the session if the value hasn't changed to avoid reopening
+ return;
+ }
+
+ $reopened = $this->reopen();
$this->sessionValues[$key] = $value;
$this->isModified = true;
+ if ($reopened) {
+ $this->close();
+ }
}
/**
@@ -131,9 +140,13 @@ class CryptoSessionData implements \ArrayAccess, ISession {
* @param string $key
*/
public function remove(string $key) {
+ $reopened = $this->reopen();
$this->isModified = true;
unset($this->sessionValues[$key]);
$this->session->remove(self::encryptedSessionName);
+ if ($reopened) {
+ $this->close();
+ }
}
/**
@@ -149,6 +162,10 @@ class CryptoSessionData implements \ArrayAccess, ISession {
$this->session->clear();
}
+ public function reopen(): bool {
+ return $this->session->reopen();
+ }
+
/**
* Wrapper around session_regenerate_id
*