Add a call to PQsetNoticeProcessor() to catch PostgreSQL "notices" and
authorPhilip Hazel <ph10@hermes.cam.ac.uk>
Fri, 14 Jul 2006 14:42:57 +0000 (14:42 +0000)
committerPhilip Hazel <ph10@hermes.cam.ac.uk>
Fri, 14 Jul 2006 14:42:57 +0000 (14:42 +0000)
thereby stop them being written to stderr (the default).

doc/doc-txt/ChangeLog
src/src/lookups/pgsql.c

index fc78884fd3e837bbfbb2cbd0aa774ae730976403..d3535f71912ba2bb49e461a629f265a4595f07e1 100644 (file)
@@ -1,4 +1,4 @@
-$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.377 2006/07/14 14:32:08 ph10 Exp $
+$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.378 2006/07/14 14:42:57 ph10 Exp $
 
 Change log file for Exim from version 4.21
 -------------------------------------------
 
 Change log file for Exim from version 4.21
 -------------------------------------------
@@ -114,6 +114,10 @@ PH/19 Added PQsetClientEncoding(conn, "SQL_ASCII") to the pgsql code module.
 PH/20 Perl can change the locale. Exim was resetting it after a ${perl call,
       but not after initializing Perl.
 
 PH/20 Perl can change the locale. Exim was resetting it after a ${perl call,
       but not after initializing Perl.
 
+PH/21 Added a call to PQsetNoticeProcessor() to catch pgsql "notices" and
+      output them only if debugging. By default they are written stderr,
+      apparently, which is not desirable.
+
 
 Exim version 4.62
 -----------------
 
 Exim version 4.62
 -----------------
index 8b777f40b33c272ea03d6802391f6cffc456d398..9c0c647c0caad219054111247c5649b3c7950591 100644 (file)
@@ -1,4 +1,4 @@
-/* $Cambridge: exim/src/src/lookups/pgsql.c,v 1.6 2006/07/14 14:21:27 ph10 Exp $ */
+/* $Cambridge: exim/src/src/lookups/pgsql.c,v 1.7 2006/07/14 14:42:57 ph10 Exp $ */
 
 /*************************************************
 *     Exim - an Internet mail transport agent    *
 
 /*************************************************
 *     Exim - an Internet mail transport agent    *
@@ -76,6 +76,28 @@ while ((cn = pgsql_connections) != NULL)
 }
 
 
 }
 
 
+/*************************************************
+*       Notice processor function for pgsql      *
+*************************************************/
+
+/* This function is passed to pgsql below, and called for any PostgreSQL
+"notices". By default they are written to stderr, which is undesirable.
+
+Arguments:
+  arg        an opaque user cookie (not used)
+  message    the notice
+
+Returns:     nothing
+*/
+
+static void
+notice_processor(void *arg, const char *message)
+{
+arg = arg;   /* Keep compiler happy */
+DEBUG(D_lookup) debug_printf("PGSQL: %s\n", message);
+}
+
+
 
 /*************************************************
 *        Internal search function                *
 
 /*************************************************
 *        Internal search function                *
@@ -253,6 +275,12 @@ if (cn == NULL)
 
   PQsetClientEncoding(pg_conn, "SQL_ASCII");
 
 
   PQsetClientEncoding(pg_conn, "SQL_ASCII");
 
+  /* Set the notice processor to prevent notices from being written to stderr
+  (which is what the default does). Our function (above) just produces debug
+  output. */
+
+  PQsetNoticeProcessor(pg_conn, notice_processor, NULL);
+
   /* Add the connection to the cache */
 
   cn = store_get(sizeof(pgsql_connection));
   /* Add the connection to the cache */
 
   cn = store_get(sizeof(pgsql_connection));