Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adding enable_user method to enable user #115

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wrighterase
Copy link

No description provided.

@tiyeuse
Copy link
Collaborator

tiyeuse commented Nov 10, 2024

Thanks for the pull request, but we can't modify the UAC attribute of an account with the hardcoded value of 512 (which stands for NORMAL_ACCOUNT). In case other attributes (such as DONT_EXPIRE_PASSWORD) are defined, they will be removed.
A bitmask operation needs to be performed in order to swap the ACCOUNTDISABLE attribute.
I will look at the changes, but meanwhile you can still use the change_uac function with the uac-decoder website to make it work:

# ldeep ldap change_uac -h
[...]
  user        Target user (dn format). Ex: "CN=bob,CN=Users,DC=CORP,DC=LOCAL"
  uac         UAC integer value. Ex: 512 for NORMAL_ACCOUNT

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants