Update Processes checkup logic for determining whether process is run as root

[zackattack01]

The processes checkup will fail for windows devices where the username is not returned in English. This is because we do a strict equality check of the process username against "NT AUTHORITY\\SYSTEM", which will not match.

Code is here-

launcher/ee/debug/checkups/processes.go

Lines 68 to 71 in b656a07

	username := pMap["username"].(string)
	if username == "root" || username == "NT AUTHORITY\\SYSTEM" {
	c.kolideRunningAsRoot = true
	}

We are now using the pass/fail status towards our checkup score generation, so should update this check to prevent erroneously reporting failures. We'll just need to figure out a better way to determine whether the process is running with administrative privileges

[zackattack01]

Update: we ended up pulling the processes checkup out of our log checkpoints/checkup scoring and doing a less resource intensive version of process analysis within our performance checkup. So this is less important, but leaving open because it would still be nice to have

[RebeccaMahany]

@zackattack01 do you think we still want this?