Skip to content
Permalink
Browse files

FIX: Fix incompatibility with Check 1.6 auth cookies of users with sp…

…ecial characters in the user ID
  • Loading branch information
LarsMichelsen committed Nov 11, 2019
1 parent 81b1723 commit 3848571d71ee22cb1d9d9f3d52c926c7d57c8e08
Showing with 7 additions and 1 deletion.
  1. +2 −0 ChangeLog
  2. +5 −1 share/server/core/classes/CoreLogonMultisite.php
@@ -2,6 +2,8 @@
* FIX: Fix PHP 5.3 incompatibility introduced with 1.9.16
* FIX: Fix IE 11 incompatibility on worldmaps in relation to lines
which was introduced with 1.9.13
* FIX: Fix incompatibility with Check 1.6 auth cookies of users with special
characters in the user ID

1.9.16
Core:
@@ -79,7 +79,11 @@ private function checkAuthCookie($cookieName) {
throw new Exception();
}
list($username, $issueTime, $cookieHash) = explode(':', $_COOKIE[$cookieName], 3);
// Checkmk 1.6+ may add double quotes round the value in some cases
// (e.g. when @ signs are found in the value)
$cookieValue = trim($_COOKIE[$cookieName], '"');
list($username, $issueTime, $cookieHash) = explode(':', $cookieValue, 3);
if($this->authFile == 'htpasswd')
$users = $this->loadAuthFile($this->htpasswdPath);

0 comments on commit 3848571

Please sign in to comment.
You can’t perform that action at this time.