search

harryhorton / node-nmap

NPM package for interfacing with local NMAP installation
MIT License
67 stars 26 forks source link

Breaks on Linux / OS X when no hostname is present. #9

Closed tstrimple closed 8 years ago

tstrimple commented 8 years ago

The scan breaks due to this line:

https://github.com/Johnhhorton/node-nmap/blob/master/index.js#L124

Non-Windows operating systems return "\n", not "\r\n".

harryhorton commented 8 years ago

Thanks for catching that. I'll check this on a vm and fix it.

harryhorton commented 8 years ago

Added additional check for \n