#50143 Ticket 50136 - Allow resetting passwords on the CLI
Closed by spichugi. Opened by firstyear.
firstyear/389-ds-base 50136-lib389-user-password  into  master

Download 50143.patch

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.

It's not a real issue but just for the information.
You can use create_test_user() method for a generic user entry creation.

Is the secure port already set, and this is a duplicate "set"?

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.

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.

It's not a real issue but just for the information.
You can use create_test_user() method for a generic user entry creation.

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

  • Ticket 50136 - Allow resetting passwords on the CLI
  • 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

Metadata