Check password requirements on login - #427
Conversation
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
|
What does this do when I try to grab a token to make API calls? If it denies the token, we need to work through that. |
Nothing in the above limits logins, it just returns the flag that the password needs to be changed in the response to the login. Exactly the same as an "expired" password. |
@mgaffigan are you sure? wont this affect a few things:
|
Changing password requirements does not affect current users. This PR modifies the login process to check requirements and handle them the same as password expiration, using the existing grace period functionality.
Note:
commanddoes not seem to handle expired passwords well.Note to reviewers: Most of this change-set is whitespace changes due to indentation. See https://github.com/OpenIntegrationEngine/engine/pull/427/changes?w=1