Merge pull request #7979 from Quix0r/fixes/prvKey-verus-prvkey

Fixed prvkey causing 3rd parameter for set() method not set
This commit is contained in:
Hypolite Petovan 2019-12-20 17:23:02 -05:00 committed by GitHub
commit 564952cf1a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -95,14 +95,14 @@ class Authentication
if ($this->dba->isResult($user)) {
if (!$this->cookie->check($data->hash,
$user['password'] ?? '',
$user['prvKey'] ?? '')) {
$user['prvkey'] ?? '')) {
$this->logger->notice("Hash doesn't fit.", ['user' => $data->uid]);
$this->session->delete();
$this->baseUrl->redirect();
}
// Renew the cookie
$this->cookie->set($user['uid'], $user['password'], $user['prvKey']);
$this->cookie->set($user['uid'], $user['password'], $user['prvkey']);
// Do the authentification if not done by now
if (!$this->session->get('authenticated')) {
@ -352,7 +352,7 @@ class Authentication
*/;
if ($this->session->get('remember')) {
$a->getLogger()->info('Injecting cookie for remembered user ' . $user_record['nickname']);
$this->cookie->set($user_record['uid'], $user_record['password'], $user_record['prvKey']);
$this->cookie->set($user_record['uid'], $user_record['password'], $user_record['prvkey']);
$this->session->remove('remember');
}
}