fix: replace BUILTIN container with domain node - Githubissues

BloodHoundAD / SharpHoundCommon

Common library used by SharpHound.

GNU General Public License v3.0

73 stars 47 forks source link

fix: replace BUILTIN container with domain node #117

Closed mistahj67 closed 3 months ago

mistahj67 commented 3 months ago

Description

Add hardcoded edge case for BUILTIN container to present to BH as domain instead.

Motivation and Context

This will fix the missing Contains edges that are tied to the BUILTIN container but are otherwise ignored.

How Has This Been Tested?

Added case to unit testing

Types of changes

[ ] Chore (a change that does not modify the application functionality)
[x] 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.
[x] All new and existing tests passed.
[ ] My changes include a database migration.