[check_postgres] [commit] Roll out 2.3.2,
fixes quoting problem notes by Glyn Astill
check_postgres at bucardo.org
check_postgres at bucardo.org
Wed Oct 8 14:29:20 UTC 2008
Committed by Greg Sabino Mullane <greg at endpoint.com>
Roll out 2.3.2, fixes quoting problem notes by Glyn Astill
---
check_postgres.pl | 16 ++++++++++------
check_postgres.pl.asc | 6 +++---
check_postgres.pl.html | 8 +++++++-
index.html | 8 ++++----
4 files changed, 24 insertions(+), 14 deletions(-)
diff --git a/check_postgres.pl b/check_postgres.pl
index 6c2ab8c..bf5515b 100755
--- a/check_postgres.pl
+++ b/check_postgres.pl
@@ -28,7 +28,7 @@ $Data::Dumper::Varname = 'POSTGRES';
$Data::Dumper::Indent = 2;
$Data::Dumper::Useqq = 1;
-our $VERSION = '2.3.1';
+our $VERSION = '2.3.2';
use vars qw/ %opt $PSQL $res $COM $SQL $db /;
@@ -3344,10 +3344,10 @@ sub check_replicate_row {
}
my ($table,$pk,$id,$col,$val1,$val2) = (@repinfo);
- ## Quote funky identifiers
- $table = qq{"$table"} if $table !~ /^\w+$/;
- $pk = qq{"$pk"} if $pk !~ /^\w+$/;
- $col = qq{"$col"} if $col !~ /^\w+$/;
+ ## Quote everything, just to be safe (e.g. columns named 'desc')
+ $table = qq{"$table"};
+ $pk = qq{"$pk"};
+ $col = qq{"$col"};
if ($val1 eq $val2) {
ndie 'Makes no sense to test replication with same values';
@@ -3549,7 +3549,7 @@ sub check_sequence {
=head1 NAME
B<check_postgres.pl> - a Postgres monitoring script for Nagios, MRTG, and others
-This documents describes check_postgres.pl version 2.3.1
+This documents describes check_postgres.pl version 2.3.2
=head1 SYNOPSIS
@@ -4690,6 +4690,10 @@ Items not specifically attributed are by Greg Sabino Mullane.
=over 4
+=item B<Version 2.3.2>
+
+ Always quote identifiers in check_replicate_row action.
+
=item B<Version 2.3.1>
Give a better error if one of the databases cannot be reached.
diff --git a/check_postgres.pl.asc b/check_postgres.pl.asc
index 9b2a785..39f0eb1 100644
--- a/check_postgres.pl.asc
+++ b/check_postgres.pl.asc
@@ -1,6 +1,6 @@
-----BEGIN PGP SIGNATURE-----
-iEYEABEDAAYFAkjrhvsACgkQvJuQZxSWSsgSEwCgy3WmBhnT+IDOHIS32D4KkWNk
-+OkAoMApAyuIV8zbL+7+QrVgSmxozNUt
-=tTHE
+iEYEABEDAAYFAkjsw6cACgkQvJuQZxSWSshhtQCfTE8t7CFn2Xfmtt8E/GoTvyOI
+kvoAoNYdHSGahtDWRINR8oRqEU+XOaqB
+=xyuQ
-----END PGP SIGNATURE-----
diff --git a/check_postgres.pl.html b/check_postgres.pl.html
index 7ec4ee7..43ea6c3 100644
--- a/check_postgres.pl.html
+++ b/check_postgres.pl.html
@@ -90,7 +90,7 @@
<hr />
<h1><a name="name">NAME</a></h1>
<p><strong>check_postgres.pl</strong> - a Postgres monitoring script for Nagios, MRTG, and others
-This documents describes check_postgres.pl version 2.3.1</p>
+This documents describes check_postgres.pl version 2.3.2</p>
<p>
</p>
<hr />
@@ -1119,6 +1119,12 @@ feature requests, and commit notices, send email to <a href="mailto:check_postgr
<h1><a name="history">HISTORY</a></h1>
<p>Items not specifically attributed are by Greg Sabino Mullane.</p>
<dl>
+<dt><strong><a name="item_version_2_2e3_2e2"><strong>Version 2.3.2</strong></a></strong></dt>
+
+<dd>
+<pre>
+ Always quote identifiers in check_replicate_row action.</pre>
+</dd>
<dt><strong><a name="item_version_2_2e3_2e1"><strong>Version 2.3.1</strong></a></strong></dt>
<dd>
diff --git a/index.html b/index.html
index e09fdf1..2632930 100644
--- a/index.html
+++ b/index.html
@@ -21,14 +21,14 @@ h1 {
<h1>check_postgres.pl</h1>
-<p><b>check_postgres.pl</b> is a script for checking the state of one or more Postgres databases and reporting back in a Nagios-friendly manner. It was developed by Greg Sabino Mullane of <a href="http://www.endpoint.com/">End Point Corporation</a> and is BSD-licensed. The latest version is <b>2.3.1</b>, and was released on October 07, 2008.</p>
+<p><b>check_postgres.pl</b> is a script for checking the state of one or more Postgres databases and reporting back in a Nagios-friendly manner. It was developed by Greg Sabino Mullane of <a href="http://www.endpoint.com/">End Point Corporation</a> and is BSD-licensed. The latest version is <b>2.3.2</b>, and was released on October 08, 2008.</p>
<ul>
- <li><a href="/check_postgres/check_postgres.pl.html">Documentation for check_postgres 2.3.1</a></li>
+ <li><a href="/check_postgres/check_postgres.pl.html">Documentation for check_postgres 2.3.2</a></li>
</ul>
<ul>
- <li><a href="/check_postgres/check_postgres.pl">Download check_postgres.pl 2.3.1</a></li>
- <li><a href="/check_postgres/check_postgres.pl.asc">PGP signature for check_postgres.pl 2.3.1</a></li>
+ <li><a href="/check_postgres/check_postgres.pl">Download check_postgres.pl 2.3.2</a></li>
+ <li><a href="/check_postgres/check_postgres.pl.asc">PGP signature for check_postgres.pl 2.3.2</a></li>
</ul>
<p>The latest development version can be downloaded via git:</p>
--
1.5.5.4
More information about the Check_postgres
mailing list