openinfra/openstackid
Code Issues Proposed changes

Fixed null pointer exception

Browse Source 
Change-Id: I69b99548696c020ef7f10e2a4b98af66b53408e0
Signed-off-by: smarcet <smarcet@gmail.com>
This commit is contained in:
smarcet 2020-03-23 22:33:28 -03:00
parent bc3b420d74
commit f7a4697b35
1 changed files with 13 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
app/libs/Auth/Models/User.php
View File

@ -20,6 +20,7 @@ use Illuminate\Support\Facades\Event;
use App\Events\UserEmailVerified;
use Doctrine\Common\Collections\Criteria;
use Illuminate\Auth\Authenticatable;
use Illuminate\Support\Facades\Log;
use models\exceptions\ValidationException;
use Models\OAuth2\ApiScope;
use Models\OAuth2\Client;
@ -804,6 +805,18 @@ class User extends BaseEntity
*/
public function checkPassword(string $password): bool
{
if(empty($this->password))
{
Log::warning(sprintf("User %s (%s) has not password set.", $this->id, $this->email));
return false;
}
if(empty($this->password_enc))
{
Log::warning(sprintf("User %s (%s) has not password encoding set.", $this->id, $this->email));
return false;
}
return AuthHelper::check($password, $this->password, $this->password_enc, $this->password_salt);
}