[smokeping-users] Modifying DNS probe to add extra options

[Shawn Rose]

Hey Niko

Well after betting myself over the head. I dropped the idea of fixing
up DNS.pm. Setup EchoPingDNS, couldn't figure out the error till I
realized echoping on debian etch is a very old version. Grabbed
echoping from the unstable repository and it now works. My project is
on it's way to being complete!! Thanks alot for your help.

Cheers
Shawn Rose

On Mon, Jun 16, 2008 at 7:58 PM, Shawn Rose <shawnjrose at gmail.com> wrote:
> Hey Niko,
>
> So I made those changes, and I see the following error in my log.
>
> Jun 16 23:12:30 njsmoke smokeping[13812]: Use of uninitialized value
> in concatenation (.) or string at
> /usr/share/perl5/smokeping/Smokeping/probes/DNS.pm line 122.
>
> I'm guessing it might have to do with multiple options I'm trying to
> pass to the command.
>
> extraopts = -6 -tcp
>
> I want to Probe DNS servers using IPv6 and 4 by UDP and TCP methods.
> As mentioned in my preiouvs email, those are the only changes I made.
>
> If theres any way I can make this work. Please let me know!!! This is
> the only thing I have left to get working :)
>
> Thanks,
> Shawn Rose
>
>
> Smokeping started without error. So I think I'm golden. If it looks
> good, just give me a thumbs up.
>
>
>>
>>> I think I solved it. Does this look good?
>>>
>>> my $query = "$self->{properties}{binary} \@$dnsserver $lookuphost $extra";
>>
>> Yeah, that should work. You could check first if the extraoptions are
>> specified at all to avoid possible undefined warnings, but that's just
>> fine tuning.
>>
>> The probes should really inherit uniform command argument construction
>> from a separate module so the extraopts wouldn't have to be coded
>> separately into each one.  That's been on my todo list for ages, but I
>> never got round to it.
>> --
>> Niko
>>
>> _______________________________________________
>> smokeping-users mailing list
>> smokeping-users at lists.oetiker.ch
>> https://lists.oetiker.ch/cgi-bin/listinfo/smokeping-users
>>
>
>
>
> --
> Check out
>
> http://www.bulldoze.ca
>
> Cheers
> Shawn Rose
>



-- 
Check out

http://www.bulldoze.ca

Cheers
Shawn Rose