From 7d83321286efc37162521f4302cf17f03c317020 Mon Sep 17 00:00:00 2001 From: Alexandre Alapetite Date: Sat, 14 Aug 2021 12:22:42 +0200 Subject: Better catch when a user does not exists (#3751) #fix https://github.com/FreshRSS/FreshRSS/issues/3735 Before, we were relying on an exception during the first stages of user initalisation. Now the check is explicit and cleaner, producing a more appropriate HTTP response for the API. --- app/Controllers/userController.php | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'app/Controllers/userController.php') diff --git a/app/Controllers/userController.php b/app/Controllers/userController.php index 0b23ff953..81f8b20a8 100644 --- a/app/Controllers/userController.php +++ b/app/Controllers/userController.php @@ -14,6 +14,10 @@ class FreshRSS_user_Controller extends Minz_ActionController { return preg_match('/^' . self::USERNAME_PATTERN . '$/', $username) === 1; } + public static function userExists($username) { + return @file_exists(USERS_PATH . '/' . $username . '/config.php'); + } + public static function updateUser($user, $email, $passwordPlain, $userConfigUpdated = array()) { $userConfig = get_user_configuration($user); if ($userConfig === null) { -- cgit v1.2.3