Nmap Development mailing list archives
Re: [NSE] [patch] Big changes to http-enum.nse
From: Ron <ron () skullsecurity net>
Date: Tue, 26 Oct 2010 22:12:18 -0500
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On Thu, 21 Oct 2010 23:54:20 -0700 David Fifield <david () bamsoftware com> wrote:
On Thu, Oct 21, 2010 at 11:47:03PM -0500, Ron wrote:I've made the changes you requested, including Patrick's recommendation. But beyond them, I've done some more work on the HTTP library. I'd like to get your opinions of revision 20720 of my branch. I made some interface changes to the http library, mostly by making functions local that probably should have been local (and that aren't being used). I also removed a couple functions that I don't think are useful, and replaced the header documentation. While writing the header documentation, I realized that the pipeline() code could use some cleanup. There are redundant arguments, and arguments that I'd argue are missing. My next task is going to be to clean up that set of functions, document all the public functions better, then call it good to go.Please hold off on doing that or do it in separate branch, so that the http-enum changes can be merged separately. Since that part is already done, you can merge it. These cleanup changes are great but I find them hard to keep track of when they're mixed with functinoal changes and new features.
Ok, I've committed it sans that change as revision 20762. I had some trouble with svn because of my not really knowing how to use it still, so please double check that I didn't break anything by accident. I'm going to continue working on my branch to improve the HTTP library. Ron -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.16 (GNU/Linux) iEYEARECAAYFAkzHmJUACgkQ2t2zxlt4g/QUuQCfXy2rze6KekOph9o1KeiY8JEi Y/AAoMoyFnCKr8BFP9atzBPdzUCoDsXn =sUKq -----END PGP SIGNATURE----- _______________________________________________ Sent through the nmap-dev mailing list http://cgi.insecure.org/mailman/listinfo/nmap-dev Archived at http://seclists.org/nmap-dev/
Current thread:
- Re: [NSE] [patch] Big changes to http-enum.nse, (continued)
- Re: [NSE] [patch] Big changes to http-enum.nse Ron (Oct 18)
- Re: [NSE] [patch] Big changes to http-enum.nse David Fifield (Oct 18)
- Re: [NSE] [patch] Big changes to http-enum.nse Ron (Oct 18)
- Re: [NSE] [patch] Big changes to http-enum.nse Ron (Oct 19)
- Re: [NSE] [patch] Big changes to http-enum.nse David Fifield (Oct 19)
- Re: [NSE] [patch] Big changes to http-enum.nse David Fifield (Oct 19)
- Re: [NSE] [patch] Big changes to http-enum.nse Patrick Donnelly (Oct 19)
- Re: [NSE] [patch] Big changes to http-enum.nse Ron (Oct 21)
- Re: [NSE] [patch] Big changes to http-enum.nse David Fifield (Oct 21)
- Re: [NSE] [patch] Big changes to http-enum.nse Jacky Jack (Oct 23)
- Re: [NSE] [patch] Big changes to http-enum.nse Ron (Oct 26)
- Re: [NSE] [patch] Big changes to http-enum.nse David Fifield (Oct 27)
- Re: [NSE] [patch] Big changes to http-enum.nse Martin Holst Swende (Oct 18)
- Re: [NSE] [patch] Big changes to http-enum.nse Ron (Oct 19)
- Re: [NSE] [patch] Big changes to http-enum.nse Patrik Karlsson (Oct 18)
- Re: [NSE] [patch] Big changes to http-enum.nse Martin Holst Swende (Oct 18)
- Re: [NSE] [patch] Big changes to http-enum.nse Martin Holst Swende (Oct 18)
- Re: [NSE] [patch] Big changes to http-enum.nse Ron (Oct 17)
- Re: [NSE] [patch] Big changes to http-enum.nse Ron (Oct 17)