BloodHoundAD / SharpHoundCommon

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

Bed-4487: fix handling for unicode control characters #141

Closed mistahj67 closed 3 months ago

mistahj67 commented 3 months ago

Description

Closes BED-4487

Add protection around control characters such as null unicode \u0000by base64 encoding any properties we encounter.

Motivation and Context

Running the collection with --collectallproperties brings in properties with binary control characters that cause issues downstream during ingestion into db's.

How Has This Been Tested?

Added unit tests Tested on AD lab

Types of changes

Checklist: