From: Mike Taylor Date: Thu, 15 Mar 2007 11:36:58 +0000 (+0000) Subject: Small but important changes: X-Git-Tag: CPAN-v1.02~54^2~478 X-Git-Url: http://jsfdemo.indexdata.com/cgi-bin?a=commitdiff_plain;h=10db92a1882ad7483176e8ba2432a8d06aafff57;p=irspy-moved-to-github.git Small but important changes: - Reconnect to the IRSpy database before rewriting XML, not (as previously) to the server being tested! - Do not return before sanity checks, which are now run on the correct set of connections. --- diff --git a/lib/ZOOM/IRSpy.pm b/lib/ZOOM/IRSpy.pm index 62df949..da29f02 100644 --- a/lib/ZOOM/IRSpy.pm +++ b/lib/ZOOM/IRSpy.pm @@ -1,4 +1,4 @@ -# $Id: IRSpy.pm,v 1.76 2007-03-10 13:02:36 mike Exp $ +# $Id: IRSpy.pm,v 1.77 2007-03-15 11:36:58 mike Exp $ package ZOOM::IRSpy; @@ -234,7 +234,7 @@ sub _rewrite_record { # Since IRSpy can run for a long time between writes back to the # database, it's quite possible for the server to have closed the # connection as idle. So re-establish it if necessary. - $conn->connect($conn->option("host")); + $this->{conn}->connect($conn->option("host")); _really_rewrite_record($this->{conn}, $rec); $conn->log("irspy", "rewrote XML record"); @@ -469,11 +469,10 @@ sub check { } $this->log("irspy", "exiting main loop"); - return $nskipped; # Sanity-checks don't work if conns are closed # Sanity checks: none of the following should ever happen my $finished = 1; - @conn = @{ $this->{connections} }; + $this->log("irspy", "performing end-of-run sanity-checks"); foreach my $conn (@conn) { my $test = $conn->option("current_test_address"); my $next = $this->_next_test($test);