Merge pull request #107 from franc-pentest/fix

Fix modify_password error handling
franc-pentest · Sep 29, 2024 · a48f3b7 · a48f3b7
2 parents 4b32873 + 8d509fb
commit a48f3b7
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/VERSION b/VERSION
@@ -1 +1 @@
-1.0.66
+1.0.67
diff --git a/ldeep/__main__.py b/ldeep/__main__.py
@@ -1903,14 +1903,12 @@ def action_modify_password(self, kwargs):
         try:
             if self.engine.modify_password(user, curr, new):
                 info("Password of {username} changed".format(username=user))
+            else:
+                error(
+                    f"Unable to change {user}'s password, check domain password policy or privileges"
+                )
         except LdapActiveDirectoryView.ActiveDirectoryLdapException as e:
             error(f"{e}, check sAMAccountName")
-        else:
-            error(
-                "Unable to change {username}'s password, check privileges".format(
-                    username=user
-                )
-            )
 
     def action_add_to_group(self, kwargs):
         """