On 03/22/2016 04:32 PM, Abhijeet Kasurde wrote:
Hi All,

Please find the updated patches as per review comments.

On 03/18/2016 07:39 PM, Petr Vobornik wrote:
On 03/18/2016 02:21 PM, Abhijeet Kasurde wrote:
Hi All,

Please review these patches.

Fixes :

Abhijeet Kasurde

'invalid' is a default and right now is meant for invalid password(not correct, see below). So by reading the patch, it will break the case when user sets invalid password.

Better would be to process kinit output in and set e.g: 'krbprincipal-expired' reason.

Then add it to a list of known errors in ipa.js:login_password:498. We should probaly add also 'invalid-password' to the list.

Then do the change as in this patch but only with: 'krbprincipal-expired'.

If 'invalid-password' is added to the list of know errors then we should change the default error from "The password or username you entered is incorrect. " to e.g.: 'Login failed from unknown reason"

Thanks Petr for suggestions.

Abhijeet Kasurde

Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA:

Reply via email to