Skip to content

Commit 03ddfb9

Browse files
ENGCOM-8539: Allow backend login without redirect #30950
- Merge Pull Request #30950 from aapokiiso/magento2:backend-allow-login-without-redirect - Merged commits: 1. df20ae1 2. 62e54bd 3. 760a4e3 4. 73ef59b 5. 217c32c 6. 6e3b5b0
2 parents 1c3837c + 6e3b5b0 commit 03ddfb9

File tree

4 files changed

+8
-7
lines changed

4 files changed

+8
-7
lines changed

app/code/Magento/Backend/App/Action/Plugin/Authentication.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -225,7 +225,8 @@ protected function _redirectIfNeededAfterLogin(\Magento\Framework\App\RequestInt
225225

226226
// Checks, whether secret key is required for admin access or request uri is explicitly set
227227
if ($this->_url->useSecretKey()) {
228-
$requestUri = $this->_url->getUrl('*/*/*', ['_current' => true]);
228+
$requestParts = explode('/', trim($request->getRequestUri(), '/'), 2);
229+
$requestUri = $this->_url->getUrl(array_pop($requestParts));
229230
} elseif ($request) {
230231
$requestUri = $request->getRequestUri();
231232
}

app/code/Magento/Backend/Controller/Adminhtml/Auth/Login.php

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -49,11 +49,10 @@ public function execute()
4949
}
5050

5151
$requestUrl = $this->getRequest()->getUri();
52-
$backendUrl = $this->getUrl('*');
53-
// redirect according to rewrite rule
54-
if ($requestUrl != $backendUrl) {
55-
return $this->getRedirect($backendUrl);
52+
if (!$requestUrl->isValid()) {
53+
return $this->getRedirect($this->getUrl('*'));
5654
}
55+
5756
return $this->resultPageFactory->create();
5857
}
5958

app/code/Magento/Security/Test/Mftf/Test/AdminNavigateWhileUserExpiredTest.xml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,8 @@
4848
<wait time="120" stepKey="waitForUserToExpire"/>
4949
<actionGroup ref="AdminOpenCustomersGridActionGroup" stepKey="navigateToCustomers"/>
5050
<!-- Confirm that user is logged out -->
51-
<seeInCurrentUrl url="{{AdminLoginPage.url}}" stepKey="seeAdminLoginUrl"/>
51+
<comment userInput="Comment is added to preserve the step key for backward compatibility" stepKey="seeAdminLoginUrl"/>
52+
<seeElement selector="{{AdminLoginFormSection.loginBlock}}" stepKey="assertAdminLoginPageIsAvailable"/>
5253

5354
<!-- Delete created user -->
5455
<actionGroup ref="AdminLoginActionGroup" stepKey="loginAsAdmin"/>

app/code/Magento/User/Controller/Adminhtml/User/Save.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ public function execute()
105105
$this->getSecurityCookie()->setLogoutReasonCookie(
106106
\Magento\Security\Model\AdminSessionsManager::LOGOUT_REASON_USER_LOCKED
107107
);
108-
$this->_redirect('adminhtml/*/');
108+
$this->_redirect('*');
109109
} catch (NotificationExceptionInterface $exception) {
110110
$this->messageManager->addErrorMessage($exception->getMessage());
111111
} catch (\Magento\Framework\Exception\AuthenticationException $e) {

0 commit comments

Comments
 (0)