fanbinit/rhymix
1
0
Fork 0
mirror of https://github.com/Lastorder-DC/rhymix.git synced 2026-01-08 03:01:43 +09:00
Code Issues Projects Releases Packages Wiki Activity

Don't trust mobile cookie if the user agent has changed

Browse source
This commit is contained in:
Kijin Sung 2016-05-11 00:23:13 +09:00
parent ad8a07096c
commit c58c905a93
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
classes/mobile/Mobile.class.php
View file

@ -45,7 +45,7 @@ class Mobile
// Try to detect from URL arguments and cookies, and finally fall back to user-agent detection. // Try to detect from URL arguments and cookies, and finally fall back to user-agent detection.
$m = Context::get('m'); $m = Context::get('m');
$cookie = isset($_COOKIE['mobile']) ? $_COOKIE['mobile'] : null; $cookie = (isset($_COOKIE['mobile']) && $_SESSION['user_agent'] === md5($_SERVER['HTTP_USER_AGENT'])) ? $_COOKIE['mobile'] : null;
if ($m === '1' || $cookie === 'true') if ($m === '1' || $cookie === 'true')
{ {
self::$_ismobile = TRUE; self::$_ismobile = TRUE;
@ -62,6 +62,7 @@ class Mobile
// Set cookie to prevent recalculation. // Set cookie to prevent recalculation.
if (!$cookie) if (!$cookie)
{ {
$_SESSION['user_agent'] = md5($_SERVER['HTTP_USER_AGENT']);
$_COOKIE['mobile'] = self::$_ismobile ? 'true' : 'false'; $_COOKIE['mobile'] = self::$_ismobile ? 'true' : 'false';
setcookie('mobile', $_COOKIE['mobile'], 0, RX_BASEURL); setcookie('mobile', $_COOKIE['mobile'], 0, RX_BASEURL);
} }