User-created Machine Accounts are not parsed properly as Computers

[0xb50d]

User-created machine accounts (e.g. with new-adcomputer cmdlet) are not included in the parsed output from AD Explorer snapshot. Per source code, it appears the object is not parsed correctly due to a lack of the "dnshostname" attribute for Computer objects created this way.

ADExplorerSnapshot.py/adexpsnapshot/__init__.py

Line 205 in 590b947

dnshostname = ADUtils.get_entry_property(obj, 'dNSHostname')

To replicate :

• Create an object with "New-ADComputer" (notice the dnshostname attribute is missing)
• Open in ADExplorer, create snapshot
• Run parser, check the computer.json file

[c3c]

Looking at this, computers without dnshostname are indeed being excluded by

ADExplorerSnapshot.py/adexpsnapshot/__init__.py

Lines 299 to 301 in 590b947

	hostname = ADUtils.get_entry_property(entry, 'dNSHostName')
	if not hostname:
	return

Based on how bloodhound.py does this per https://github.com/fox-it/BloodHound.py/blob/master/bloodhound/ad/computer.py#L82-L85, I'm now using resolve_ad_entry to fill in the hostname.

Commit 9fc8412 should hopefully fix this; closing for now.