Re: Updated output for scripts

[Fyodor <fyodor () insecure org>]

On Tue, Nov 10, 2009 at 11:14:02PM -0600, Ron wrote:
> Fyodor wrote:
> > > | | WINDOWS2003\ASPNET (RID: 1008)
> > > | | | Full name:   ASP.NET Machine Account
> > > | | | Description: Account used for running the ASP.NET worker process (aspnet_wp.exe)
> > > | | |_Flags:       Password not required, Password does not expire, Normal user account
> >
> > Or perhaps it would be even better with the "|" replaced by a space
> > for levels 2 and greater:
> Hmm, the reason I went with two spaces is because NSE adds two spaces
> after the first pipe. I'm cool with either way, though.

Yeah, I think that if the extra space is removed from format_output,
it should also be removed from the first level of indentation (printed
by NSE).

Here is an example with and without the "extra" space for single line
output:

80/tcp    open   http    Apache httpd 2.2.3 ((Red Hat))
|_ html-title: Go ahead and ScanMe!

80/tcp    open   http    Apache httpd 2.2.3 ((Red Hat))
|_html-title: Go ahead and ScanMe!


> Hmm, I can easily make that change, and it'll change every script
> automatically, but I'm not sure I like it. I sort of like how the pipes
> group everything together. Without them, I also think multiple scripts'
> outputs will be difficult to tell apart.

I'm not opposed to keeping the pipes.  Kris seems to like them too.
Maybe we should kill the "extra" space, but keep the pipes rather than
change them to spaces.

Cheers,
-F

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