Skip to content
GitLab
Verified Commit 2394ef23 authored by dockx thibault's avatar dockx thibault
Browse files

:sparkles: (CORE) - add logging when user is locked 

Fixing code style.
parent 13ded56b
dev 6344-template-issue-when-creating-a-template-with-empty-password-error-message-should-not-be-seen 6365-core-locking-mechanism-is-not-changing-the-mail-ressource-it-does-lock-the-mail-account 6365-core-when-lock-mechanism-is-trigger-the-user-should-not-be-editable-if-not-unlock 6378-orcid-test-method-is-wrong-and-break-orcid-saving core-php8 master fusiondirectory-1.5
No related merge requests found
Hide whitespace changes
Inline Side-by-side
Showing
with 21 additions and 20 deletions
+21 -20
plugins/management/users/class_userManagement.inc
+ 21
− 20
View file @ 2394ef23
...@@ -25,14 +25,15 @@ class userManagement extends management ...@@ -25,14 +25,15 @@ class userManagement extends management
public static $skipTemplates = FALSE; public static $skipTemplates = FALSE;
public static $columns = [ public static $columns
['ObjectTypeColumn', []], = [
['LinkColumn', ['attributes' => 'sn', 'label' => 'Last name']], ['ObjectTypeColumn', []],
['LinkColumn', ['attributes' => 'givenName', 'label' => 'First name']], ['LinkColumn', ['attributes' => 'sn', 'label' => 'Last name']],
['Column', ['attributes' => 'uid', 'label' => 'Login']], ['LinkColumn', ['attributes' => 'givenName', 'label' => 'First name']],
['PropertiesColumn', ['label' => 'Properties']], ['Column', ['attributes' => 'uid', 'label' => 'Login']],
['ActionsColumn', ['label' => 'Actions']], ['PropertiesColumn', ['label' => 'Properties']],
]; ['ActionsColumn', ['label' => 'Actions']],
];
static function plInfo (): array static function plInfo (): array
{ {
...@@ -117,10 +118,10 @@ class userManagement extends management ...@@ -117,10 +118,10 @@ class userManagement extends management
} }
$error = new FusionDirectoryError( $error = new FusionDirectoryError(
htmlescape(sprintf( htmlescape(sprintf(
_('Password method "%s" does not support locking. Account "%s" has not been locked!'), _('Password method "%s" does not support locking. Account "%s" has not been locked!'),
$hn, $hn,
$dn $dn
)) ))
); );
$error->display(); $error->display();
return; return;
...@@ -130,14 +131,14 @@ class userManagement extends management ...@@ -130,14 +131,14 @@ class userManagement extends management
if (($action == 'lock') && !$method->is_locked($dn)) { if (($action == 'lock') && !$method->is_locked($dn)) {
$success = $method->lock_account($dn); $success = $method->lock_account($dn);
// Requiring logging mechanism for audit. // Requiring logging mechanism for audit.
if($success) { if ($success) {
logging::log('security', 'account', $dn, [], 'DN : '.$dn.' is locked.' ); logging::log('security', 'account', $dn, [], 'DN : ' . $dn . ' is locked.');
} }
} elseif (($action == 'unlock') && $method->is_locked($dn)) { } elseif (($action == 'unlock') && $method->is_locked($dn)) {
$success = $method->unlock_account($dn); $success = $method->unlock_account($dn);
// Requiring logging mechanism for audit. // Requiring logging mechanism for audit.
if($success) { if ($success) {
logging::log('security', 'account', $dn, [], 'DN : '.$dn.' is unlocked.' ); logging::log('security', 'account', $dn, [], 'DN : ' . $dn . ' is unlocked.');
} }
} }
...@@ -149,10 +150,10 @@ class userManagement extends management ...@@ -149,10 +150,10 @@ class userManagement extends management
} }
$error = new FusionDirectoryError( $error = new FusionDirectoryError(
htmlescape(sprintf( htmlescape(sprintf(
_('Locking failed using password method "%s". Account "%s" has not been locked!'), _('Locking failed using password method "%s". Account "%s" has not been locked!'),
$hn, $hn,
$dn $dn
)) ))
); );
$error->display(); $error->display();
} }
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment