Nmap Development mailing list archives

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


From: "Patrick Donnelly" <batrick.donnelly () gmail com>
Date: Wed, 3 Sep 2008 16:43:21 -0600

On Wed, Sep 3, 2008 at 4:32 PM, Sven Klemm <sven () c3d2 de> wrote:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

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 agree. This solution is actually much nicer as the user may have
installed a library nmap does not know about. Like manually building
the openssl module while having nmap without openssl.

But the places it looked for the library should still be shown at
least when -d is active.

This will be the case. The default will be a short message saying it
couldn't be loaded for whatever reason. If debugging is on, then the
actual error message is printed.

Cheers,

-- 
-Patrick Donnelly

"One of the lessons of history is that nothing is often a good thing
to do and always a clever thing to say."

-Will Durant

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


Current thread: