Update RegisterController

pixelfed · Jun 20, 2023 · a6d10f0 · a6d10f0
1 parent 89c3710
commit a6d10f0
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 7 deletions.
diff --git a/app/Http/Controllers/Auth/RegisterController.php b/app/Http/Controllers/Auth/RegisterController.php
@@ -178,8 +178,9 @@ public function showRegistrationForm()
 			if(config('pixelfed.bouncer.cloud_ips.ban_signups')) {
 				abort_if(BouncerService::checkIp(request()->ip()), 404);
 			}
-			$limit = config('pixelfed.max_users');
-			if($limit) {
+			$hasLimit = config('pixelfed.enforce_max_users');
+			if($hasLimit) {
+				$limit = config('pixelfed.max_users');
 				$count = User::where(function($q){ return $q->whereNull('status')->orWhereNotIn('status', ['deleted','delete']); })->count();
 				if($limit <= $count) {
 					return redirect(route('help.instance-max-users-limit'));
@@ -208,13 +209,17 @@ public function register(Request $request)
 			abort_if(BouncerService::checkIp($request->ip()), 404);
 		}
 
-		$count = User::where(function($q){ return $q->whereNull('status')->orWhereNotIn('status', ['deleted','delete']); })->count();
-		$limit = config('pixelfed.max_users');
+		$hasLimit = config('pixelfed.enforce_max_users');
+		if($hasLimit) {
+			$count = User::where(function($q){ return $q->whereNull('status')->orWhereNotIn('status', ['deleted','delete']); })->count();
+			$limit = config('pixelfed.max_users');
 
-		if(false == config_cache('pixelfed.open_registration') || $limit && $limit <= $count) {
-			return redirect(route('help.instance-max-users-limit'));
+    		if($limit && $limit <= $count) {
+    			return redirect(route('help.instance-max-users-limit'));
+    		}
 		}
 
+
 		$this->validator($request->all())->validate();
 
 		event(new Registered($user = $this->create($request->all())));

diff --git a/config/pixelfed.php b/config/pixelfed.php
@@ -198,7 +198,8 @@
 	| Allow a maximum number of user accounts. Default: off
 	|
 	*/
-	'max_users' => env('PF_MAX_USERS', false),
+    'max_users' => env('PF_MAX_USERS', 1000),
+    'enforce_max_users' => env('PF_ENFORCE_MAX_USERS', true),
 
 	/*
 	|--------------------------------------------------------------------------