diff --git a/app/controllers/api/account.php b/app/controllers/api/account.php index e7c90acf96..d8be565c8f 100644 --- a/app/controllers/api/account.php +++ b/app/controllers/api/account.php @@ -1803,7 +1803,7 @@ App::get('/v1/account/sessions/oauth2/:provider/redirect') 'userAgent' => $request->getUserAgent('UNKNOWN'), 'ip' => $request->getIP(), 'factors' => [TYPE::EMAIL, 'oauth2'], // include a special oauth2 factor to bypass MFA checks - 'countryCode' => $geoRecord['countryCode'], + 'countryCode' => $geoRecord['countryCode'] ?? '', 'expire' => DateTime::addSeconds(new \DateTime(), $duration) ], $detector->getOS(), $detector->getClient(), $detector->getDevice())); @@ -2944,8 +2944,8 @@ App::get('/v1/account/logs') $detector->getDevice() )); - $output[$i]['countryCode'] = $geoRecord['countryCode']; - $output[$i]['countryName'] = $geoRecord['countryName']; + $output[$i]['countryCode'] = $geoRecord['countryCode'] ?? ''; + $output[$i]['countryName'] = $geoRecord['countryName'] ?? ''; } $response->dynamic(new Document([ diff --git a/app/controllers/api/messaging.php b/app/controllers/api/messaging.php index 7c7b753f52..05dc05420f 100644 --- a/app/controllers/api/messaging.php +++ b/app/controllers/api/messaging.php @@ -1199,8 +1199,8 @@ App::get('/v1/messaging/providers/:providerId/logs') 'deviceModel' => $device['deviceModel'] ]); - $output[$i]['countryCode'] = $geoRecord['countryCode']; - $output[$i]['countryName'] = $geoRecord['countryName']; + $output[$i]['countryCode'] = $geoRecord['countryCode'] ?? ''; + $output[$i]['countryName'] = $geoRecord['countryName'] ?? ''; } $response->dynamic(new Document([ @@ -2601,8 +2601,8 @@ App::get('/v1/messaging/topics/:topicId/logs') 'deviceModel' => $device['deviceModel'] ]); - $output[$i]['countryCode'] = $geoRecord['countryCode']; - $output[$i]['countryName'] = $geoRecord['countryName']; + $output[$i]['countryCode'] = $geoRecord['countryCode'] ?? ''; + $output[$i]['countryName'] = $geoRecord['countryName'] ?? ''; } $response->dynamic(new Document([ @@ -3016,8 +3016,8 @@ App::get('/v1/messaging/subscribers/:subscriberId/logs') 'deviceModel' => $device['deviceModel'] ]); - $output[$i]['countryCode'] = $geoRecord['countryCode']; - $output[$i]['countryName'] = $geoRecord['countryName']; + $output[$i]['countryCode'] = $geoRecord['countryCode'] ?? ''; + $output[$i]['countryName'] = $geoRecord['countryName'] ?? ''; } $response->dynamic(new Document([ @@ -3812,8 +3812,8 @@ App::get('/v1/messaging/messages/:messageId/logs') 'deviceModel' => $device['deviceModel'] ]); - $output[$i]['countryCode'] = $geoRecord['countryCode']; - $output[$i]['countryName'] = $geoRecord['countryName']; + $output[$i]['countryCode'] = $geoRecord['countryCode'] ?? ''; + $output[$i]['countryName'] = $geoRecord['countryName'] ?? ''; } diff --git a/app/controllers/api/teams.php b/app/controllers/api/teams.php index cc1306d956..03e62b2e4a 100644 --- a/app/controllers/api/teams.php +++ b/app/controllers/api/teams.php @@ -1275,7 +1275,7 @@ App::patch('/v1/teams/:teamId/memberships/:membershipId/status') 'userAgent' => $request->getUserAgent('UNKNOWN'), 'ip' => $request->getIP(), 'factors' => ['email'], - 'countryCode' => $geoRecord['countryCode'], + 'countryCode' => $geoRecord['countryCode'] ?? '', 'expire' => DateTime::addSeconds(new \DateTime(), $authDuration) ], $detector->getOS(), $detector->getClient(), $detector->getDevice())); @@ -1513,8 +1513,8 @@ App::get('/v1/teams/:teamId/logs') 'deviceModel' => $device['deviceModel'] ]); - $output[$i]['countryCode'] = $geoRecord['countryCode']; - $output[$i]['countryName'] = $geoRecord['countryName']; + $output[$i]['countryCode'] = $geoRecord['countryCode'] ?? ''; + $output[$i]['countryName'] = $geoRecord['countryName'] ?? ''; } $response->dynamic(new Document([ 'total' => $includeTotal ? $audit->countLogsByResource($resource, $queries) : 0, diff --git a/app/controllers/api/users.php b/app/controllers/api/users.php index ac2f6a8522..833c2a8979 100644 --- a/app/controllers/api/users.php +++ b/app/controllers/api/users.php @@ -969,8 +969,8 @@ App::get('/v1/users/:userId/logs') 'deviceModel' => $device['deviceModel'] ]); - $output[$i]['countryCode'] = $geoRecord['countryCode']; - $output[$i]['countryName'] = $geoRecord['countryName']; + $output[$i]['countryCode'] = $geoRecord['countryCode'] ?? ''; + $output[$i]['countryName'] = $geoRecord['countryName'] ?? ''; } $response->dynamic(new Document([ @@ -2210,7 +2210,7 @@ App::post('/v1/users/:userId/sessions') 'userAgent' => $request->getUserAgent('UNKNOWN'), 'factors' => ['server'], 'ip' => $request->getIP(), - 'countryCode' => $geoRecord['countryCode'], + 'countryCode' => $geoRecord['countryCode'] ?? '', 'expire' => $expire, ], $detector->getOS(), diff --git a/app/controllers/general.php b/app/controllers/general.php index be447f1f9b..94fb9b7a79 100644 --- a/app/controllers/general.php +++ b/app/controllers/general.php @@ -383,7 +383,7 @@ function router(App $utopia, Database $dbForPlatform, callable $getProjectDB, Sw $headers['x-appwrite-country-code'] = $geoRecord['countryCode'] ?? ''; $headers['x-appwrite-continent-code'] = $geoRecord['continentCode'] ?? ''; - $headers['x-appwrite-continent-eu'] = (\in_array($geoRecord['countryCode'], $eu)) ? 'true' : 'false'; + $headers['x-appwrite-continent-eu'] = (\in_array(($geoRecord['countryCode'] ?? ''), $eu)) ? 'true' : 'false'; } } diff --git a/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Logs/XList.php b/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Logs/XList.php index b0ee11d77a..f0123343ff 100644 --- a/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Logs/XList.php +++ b/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Documents/Logs/XList.php @@ -146,8 +146,8 @@ class XList extends Action 'deviceModel' => $device['deviceModel'] ]); - $output[$i]['countryCode'] = $geoRecord['countryCode']; - $output[$i]['countryName'] = $geoRecord['countryName']; + $output[$i]['countryCode'] = $geoRecord['countryCode'] ?? ''; + $output[$i]['countryName'] = $geoRecord['countryName'] ?? ''; } $response->dynamic(new Document([ diff --git a/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Logs/XList.php b/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Logs/XList.php index e3770e6043..bada9f1bc5 100644 --- a/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Logs/XList.php +++ b/src/Appwrite/Platform/Modules/Databases/Http/Databases/Collections/Logs/XList.php @@ -22,7 +22,6 @@ use Utopia\Database\Validator\Queries; use Utopia\Database\Validator\Query\Limit; use Utopia\Database\Validator\Query\Offset; use Utopia\Database\Validator\UID; -use Utopia\Locale\Locale; use Utopia\Swoole\Response as SwooleResponse; class XList extends Action diff --git a/src/Appwrite/Platform/Modules/Databases/Http/Databases/Logs/XList.php b/src/Appwrite/Platform/Modules/Databases/Http/Databases/Logs/XList.php index 7f9143176d..940ef61c72 100644 --- a/src/Appwrite/Platform/Modules/Databases/Http/Databases/Logs/XList.php +++ b/src/Appwrite/Platform/Modules/Databases/Http/Databases/Logs/XList.php @@ -127,8 +127,8 @@ class XList extends Action 'deviceModel' => $device['deviceModel'], ]); - $output[$i]['countryCode'] = $geoRecord['countryCode']; - $output[$i]['countryName'] = $geoRecord['countryName']; + $output[$i]['countryCode'] = $geoRecord['countryCode'] ?? ''; + $output[$i]['countryName'] = $geoRecord['countryName'] ?? ''; } $response->dynamic(new Document([ diff --git a/src/Appwrite/Platform/Modules/Databases/Http/TablesDB/Logs/XList.php b/src/Appwrite/Platform/Modules/Databases/Http/TablesDB/Logs/XList.php index a34e9b24f5..1cbee80150 100644 --- a/src/Appwrite/Platform/Modules/Databases/Http/TablesDB/Logs/XList.php +++ b/src/Appwrite/Platform/Modules/Databases/Http/TablesDB/Logs/XList.php @@ -122,8 +122,8 @@ class XList extends Action 'deviceModel' => $device['deviceModel'], ]); - $output[$i]['countryCode'] = $geoRecord['countryCode']; - $output[$i]['countryName'] = $geoRecord['countryName']; + $output[$i]['countryCode'] = $geoRecord['countryCode'] ?? ''; + $output[$i]['countryName'] = $geoRecord['countryName'] ?? ''; } $response->dynamic(new Document([ diff --git a/src/Appwrite/Platform/Modules/Functions/Http/Executions/Create.php b/src/Appwrite/Platform/Modules/Functions/Http/Executions/Create.php index 7aee851798..2083f7c0ca 100644 --- a/src/Appwrite/Platform/Modules/Functions/Http/Executions/Create.php +++ b/src/Appwrite/Platform/Modules/Functions/Http/Executions/Create.php @@ -239,7 +239,7 @@ class Create extends Base $headers['x-appwrite-country-code'] = $geoRecord['countryCode'] ?? ''; $headers['x-appwrite-continent-code'] = $geoRecord['continentCode'] ?? ''; - $headers['x-appwrite-continent-eu'] = (\in_array($geoRecord['countryCode'], $eu)) ? 'true' : 'false'; + $headers['x-appwrite-continent-eu'] = (\in_array(($geoRecord['countryCode'] ?? ''), $eu)) ? 'true' : 'false'; } }