[Nagiosplug-devel] [ nagiosplug-Patches-1041571 ] simap,spop certificate check and new check_nntps check

[SourceForge.net]

Patches item #1041571, was opened at 2004-10-06 17:05
Message generated for change (Settings changed) made by tonvoon
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397599&aid=1041571&group_id=29880

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Enhancement
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Eric Chen (chen23)
>Assigned to: Ton Voon (tonvoon)
Summary: simap,spop certificate check and new check_nntps check

Initial Comment:
these patches add a certificate check for check_simap and 
check_spop. 

this also adds a new check for nntps

this also requires adding to configure.in:

check_tcp_ssl="check_simap check_spop check_nntps"

patches agains nagios-plugins-1.3.1
built on Solaris 9 w/ Forte Developer 7 C 5.4

----------------------------------------------------------------------

Comment By: Stanley Hopcroft (stanleyhopcroft)
Date: 2004-12-03 21:57

Message:
Logged In: YES 
user_id=395628

Dear Eric,

Thanks for your patience once again.

Corrected -D option processing in CVS.
pc09011> ./check_tcp -H pericles.IPAustralia.Gov.AU -p 443 -D 3
Certificate will expire on 04/30/2005 23:5.
pc09011> 



----------------------------------------------------------------------

Comment By: Eric Chen (chen23)
Date: 2004-12-03 12:16

Message:
Logged In: YES 
user_id=271106

the last patch isn't quite what i submitted.  the diff added ":D" when it 
should have been "D:".

i haven't tested the code, but i expect that it probably still fails.  thanks.

  -eric


----------------------------------------------------------------------

Comment By: Stanley Hopcroft (stanleyhopcroft)
Date: 2004-12-03 04:00

Message:
Logged In: YES 
user_id=395628

Dear Eric,

Thank you for your patches, testing and feedback.

Your corection has been added to the  CVS.

Thank you !

revision 1.49
date: 2004/12/02 21:03:26;  author: stanleyhopcroft;  state:
Exp;  lines: +4 -4
Tracker 1041571: appended :D to getopt_long() args (for cert
expiry check). Eric Chen.
----------------------------


----------------------------------------------------------------------

Comment By: Eric Chen (chen23)
Date: 2004-12-02 14:59

Message:
Logged In: YES 
user_id=271106

from: nagios-plugins-HEAD-200412020547
missing "-D" arg in getopt in plugins/check_tcp.c:

c = getopt_long (argc, argv, "+hVv46H:s:e:q:m:c:w:t:p:C:W:d:Sr:
jD:",longopts, &option);

otherwise functioning as expected

----------------------------------------------------------------------

Comment By: Stanley Hopcroft (stanleyhopcroft)
Date: 2004-12-01 08:34

Message:
Logged In: YES 
user_id=395628

Committed to head.

Sorry for the delay - blast need to deal with configure.in also.

Would you mind testing ?

----------------------------------------------------------------------

Comment By: Stanley Hopcroft (stanleyhopcroft)
Date: 2004-11-30 03:23

Message:
Logged In: YES 
user_id=395628

Dear Eric,

Thanks for the patches and your patience with the long wait
for a response.

I hope to apply them as soon as I can understand them.



----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397599&aid=1041571&group_id=29880