Connections breakdowns - Githubissues

BloodHoundAD / SharpHoundCommon

Common library used by SharpHound.

GNU General Public License v3.0

73 stars 47 forks source link

Connections breakdowns #128

Open definitelynotagoblin opened 2 months ago

definitelynotagoblin commented 2 months ago

Description

A first pass at breaking down monolithic DC connection methods in LDAPUtils

Motivation and Context

Hopefully makes things more legible, testable and maintainable.

How Has This Been Tested?

Will be tested locally

Screenshots (if appropriate):

Types of changes

[x] Chore (a change that does not modify the application functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] Documentation updates are needed, and have been made accordingly.
[ ] I have added and/or updated tests to cover my changes.
[ ] All new and existing tests passed.
[ ] My changes include a database migration.