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

new module: ldap > dump-computers #556

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

crosscutsaw
Copy link

Description

i'm doing a lot of local infrastructure and active directory pentests. during my engagements, i always needing all computers of a domain. so i can first eliminate active directory then scan complete infrastructure with nessus. i was pulling bloodhound data then manipulating text so i retrieve domain computers. it's working, but, why can't i do it with precious netexec? that's why i created dump-computers module.

it's my first pr so please forgive my mistakes.

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

i tested it with a dummy domain, domain controller and two clients.

Screenshots (if appropriate):

dc1
dc2
dco1
dco2

Checklist:

  • I have ran Ruff against my changes (via poetry: poetry run python -m ruff check . --preview, use --fix to automatically fix what it can)
  • I have added or updated the tests/e2e_commands.txt file if necessary
  • New and existing e2e tests pass locally with my changes
  • My code follows the style guidelines of this project (should be covered by Ruff above)
  • If reliant on third party dependencies, such as Impacket, dploot, lsassy, etc, I have linked the relevant PRs in those projects
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (PR here: https://github.com/Pennyw0rth/NetExec-Wiki)

added fqdn only
@crosscutsaw
Copy link
Author

25ac2e2

added only fqdn.

image

made some adjustments for better experience
@crosscutsaw
Copy link
Author

fa3eba4

sorry for keep updating this pr. i made some adjustments for better usage.

image

@NeffIsBack
Copy link
Contributor

sorry for keep updating this pr. i made some adjustments for better usage.

No worries about that :)

Thanks for the PR! I will take a look at it as soon as i have some time.

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

Successfully merging this pull request may close these issues.

2 participants