Re: [NSE][PATCH] OpenSSL bindings for NSE

[David Fifield <david () bamsoftware com>]

On Wed, Sep 03, 2008 at 04:22:58PM -0600, Patrick Donnelly wrote:
> On Wed, Sep 3, 2008 at 4:07 PM, David Fifield <david () bamsoftware com> wrote:
> > Just using nmap.have_ssl in every script that uses the openssl modules
> > sounds okay. I worry a little that script authors will forget to do that
> > and we will release a broken script.
>
> I'm of the opposite opinion. Allowing require to error because of a
> missing dependency is probably the best solution for this. I am
> finishing a patch that will issue an error warning that informs the
> user the script could not be loaded (compilation, run time errors, and
> missing required fields), as well as the actual error if debugging is
> on.

I just saw your checkin r9922 and it worked, skipping SSH-hostkey.nse
and running the other scripts. I agree that's a better solution.

David Fifield

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