Bug #68087

SaltedPasswordService.php calls $this->writeLogMessage with too many parameters

Added by Tymoteusz Motylewski over 4 years ago. Updated over 4 years ago.

Status:
Rejected
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2015-07-13
Due date:
% Done:

0%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
no-brainer
Is Regression:
No
Sprint Focus:

Description

writeLogMessage in SaltedPasswordService.php has just one param. However it is called with few additional in multiple places.

History

#1 Updated by David Greiner over 4 years ago

Hi Tymoteusz!

Method writeLogMessage() in SaltedPasswordService.php checks whether further arguments existing (using func_num_args(), func_get_args()), and if so, these arguments are used within vsprintf(...) to form proper messages.

Furthermore this is documented in the phpdoc comments above this method.

However, are you in trouble with this? Do you noticed some errors or warnings in your log files?

#2 Updated by Tymoteusz Motylewski over 4 years ago

  • Status changed from New to Rejected

hi,
Didn't notice that. I'm closing the issue.

Also available in: Atom PDF