Revert "Add ability to cancel out of second factor authentication pag…

…es to invalidate session and return to primary login" This reverts commit 9c5eb41.
openmrs · Dec 11, 2023 · 833aa1c · 833aa1c
1 parent 9c5eb41
commit 833aa1c
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 9 deletions.
diff --git a/...n/java/org/openmrs/module/authenticationui/page/controller/login/LoginPageController.java b/...n/java/org/openmrs/module/authenticationui/page/controller/login/LoginPageController.java
@@ -40,9 +40,6 @@ public String get(
 		if (Context.isAuthenticated()) {
 			return "redirect:" + authenticationUiConfig.getHomePageUrl(ui);
 		}
-		else {
-			request.getRequest().getSession().invalidate();
-		}
 
 		AuthenticationSession authenticationSession = new AuthenticationSession(request.getRequest(), request.getResponse());
 		List<Location> loginLocations = new ArrayList<>();

diff --git a/omod/src/main/webapp/pages/login/loginSecret.gsp b/omod/src/main/webapp/pages/login/loginSecret.gsp
@@ -22,12 +22,11 @@
                     <input type="hidden" name="question" value="${question}"/>
                     ${ ui.message(question) }:
                 </label>
-                <input id="answer" type="password" name="answer" tabindex="10" placeholder="${ ui.message("authenticationui.loginSecret.secret.placeholder") }"/>
+                <input id="answer" type="password" name="answer" placeholder="${ ui.message("authenticationui.loginSecret.secret.placeholder") }"/>
             </p>
 
             <p>
-                <input id="cancel-button" class="cancel" type="button" tabindex="30" value="${ ui.message("authenticationui.login.cancel") }" onclick="document.location.href='${ui.pageLink("authenticationui", "login/login")}'" />
-                <input id="login-button" class="confirm" type="submit" tabindex="20" value="${ ui.message("authenticationui.loginSecret.button") }"/>
+                <input id="login-button" class="confirm" type="submit" value="${ ui.message("authenticationui.loginSecret.button") }"/>
             </p>
 
         </fieldset>

diff --git a/omod/src/main/webapp/pages/login/loginTotp.gsp b/omod/src/main/webapp/pages/login/loginTotp.gsp
@@ -19,12 +19,11 @@
 
             <p>
                 <label for="code-input">${ ui.message("authenticationui.loginTotp.code") }</label>
-                <input id="code-input" type="text" name="code" tabindex="10" value="" placeholder="${ ui.message("authenticationui.loginTotp.code.placeholder") }"/>
+                <input id="code-input" type="text" name="code" value="" placeholder="${ ui.message("authenticationui.loginTotp.code.placeholder") }"/>
             </p>
 
             <p>
-                <input id="cancel-button" class="cancel" type="button" tabindex="30" value="${ ui.message("authenticationui.login.cancel") }" onclick="document.location.href='${ui.pageLink("authenticationui", "login/login")}'" />
-                <input id="login-button" class="confirm" type="submit" tabindex="20" value="${ ui.message("authenticationui.loginTotp.button") }"/>
+                <input id="login-button" class="confirm" type="submit" value="${ ui.message("authenticationui.loginTotp.button") }"/>
             </p>
 
         </fieldset>