feat(oauth2): fix check null value, added more user_info field

nextcloud · Feb 21, 2024 · 966da08 · 966da08
1 parent 48592c2
commit 966da08
Showing 1 changed file with 18 additions and 7 deletions.
diff --git a/apps/oauth2/lib/Controller/OauthApiController.php b/apps/oauth2/lib/Controller/OauthApiController.php
@@ -237,12 +237,23 @@ public function getToken(
 	 */
 	public function getUserInfo() {
 		$user = $this->userSession->getUser();
-		$displayname = explode(' ', $user->getDisplayName());
-		return new JSONResponse([
-			'sub' => $user->getUID(),
-			'given_name' => $displayname[0],
-			'family_name' => $displayname[1] ? $displayname[1] : $displayname[0],
-			'email' => $user->getEMailAddress()
-		]);
+		if ($user) {
+			$displayName = $user->getDisplayName();
+			$partedName = explode(' ', $displayName);
+			$userId = $user->getUID();
+			$response = new JSONResponse([
+				'sub' => $userId,
+				'name' => $displayName,
+				'given_name' => $partedName[0],
+				'family_name' => $partedName[1] ?? $partedName[0],
+				'email' => $user->getEMailAddress(),
+				'picture' => \OC::$server->getURLGenerator()->getAbsoluteURL("index.php/avatar/$userId/512"),
+			]);
+		}else{
+			$response = new JSONResponse([
+				'error' => 'user_not_found',
+			], Http::STATUS_NOT_FOUND);
+		}
+		return $response;
 	}
 }