BloodHoundAD / SharpHoundCommon

Common library used by SharpHound.
GNU General Public License v3.0
73 stars 47 forks source link

fix: create WriteAccountRestrictions edges from DA #112

Closed JonasBK closed 4 months ago

JonasBK commented 4 months ago

Description

We filter out WriteAccountRestrictions if it originates from Domain Admins. DA does have this permission by default on all computer objects, so I assume we did this to avoid creating another edge from Domain Admins to all computer nodes.

It is the only edge where we do that, why it could lead to confusion.

This commit removes this filter.

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

Checklist: