HMI device no assigned to proper Os.Device

[nstylepro]

recog/xml/snmp_sysdescr.xml

Line 6315 in c2beb75

 <fingerprint pattern="^Siemens, SIMATIC HMI, ([^,]+),.*FW:\s*V([^,]+).*$"> 

This is an HMI (there is one more in the repo) that is 'HMI controller' yet it is under Os.Device='Monitoring'
Is this intentional or bug? if it's intentional please describe why

[HD Moore]

This looks like a bug and the correct value should be HMI Controller to match other entries. Would you like to PR this?

[HD Moore]

I can include this change in my next rollup PR if you prefer (should be in the next few days).

[nstylepro]

I can PR, tried to push my local branch but I do not have appropriate permissions

[HD Moore]

Thanks! Take a look at https://github.com/rapid7/recog/blob/master/CONTRIBUTING.md for instructions. The CPE/normalization stuff is not needed for this small change.

[nstylepro]

#286

[Tom Sellers]

The issue has been addressed in PR #286 and #291