Re: New script - http-git.nse

[David Fifield <david () bamsoftware com>]

On Wed, Jul 25, 2012 at 10:46:12PM -0500, Alex Weber wrote:
> On Fri, Jul 20, 2012 at 9:14 AM, David Fifield <david () bamsoftware com>wrote:
>
> > On Thu, Jul 19, 2012 at 11:50:32PM -0500, Alex Weber wrote:
> > > On Thu, Jul 19, 2012 at 1:20 PM, David Fifield <david () bamsoftware com
> > > wrote:
> > > > Great job on this script. I think it's going to be well appreciated. I
> > > > committed it in r29273. If you would, there are some small additional
> > > > changes I'd like you to make.
> > >
> > > A stray `ap(config)` ended up in the committed version - Ron removed it
> > and
> > > committed it as r29279.
> > > Thanks for the suggestions - I'll be sending a patch along soon :)
> >
> > Oops, that was my fault while testing to see what the remote URLs looked
> > like.
> >
> > David Fifield
>
> I made the changes you guys recommended - you can specify a different root
> (or a table of them) with --script-args, the output shows links to the
> GitHub/Gitorious/etc page, the (stupid) ap() function was removed, and
> replaced by strbuf where appropriate, and I added a bunch more .gitignore
> fingerprints. Comments welcome!

I checked your latest
https://github.com/AlexWebr/nse/blob/1f125ab359d59453b5b04a0ad3d14eef7adbcc0a/http-git.nse
and it seemed to be the same as what Ron checked in, so I guess it's up
to date.

I made the requested change to the script to display remote URLs
directly. I removed some of the interpretation of URLs; I feel that a
copy-and-pastable string like "git () github com:skullspace/breachdb.git"
is more useful than something like "GitHub remote: skullspace/breachdb
(accessed over SSH)".

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