Merge pull request #169 from warcooft/patch-oauth

datamweb · web-flow · commit 2b1672e23245 · 2024-09-09T10:17:50.000+03:30
fix: ensure ban check for existing users
diff --git a/src/Controllers/OAuthController.php b/src/Controllers/OAuthController.php
@@ -111,7 +111,7 @@ public function callBack(): RedirectResponse
             $users->addToDefaultGroup($user);
         }
 
-        if ($this->userExist->isBanned()) {
+        if ($this->userExist && $this->userExist->isBanned()) {
             return redirect()->to(config('Auth')->logoutRedirect())->with('error', $this->userExist->getBanMessage() ?? lang('Auth.bannedUser'));
         }
 

Original file line number	Diff line number	Diff line change
`@@ -111,7 +111,7 @@ public function callBack(): RedirectResponse`
`111`	`111`	`$users->addToDefaultGroup($user);`
`112`	`112`	`}`
`113`	`113`
`114`		`- if ($this->userExist->isBanned()) {`
	`114`	`+ if ($this->userExist && $this->userExist->isBanned()) {`
`115`	`115`	`return redirect()->to(config('Auth')->logoutRedirect())->with('error', $this->userExist->getBanMessage() ?? lang('Auth.bannedUser'));`
`116`	`116`	`}`
`117`	`117`