refactoring to avoid tampering with $_SERVER

This commit is contained in:
Nicola Beghin 2017-03-18 13:34:26 +01:00
parent ff210b04d7
commit ca6ca9376e

View file

@ -82,17 +82,17 @@ class BasicAuthenticate extends BaseAuthenticate {
* @return mixed Either false or an array of user information * @return mixed Either false or an array of user information
*/ */
public function getUser(CakeRequest $request) { public function getUser(CakeRequest $request) {
$username = '';
$pass = '';
if (!isset($_SERVER['PHP_AUTH_USER'])) { if (!isset($_SERVER['PHP_AUTH_USER'])) {
$httpAuthorization = $request->header('Authorization');
if ($httpAuthorization !== false && strlen($httpAuthorization) > 0 && strpos($httpAuthorization, 'basic') !== false) {
list($_SERVER['PHP_AUTH_USER'], $_SERVER['PHP_AUTH_PW']) = explode(':', base64_decode(substr($httpAuthorization, 6)));
if (strlen($_SERVER['PHP_AUTH_USER']) === 0 || strlen($_SERVER['PHP_AUTH_PW']) === 0) {
unset($_SERVER['PHP_AUTH_USER'], $_SERVER['PHP_AUTH_PW']);
}
}
}
$username = env('PHP_AUTH_USER'); $username = env('PHP_AUTH_USER');
$pass = env('PHP_AUTH_PW'); $pass = env('PHP_AUTH_PW');
} else {
$httpAuthorization = $request->header('Authorization');
if ($httpAuthorization !== false && strlen($httpAuthorization) > 0 && strpos($httpAuthorization, 'basic') !== false) {
list($username, $pass) = explode(':', base64_decode(substr($httpAuthorization, 6)));
}
}
if (!is_string($username) || $username === '' || !is_string($pass) || $pass === '') { if (!is_string($username) || $username === '' || !is_string($pass) || $pass === '') {
return false; return false;