Ticket 50136 - Allow resetting passwords on the CLI
Bug Description: This allows resetting passwords on the CLI for accounts, as well as allowing accounts to self-change their passwords. Fix Description: Add reset and change password functions, and fix a number of issues with non-DM bind in the server, regrissions in tls enable during tests. https://pagure.io/389-ds-base/issue/50136 Author: William Brown <william@blackhats.net.au> Review by: ???
So as this doesn't work on all classes of objects, shouldn't we just add it to Account(DSLDapObject)? The same way we have bind() method only for Account objects.
doesn't work on all classes of objects
bind()
It's not a real issue but just for the information. You can use create_test_user() method for a generic user entry creation.
create_test_user()
Is the secure port already set, and this is a duplicate "set"?
I wish you were sitting next to me yesterday when I had this internal conflict about where to put this function :)
I think you are indeed correct, and I'll move the method.
I knew there was a test_user create method, but I forgot what it was! I'll check to see if I can use it here.
2 new commits added
Update .gitignore to hide extra files
Updated based on your feedback
LGTM! Ack :)
rebased onto 9f433e82e153f8f6ccd6a6d32cd6440fa9beec0c
Pull-Request has been merged by firstyear
389-ds-base is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in 389-ds-base's github repository.
This pull request has been cloned to Github as issue and is available here: - https://github.com/389ds/389-ds-base/issues/3202
If you want to continue to work on the PR, please navigate to the github issue, download the patch from the attachments and file a new pull request.
Thank you for understanding. We apologize for all inconvenience.
Pull-Request has been closed by spichugi
Ticket 50136 - Allow resetting passwords on the CLI