mirror of
https://github.com/Yubico/yubikey-val.git
synced 2025-01-20 19:52:15 +01:00
Don't LOG_NOTICE when remote server has seen latest OTP too.
This is an expected condition for many (most) validation requests using the asynchronous validation protocol 2.0.
This commit is contained in:
parent
afb42d7c0a
commit
2a7a6e3503
@ -328,8 +328,12 @@ class SyncLib
|
|||||||
|
|
||||||
/* Retrieve info from entry info string */
|
/* Retrieve info from entry info string */
|
||||||
|
|
||||||
|
/* This is the counter values we had in our database *before* processing the current OTP. */
|
||||||
$validationParams=$this->localParamsFromInfoString($entry['info']);
|
$validationParams=$this->localParamsFromInfoString($entry['info']);
|
||||||
|
/* This is the data from the current OTP. */
|
||||||
$otpParams=$this->otpParamsFromInfoString($entry['info']);
|
$otpParams=$this->otpParamsFromInfoString($entry['info']);
|
||||||
|
|
||||||
|
/* Fetch current information from our database */
|
||||||
$localParams=$this->getLocalParams($otpParams['yk_publicname']);
|
$localParams=$this->getLocalParams($otpParams['yk_publicname']);
|
||||||
|
|
||||||
$this->log(LOG_DEBUG, "validation params: ", $validationParams);
|
$this->log(LOG_DEBUG, "validation params: ", $validationParams);
|
||||||
@ -342,7 +346,11 @@ class SyncLib
|
|||||||
}
|
}
|
||||||
|
|
||||||
if ($this->countersHigherThan($resParams, $validationParams)) {
|
if ($this->countersHigherThan($resParams, $validationParams)) {
|
||||||
$this->log(LOG_NOTICE, "Local server out of sync compared to counters at validation request time. ");
|
if ($this->countersEqual($resParams, $otpParams)) {
|
||||||
|
$this->log(LOG_INFO, "Remote server had received the current counter values already. ");
|
||||||
|
} else {
|
||||||
|
$this->log(LOG_NOTICE, "Local server out of sync compared to counters at validation request time. ");
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if ($this->countersHigherThan($localParams, $resParams)) {
|
if ($this->countersHigherThan($localParams, $resParams)) {
|
||||||
|
Loading…
x
Reference in New Issue
Block a user