Re: http-vhosts.nse ready for beta

[Carlos Pantelides <carlos_pantelides () yahoo com>]

David:

Thank you for your patience! 

I see the point in your critics, agree with all but number 1. Anyway I accepted the changes, because I understand that 
is easier for nmap to deal with one "incorrect" file than deal with three "correct" files. I withdraw the changes to 
unpwdb.lua. But please take a look some time at http://seclists.org/nmap-dev/2010/q4/445


> 1. Embed the list of hostnames inside the script as a big
> array. Remove
>    hostnames.lst and the hostnamesdb script
> argument.

Done


 
> 2. Change the portrule to be
>      portrule = shortport.http
>    Remove the http-vhosts.service argument.

Done

> 3. Don't modify host.targetname. When I comment that out
> and just use
>    the Host header as you have already
> implemented, it works fine.

funny, a forgotten refactoring

> 4. Change the name of the http-vhosts.resource script
> argument to
>    http-vhosts.path.

Done

Carlos Pantelides

Attachment: http-vhosts.nse
Description:

_______________________________________________
Sent through the nmap-dev mailing list
http://cgi.insecure.org/mailman/listinfo/nmap-dev
Archived at http://seclists.org/nmap-dev/