BloodHoundAD / SharpHoundCommon

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

fix: switch to SHA1 and add a try/catch block on acl inheritance hashing #172

Closed rvazarkar closed 1 month ago

rvazarkar commented 1 month ago

Description

MD5 hashing is not FIPS compliant and in environments where compliance is enforced, .net will throw an exception. Switching to SHA1 will solve this issue. Additional try/catch for safety added.

Motivation and Context

https://specterops.atlassian.net/browse/BP-988

How Has This Been Tested?

Tested in GOAD w/ FIPS enforcement enabled

Screenshots (if appropriate):

Types of changes

Checklist: