URL: https://github.com/freeipa/freeipa/pull/621
Title: #621: Add --force-password-reset to user_mod in user.py
HonzaCholasta commented:
"""
@redhatrises, do not handle the format yourself, use the `DateTime` param type.
Note that you will need to extend it to correctly interpret the "now" valu
URL: https://github.com/freeipa/freeipa/pull/621
Title: #621: Add --force-password-reset to user_mod in user.py
redhatrises commented:
"""
@HonzaCholasta that's an interesting idea. Most of the time, a password reset
is forced immediately, but that does provide more flexibility. I assume that
URL: https://github.com/freeipa/freeipa/pull/621
Title: #621: Add --force-password-reset to user_mod in user.py
HonzaCholasta commented:
"""
I have given this some thought over the night - maybe we should make the option
more generic and allow the user to specify the expiration time rather than
URL: https://github.com/freeipa/freeipa/pull/621
Title: #621: Add --force-password-reset to user_mod in user.py
abbra commented:
"""
Ok, let's go with `user-mod` as original request goes, based on the fact that
we are not changing the password, we are changing its properties.
LGTM.
"""
See th
URL: https://github.com/freeipa/freeipa/pull/621
Title: #621: Add --force-password-reset to user_mod in user.py
redhatrises commented:
"""
Okay, so since it will reside in one location, should it be `user-mod` (PR
already uses `user-mod`) or `passwd`?
"""
See the full comment at
https://githu
URL: https://github.com/freeipa/freeipa/pull/621
Title: #621: Add --force-password-reset to user_mod in user.py
HonzaCholasta commented:
"""
Actually, maybe `user-mod` is a better place for the option, as it does LDAP
modify operation, whereas `passwd` does LDAP password change extended operati
URL: https://github.com/freeipa/freeipa/pull/621
Title: #621: Add --force-password-reset to user_mod in user.py
HonzaCholasta commented:
"""
I don't agree. There should be one and only one obvious way to do it. There is
no real benefit in having this in multiple different places, it just adds
URL: https://github.com/freeipa/freeipa/pull/621
Title: #621: Add --force-password-reset to user_mod in user.py
HonzaCholasta commented:
"""
I don't agree. There should be one and only one obvious way to do it. There is
no real benefit in having this in multiple different places, it just adds
URL: https://github.com/freeipa/freeipa/pull/621
Title: #621: Add --force-password-reset to user_mod in user.py
abbra commented:
"""
Hm. `ipa user-mod` has --random and also supports specifying --password, so
yes, both interfaces should be provided.
"""
See the full comment at
https://github.
URL: https://github.com/freeipa/freeipa/pull/621
Title: #621: Add --force-password-reset to user_mod in user.py
redhatrises commented:
"""
@abbra why not have it in both `ipa user-mod` and `ipa passwd`?
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/621#issuecomment-287761
URL: https://github.com/freeipa/freeipa/pull/621
Title: #621: Add --force-password-reset to user_mod in user.py
abbra commented:
"""
I would prefer this to be an option in `ipa passwd`, e.g. `ipa passwd
--force-reset` which instead of modifying a user password would modify
krbPasswordExpiratio
11 matches
Mail list logo