Show user account on grant loginflow step

Signed-off-by: Joas Schilling <coding@schilljs.com>

diff --git a/core/Controller/ClientFlowLoginController.php b/core/Controller/ClientFlowLoginController.php
index d67a065a14efff2a3a2cd4f47e798022e18df18f..6a88705c6e475c9e3c6c7ace2dd1ddee48e3b25e 100644 (file)
--- a/core/Controller/ClientFlowLoginController.php
+++ b/core/Controller/ClientFlowLoginController.php
@@ -49,6 +49,7 @@ use OCP\IL10N;
 use OCP\IRequest;
 use OCP\ISession;
 use OCP\IURLGenerator;
+use OCP\IUser;
 use OCP\IUserSession;
 use OCP\Security\ICrypto;
 use OCP\Security\ISecureRandom;
@@ -250,10 +251,15 @@ class ClientFlowLoginController extends Controller {
                        $csp->addAllowedFormActionDomain('nc://*');
                }
 
+               /** @var IUser $user */
+               $user = $this->userSession->getUser();
+
                $response = new StandaloneTemplateResponse(
                        $this->appName,
                        'loginflow/grant',
                        [
+                               'userId' => $user->getUID(),
+                               'userDisplayName' => $user->getDisplayName(),
                                'client' => $clientName,
                                'clientIdentifier' => $clientIdentifier,
                                'instanceName' => $this->defaults->getName(),

diff --git a/core/Controller/ClientFlowLoginV2Controller.php b/core/Controller/ClientFlowLoginV2Controller.php
index 205c1ff8a1c7496c8d72f769711b27b1a48ec722..f2b9399301f4218998f001f7ce8b80a77660c273 100644 (file)
--- a/core/Controller/ClientFlowLoginV2Controller.php
+++ b/core/Controller/ClientFlowLoginV2Controller.php
@@ -41,6 +41,8 @@ use OCP\IL10N;
 use OCP\IRequest;
 use OCP\ISession;
 use OCP\IURLGenerator;
+use OCP\IUser;
+use OCP\IUserSession;
 use OCP\Security\ISecureRandom;
 
 class ClientFlowLoginV2Controller extends Controller {
@@ -53,6 +55,8 @@ class ClientFlowLoginV2Controller extends Controller {
        private $urlGenerator;
        /** @var ISession */
        private $session;
+       /** @var IUserSession */
+       private $userSession;
        /** @var ISecureRandom */
        private $random;
        /** @var Defaults */
@@ -67,6 +71,7 @@ class ClientFlowLoginV2Controller extends Controller {
                                                                LoginFlowV2Service $loginFlowV2Service,
                                                                IURLGenerator $urlGenerator,
                                                                ISession $session,
+                                                               IUserSession $userSession,
                                                                ISecureRandom $random,
                                                                Defaults $defaults,
                                                                ?string $userId,
@@ -75,6 +80,7 @@ class ClientFlowLoginV2Controller extends Controller {
                $this->loginFlowV2Service = $loginFlowV2Service;
                $this->urlGenerator = $urlGenerator;
                $this->session = $session;
+               $this->userSession = $userSession;
                $this->random = $random;
                $this->defaults = $defaults;
                $this->userId = $userId;
@@ -160,10 +166,15 @@ class ClientFlowLoginV2Controller extends Controller {
                        return $this->loginTokenForbiddenResponse();
                }
 
+               /** @var IUser $user */
+               $user = $this->userSession->getUser();
+
                return new StandaloneTemplateResponse(
                        $this->appName,
                        'loginflowv2/grant',
                        [
+                               'userId' => $user->getUID(),
+                               'userDisplayName' => $user->getDisplayName(),
                                'client' => $flow->getClientName(),
                                'instanceName' => $this->defaults->getName(),
                                'urlGenerator' => $this->urlGenerator,

diff --git a/core/templates/loginflow/grant.php b/core/templates/loginflow/grant.php
index c537c47ea648807e12bed4f3a5427012bb71a118..04fdced1c62b37e5a9d7e0af21a58531aa418a59 100644 (file)
--- a/core/templates/loginflow/grant.php
+++ b/core/templates/loginflow/grant.php
@@ -29,6 +29,12 @@ $urlGenerator = $_['urlGenerator'];
 
 <div class="picker-window">
        <h2><?php p($l->t('Account access')) ?></h2>
+       <p class="info">
+               <?php p($l->t('Currently logged in as %1$s (%2$s).', [
+                       $_['userDisplayName'],
+                       $_['userId'],
+               ])) ?>
+       </p>
        <p class="info">
                <?php print_unescaped($l->t('You are about to grant %1$s access to your %2$s account.', [
                        '<strong>' . \OCP\Util::sanitizeHTML($_['client']) . '</strong>',
@@ -44,7 +50,7 @@ $urlGenerator = $_['urlGenerator'];
                        <input type="hidden" name="requesttoken" value="<?php p($_['requesttoken']) ?>" />
                        <input type="hidden" name="stateToken" value="<?php p($_['stateToken']) ?>" />
                        <input type="hidden" name="oauthState" value="<?php p($_['oauthState']) ?>" />
-                       <?php if (p($_['direct'])) { ?>
+                       <?php if ($_['direct']) { ?>
                        <input type="hidden" name="direct" value="1" />
                        <?php } ?>
                        <div id="submit-wrapper">

diff --git a/core/templates/loginflowv2/grant.php b/core/templates/loginflowv2/grant.php
index b036d33ad7c926f9249f5175c6ff2f01be57c1ec..19005a20e2c1a00925ded62e74825736cf659f86 100644 (file)
--- a/core/templates/loginflowv2/grant.php
+++ b/core/templates/loginflowv2/grant.php
@@ -29,6 +29,12 @@ $urlGenerator = $_['urlGenerator'];
 
 <div class="picker-window">
        <h2><?php p($l->t('Account access')) ?></h2>
+       <p class="info">
+               <?php p($l->t('Currently logged in as %1$s (%2$s).', [
+                       $_['userDisplayName'],
+                       $_['userId'],
+               ])) ?>
+       </p>
        <p class="info">
                <?php print_unescaped($l->t('You are about to grant %1$s access to your %2$s account.', [
                        '<strong>' . \OCP\Util::sanitizeHTML($_['client']) . '</strong>',
@@ -41,10 +47,10 @@ $urlGenerator = $_['urlGenerator'];
        <p id="redirect-link">
                <form method="POST" action="<?php p($urlGenerator->linkToRouteAbsolute('core.ClientFlowLoginV2.generateAppPassword')) ?>">
                        <input type="hidden" name="requesttoken" value="<?php p($_['requesttoken']) ?>" />
-                       <input type="hidden" name="stateToken" value="<?php p($_['stateToken']) ?>" />  
+                       <input type="hidden" name="stateToken" value="<?php p($_['stateToken']) ?>" />
                        <div id="submit-wrapper">
                                <input type="submit" class="login primary icon-confirm-white" title="" value="<?php p($l->t('Grant access')); ?>" />
-                       </div>  
+                       </div>
                </form>
        </p>
 </div>