[Nagiosplug-devel] [ nagiosplug-Bugs-1292404 ] Can't send newline with check_tcp

[SourceForge.net]

Bugs item #1292404, was opened at 2005-09-15 23:24
Message generated for change (Comment added) made by tonvoon
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397597&aid=1292404&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: Argument proccessing
Group: Release (specify)
>Status: Closed
Resolution: None
Priority: 5
Submitted By: Sebastian Wiesinger (swiesinger)
Assigned to: Ton Voon (tonvoon)
Summary: Can't send newline with check_tcp

Initial Comment:
In v1.4.1 check_tcp doesn't send \r\n at the end of the
-s string. that breaks many checks.

I didn't see a way to send a newline, so I made the
following patch which parses \(n,r,t) in the -s string.

I don't know if that should be added to the expect
string, too, I'm a C novice and didn't find an easy way
to add that functionality.

Regards,

Sebastian

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

>Comment By: Ton Voon (tonvoon)
Date: 2006-03-29 17:35

Message:
Logged In: YES 
user_id=664364

Sebastien,

This has been added to CVS now. Have regressed John's patch re: command line 
options.

Ton

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

Comment By: Ton Voon (tonvoon)
Date: 2006-03-23 16:04

Message:
Logged In: YES 
user_id=664364

Sebastian,

Thanks for the patch. I'm currently asking the nagiosplug-devel mailing list 
whether it is best to go with your idea of escaping chars or John's idea of 
option flags to specify endings.

Will respond soon.

Ton

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

Comment By: John Rouillard (rouilj)
Date: 2005-11-04 19:31

Message:
Logged In: YES 
user_id=707416

See:
 https://sourceforge.net/tracker/index.php?func=detail&aid=1346104&group_id=29880&atid=397599

for a different patch that addresses the same problem and
includes support for the quit string as well.

The expect string could probably benefit from your pasing of
\(n,r,t)
in it provided \\ works to embed a \.

Hopefully one of our patches will make it into the mainline.

-- rouilj


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

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