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

Merge Ingestor Tuneup into main #81

Merged
merged 7 commits into from
Feb 2, 2017
Merged

Merge Ingestor Tuneup into main #81

merged 7 commits into from
Feb 2, 2017

Conversation

rvazarkar
Copy link
Contributor

No description provided.

-Combined Export-BloodHoundData and Export-BloodHoundCSV into Get-BloodHound data, new function renamed to Invoke-BloodHound
-Eliminated superfluous usage of the pipeline
-Eliminated PSObjects, migrated to hashtables for results
-Reduced query counts (e.g. combined Get-NetGroup/Get-NetGroupMember logic into a custom query)
-.NET StreamWriters instead of Out-File
-Output is exported as soon as it comes in
-Threading now mandatory/built in for computer enumeration
-'Moving window' introduced for retrieving threaded results before all threads complete
-LocalGroup enumeration tries API and falls back to WinNT enumeration
-PropertiesToLoad used in LDAP to only return needed attributes
-Many of these optimizations adapted from Meatballs1 (#7 (comment))
Trimmed BloodHound.ps1 to only required functions
…erof extracted, instead of other way around.

Removed Get-NetLoggedOn/Get-NetLoggedOnLocal from default collection.
First attempt at distinguishedname->samaccountname resolution for groups
…f local admins/sessions

Added -ComputerName for targeting specific computers
Added -ComputerADSPath to allow for restricting computer enumeration to specific OUs
Added -UserADSPath to allow for restricting user/group enumeration to specific OUs
Fixed primary group name duplication bug
@rvazarkar rvazarkar merged commit 3d86ced into master Feb 2, 2017
@HarmJ0y HarmJ0y deleted the ingestor_tuneup branch May 22, 2017 23:19
porterhau5 pushed a commit to porterhau5/BloodHound that referenced this pull request Oct 22, 2017
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.

5 participants