diff options
author | Simon Rettberg | 2018-12-07 10:09:19 +0100 |
---|---|---|
committer | Simon Rettberg | 2018-12-07 10:09:19 +0100 |
commit | 51ef0363805f988b58ea738d4c11c04ef623d07a (patch) | |
tree | b7b18ea4c42cc2ca8e1b05df3fb7dcece73e9973 | |
parent | Improve shibboleth eppn/affiliation handling (diff) | |
download | bwlp-webadmin-51ef0363805f988b58ea738d4c11c04ef623d07a.tar.gz bwlp-webadmin-51ef0363805f988b58ea738d4c11c04ef623d07a.tar.xz bwlp-webadmin-51ef0363805f988b58ea738d4c11c04ef623d07a.zip |
[inc/Session] Don't suppress setcookie errors
-rw-r--r-- | inc/session.inc.php | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/inc/session.inc.php b/inc/session.inc.php index 93c33e5..17e3184 100644 --- a/inc/session.inc.php +++ b/inc/session.inc.php @@ -87,7 +87,7 @@ class Session { if (self::$sid === false) return; Database::exec('DELETE FROM websession WHERE sid = :sid', array('sid' => self::$sid)); - @setcookie('sid', '', time() - CONFIG_SESSION_TIMEOUT, null, null, !empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] != 'off', true); + setcookie('sid', '', time() - CONFIG_SESSION_TIMEOUT, null, null, !empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] != 'off', true); self::$sid = false; self::$data = false; } @@ -103,7 +103,7 @@ class Session array('sid' => self::$sid, 'data' => $data)); if ($ret === false) Util::traceError('Storing session data in Dahdähbank failed.'); - $ret = @setcookie('sid', self::$sid, time() + CONFIG_SESSION_TIMEOUT, null, null, !empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] != 'off', true); + $ret = setcookie('sid', self::$sid, time() + CONFIG_SESSION_TIMEOUT, null, null, !empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] != 'off', true); if ($ret === false) Util::traceError('Error: Could not set Cookie for Client (headers already sent)'); } |