BloodHoundAD / SharpHound2

The Old BloodHound C# Ingestor (Deprecated)
509 stars 113 forks source link

Make sure that Computer.Name is always full uppercase #54

Closed cnotin closed 5 years ago

cnotin commented 5 years ago

My situation is the following:

What I see is:

What I expected is that both computer objects were merged since it's actually the same.

My suggestion is to make sure that Computer.Name is always full uppercase by two ways:

I've tested successfully the patch.

rvazarkar commented 5 years ago

I'll accept this, although I thought I had logic in the upload functionality to fix this anyways. I guess not. More redundancy is better anyways.