LSauthMethod_HTTP: fix logging HTTP_AUTHORIZATION parsing error

This commit is contained in:
Benjamin Renard 2021-08-25 09:39:25 +02:00
parent b5c5123bbb
commit fed9acfedd

View file

@ -59,6 +59,7 @@ class LSauthMethod_HTTP extends LSauthMethod_basic {
$missing_info = null; $missing_info = null;
switch(constant('LSAUTHMETHOD_HTTP_METHOD')) { switch(constant('LSAUTHMETHOD_HTTP_METHOD')) {
case 'AUTHORIZATION': case 'AUTHORIZATION':
$missing_info = 'HTTP_AUTHORIZATION';
if (isset($_SERVER['HTTP_AUTHORIZATION']) && !empty($_SERVER['HTTP_AUTHORIZATION'])) { if (isset($_SERVER['HTTP_AUTHORIZATION']) && !empty($_SERVER['HTTP_AUTHORIZATION'])) {
$authData = explode(':', base64_decode(substr($_SERVER['HTTP_AUTHORIZATION'], 6))); $authData = explode(':', base64_decode(substr($_SERVER['HTTP_AUTHORIZATION'], 6)));
if (is_array($authData) && count($authData) == 2) { if (is_array($authData) && count($authData) == 2) {
@ -71,7 +72,6 @@ class LSauthMethod_HTTP extends LSauthMethod_basic {
else else
self :: log_warning("Fail to decode and parse $missing_info environment variable."); self :: log_warning("Fail to decode and parse $missing_info environment variable.");
} }
$missing_info = 'HTTP_AUTHORIZATION';
break; break;
case 'REMOTE_USER': case 'REMOTE_USER':
if (isset($_SERVER['REMOTE_USER']) && !empty($_SERVER['REMOTE_USER'])) { if (isset($_SERVER['REMOTE_USER']) && !empty($_SERVER['REMOTE_USER'])) {