fix: Improve domain check logic in SocialiteController callback method

Author: promatik

src/app/Http/Controllers/SocialiteController.php

@@ -25,15 +25,18 @@ public function login(string $driver): RedirectResponse
     public function callback(string $driver): RedirectResponse
     {
         $socialUser = Socialite::driver($driver)->user();
+        $inAllowedDomains = in_array(Str::afterLast($socialUser->getEmail(), '@'), config('gemadigital.auto_admin_domains', []));
 
         // Find user
         $user = User::query()
             ->where('email', $socialUser->getEmail())
             ->first();
 
         if (! $user) {
-            if (! config('gemadigital.registration_open')) {
-                return redirect()->route('login')->with('error', 'Registration is closed');
+            if (! config('gemadigital.registration_open') && ! $inAllowedDomains) {
+                return redirect()
+                    ->route(route(config('gemadigital.routes.list.login', 'backpack.auth.login')))
+                    ->with('error', 'Registration is closed');
             }
 
             $user = User::create([
@@ -57,7 +60,7 @@ public function callback(string $driver): RedirectResponse
         ];
 
         // Check user domain
-        if (in_array(Str::afterLast($user->email, '@'), config('gemadigital.auto_admin_domains', []))) {
+        if ($inAllowedDomains) {
             $user->is_admin = true;
         }