rooty/yubikey-val
1
0
Fork 0
mirror of https://github.com/Yubico/yubikey-val.git synced 2025-02-08 03:54:20 +01:00
Code Issues Projects Releases Wiki Activity

Modified log messages.

Browse Source 
- avoid doing what is already handled by the Log class.

- the log name is appended automatically,
	so don't append it again in the invocation.

	i.e. "ykval-verify"

- the log level name is also appended automatically,
	so don't append it manually, especially when it doesn't match the log priority!

	i.e. LOG_WARNING -> ":notice:"

- fix whitespace in some messages.
This commit is contained in:
Jean Paul Galea 2015-09-10 20:53:56 +02:00
parent 2df72604ed
commit ad167cd38a
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ykval-sync.php
View File

@ -161,7 +161,7 @@ if ($sync->countersEqual($localParams, $syncParams))
if ($syncParams['nonce'] != $localParams['nonce']) if ($syncParams['nonce'] != $localParams['nonce'])
{ {
$myLog->log(LOG_WARNING, 'Remote server has received a request to validate an already validated OTP '); $myLog->log(LOG_WARNING, 'Remote server has received a request to validate an already validated OTP');
} }
} }

6
ykval-verify.php
View File

@ -371,7 +371,7 @@ if (!$sync->updateDbCounters($otpParams))
if (!$sync->queue($otpParams, $localParams)) if (!$sync->queue($otpParams, $localParams))
{ {
$myLog->log(LOG_CRIT, 'ykval-verify:critical:failed to queue sync requests'); $myLog->log(LOG_CRIT, 'failed to queue sync requests');
sendResp(S_BACKEND_ERROR, $myLog, $apiKey); sendResp(S_BACKEND_ERROR, $myLog, $apiKey);
} }
@ -392,7 +392,7 @@ else
$sl_success_rate = 0; $sl_success_rate = 0;
} }
$myLog->log(LOG_INFO, "ykval-verify:notice:synclevel=" . $sl . $myLog->log(LOG_INFO, "synclevel=" . $sl .
" nr servers=" . $nr_servers . " nr servers=" . $nr_servers .
" req answers=" . $req_answers . " req answers=" . $req_answers .
" answers=" . $nr_answers . " answers=" . $nr_answers .
@ -405,7 +405,7 @@ if ($syncres == False)
/* sync returned false, indicating that /* sync returned false, indicating that
either at least 1 answer marked OTP as invalid or either at least 1 answer marked OTP as invalid or
there were not enough answers */ there were not enough answers */
$myLog->log(LOG_WARNING, 'ykval-verify:notice:Sync failed'); $myLog->log(LOG_WARNING, 'Sync failed');
if ($nr_valid_answers != $nr_answers) if ($nr_valid_answers != $nr_answers)
sendResp(S_REPLAYED_OTP, $myLog, $apiKey, $extra); sendResp(S_REPLAYED_OTP, $myLog, $apiKey, $extra);