Reviewed solution as per reviewer's comments

This commit is contained in:
Daniele Martinoli 2023-07-07 10:27:43 +02:00 committed by Pedro Igor
parent d148a789f7
commit 1644432df3
3 changed files with 4 additions and 8 deletions

View file

@ -38,7 +38,6 @@ public class ResetPassword extends AbstractSetRequiredActionAuthenticator {
(context.getExecution().isConditional() && (context.getExecution().isConditional() &&
configuredFor(context))) { configuredFor(context))) {
context.getAuthenticationSession().addRequiredAction(UserModel.RequiredAction.UPDATE_PASSWORD); context.getAuthenticationSession().addRequiredAction(UserModel.RequiredAction.UPDATE_PASSWORD);
context.getAuthenticationSession().setClientNote(Constants.KC_ACTION_EXECUTING, UserModel.RequiredAction.UPDATE_PASSWORD.name());
} }
context.success(); context.success();
} }

View file

@ -125,8 +125,7 @@ public class UpdatePassword implements RequiredActionProvider, RequiredActionFac
return; return;
} }
if (getId().equals(authSession.getClientNote(Constants.KC_ACTION_EXECUTING)) if ("on".equals(formData.getFirst("logout-sessions")))
&& "on".equals(formData.getFirst("logout-sessions")))
{ {
session.sessions().getUserSessionsStream(realm, user) session.sessions().getUserSessionsStream(realm, user)
.filter(s -> !Objects.equals(s.getId(), authSession.getParentSession().getId())) .filter(s -> !Objects.equals(s.getId(), authSession.getParentSession().getId()))

View file

@ -49,11 +49,9 @@
<div class="${properties.kcFormGroupClass!}"> <div class="${properties.kcFormGroupClass!}">
<div id="kc-form-options" class="${properties.kcFormOptionsClass!}"> <div id="kc-form-options" class="${properties.kcFormOptionsClass!}">
<div class="${properties.kcFormOptionsWrapperClass!}"> <div class="${properties.kcFormOptionsWrapperClass!}">
<#if isAppInitiatedAction??> <div class="checkbox">
<div class="checkbox"> <label><input type="checkbox" id="logout-sessions" name="logout-sessions" value="on" checked> ${msg("logoutOtherSessions")}</label>
<label><input type="checkbox" id="logout-sessions" name="logout-sessions" value="on" checked> ${msg("logoutOtherSessions")}</label> </div>
</div>
</#if>
</div> </div>
</div> </div>