Skip to content

Commit

Permalink
[TASK] Streamline backend password reset log message
Browse files Browse the repository at this point in the history
The current backend password reset log message misses a `}` and logs
the `user_id` if a password has been reset successfully. Logging the
`user_id` is however not inline with other user related log messages,
where always the `username` is logged.

This patch streamlines the log message with other user related log
messages by including the `username` instead of the `user_id`.

Resolves: #99354
Releases: main, 12.4
Signed-off-by: Torben Hansen <derhansen@gmail.com>
Change-Id: If18d3b4cd1a76008f56f5433e8ff5d591898d2ad
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/79824
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Tested-by: core-ci <typo3@b13.com>
  • Loading branch information
derhansen authored and bmack committed Jul 6, 2023
1 parent d9c2998 commit 6d6eff1
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -352,7 +352,7 @@ public function resetPassword(ServerRequestInterface $request, Context $context)

$this->invalidateUserSessions($userId);

$this->logger->info('Password reset successful for user {user_id)', ['user_id' => $userId]);
$this->logger->info('Password reset successful for user \'{username}\'', ['username' => $user['username'], 'user_id' => $userId]);
$this->log(
'Password reset successful for user %s',
SystemLogLoginAction::PASSWORD_RESET_ACCOMPLISHED,
Expand Down

0 comments on commit 6d6eff1

Please sign in to comment.