Skip to content
Snippets Groups Projects
Commit 8f613a1e authored by Hendrik Grahl's avatar Hendrik Grahl
Browse files

Improve debug message and avoid warning due to escape format

parent 9a1ce5ca
No related branches found
No related tags found
No related merge requests found
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
namespace Drupal\ldap_sso\Controller; namespace Drupal\ldap_sso\Controller;
use Drupal\Component\Utility\SafeMarkup; use Drupal\Component\Utility\Html;
use Drupal\Core\Access\AccessResult; use Drupal\Core\Access\AccessResult;
use Drupal\Core\Controller\ControllerBase; use Drupal\Core\Controller\ControllerBase;
use Drupal\Core\Url; use Drupal\Core\Url;
...@@ -200,11 +200,11 @@ class LoginController extends ControllerBase implements LdapSsoInterface { ...@@ -200,11 +200,11 @@ class LoginController extends ControllerBase implements LdapSsoInterface {
*/ */
private function validateUser($remote_user) { private function validateUser($remote_user) {
if ($this->detailedWatchdogLog) { if ($this->detailedWatchdogLog) {
\Drupal::logger('ldap_sso')->debug('Checking for invalid user.'); \Drupal::logger('ldap_sso')->debug('Starting validation for SSO user.');
} }
$validator = new LoginValidator(); $validator = new LoginValidator();
$validator->processSsoLogin(SafeMarkup::checkPlain($remote_user)); $validator->processSsoLogin(Html::escape($remote_user));
if ($this->detailedWatchdogLog) { if ($this->detailedWatchdogLog) {
\Drupal::logger('ldap_sso')->debug('Remote user has local uid @uid', \Drupal::logger('ldap_sso')->debug('Remote user has local uid @uid',
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment