-
Notifications
You must be signed in to change notification settings - Fork 598
Open
Labels
component:authenticationAuthentication flow issuesAuthentication flow issueshelp wantedIndicates that a maintainer wants help on an issue or pull requestIndicates that a maintainer wants help on an issue or pull requestpriority:mediumAffects some users but has workaroundsAffects some users but has workaroundsprovider:keycloakstatus:needs-reproductionNeeds a reproducable exampleNeeds a reproducable exampletype:enhancementIndicates new feature requestsIndicates new feature requests
Description
Currently for Keycloak catching only [div#input-error-username] (https://github.com/Versent/saml2aws/blob/master/pkg/provider/keycloak/keycloak.go#L37) when incorrect username entered. But, we alse need to catch incorrect password error: div#input-error-password. These htmls have different ids for auth errors:
Incorrect username form:
<div class="pf-v5-c-helper-text__item pf-m-error" id="input-error-username">
<span class="pf-v5-c-helper-text__item-text pf-m-error kc-feedback-text">
Invalid username or password.
</span>
</div>
Incorrect password form:
<div class="pf-v5-c-helper-text__item pf-m-error" id="input-error-password">
<span class="pf-v5-c-helper-text__item-text pf-m-error kc-feedback-text">
Invalid username or password.
</span>
</div>
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
component:authenticationAuthentication flow issuesAuthentication flow issueshelp wantedIndicates that a maintainer wants help on an issue or pull requestIndicates that a maintainer wants help on an issue or pull requestpriority:mediumAffects some users but has workaroundsAffects some users but has workaroundsprovider:keycloakstatus:needs-reproductionNeeds a reproducable exampleNeeds a reproducable exampletype:enhancementIndicates new feature requestsIndicates new feature requests