Add PQconnectdb_safe function which filters out some parameters incompatible with libpq.

kvap · kelvich · commit 015aa8f7b53e · 2017-11-13T02:17:30.000+03:00
diff --git a/multimaster.c b/multimaster.c
@@ -1859,8 +1859,6 @@ void MtmUpdateNodeConnectionInfo(MtmConnectionInfo* conn, char const* connStr)
 			elog(ERROR, "Invalid raftable port: %s", port+9);
 		}
 		n += 9;
-		memmove(port, port+n, connStrLen - n + 1);
-		connStrLen -= n;
 	} else { 
 		conn->raftablePort = 0;
 	}
@@ -1872,8 +1870,6 @@ void MtmUpdateNodeConnectionInfo(MtmConnectionInfo* conn, char const* connStr)
 			elog(ERROR, "Invalid arbiter port: %s", port+12);
 		}
 		n += 12;
-		memmove(port, port+n, connStrLen - n + 1);
-		connStrLen -= n;
 	} else { 
 		conn->arbiterPort = 0;
 	}
@@ -2796,6 +2792,32 @@ typedef struct
 	int       nodeId;
 } MtmGetClusterInfoCtx;
 
+static void erase_option_from_connstr(const char *option, char *connstr)
+{
+	char *needle = psprintf("%s=", option);
+	while (1) {
+		char *found = strstr(connstr, needle);
+		if (found == NULL) break;
+		while (*found != '\0' && *found != ' ') {
+			*found = ' ';
+			found++;
+		}
+	}
+	pfree(needle);
+}
+
+PGconn *PQconnectdb_safe(const char *conninfo)
+{
+	PGconn *conn;
+	char *safe_connstr = pstrdup(conninfo);
+	erase_option_from_connstr("raftport", safe_connstr);
+	erase_option_from_connstr("arbiterport", safe_connstr);
+
+	conn = PQconnectdb(safe_connstr);
+
+	pfree(safe_connstr);
+	return conn;
+}
 
 Datum
 mtm_get_cluster_info(PG_FUNCTION_ARGS)
@@ -2828,9 +2850,9 @@ mtm_get_cluster_info(PG_FUNCTION_ARGS)
 	if (usrfctx->nodeId > Mtm->nAllNodes) {
 		SRF_RETURN_DONE(funcctx);      
 	}	
-	conn = PQconnectdb(Mtm->nodes[usrfctx->nodeId-1].con.connStr);
+	conn = PQconnectdb_safe(Mtm->nodes[usrfctx->nodeId-1].con.connStr);
 	if (PQstatus(conn) != CONNECTION_OK) {
-		elog(ERROR, "Failed to establish connection '%s' to node %d", Mtm->nodes[usrfctx->nodeId-1].con.connStr, usrfctx->nodeId);
+		elog(ERROR, "Failed to establish connection '%s' to node %d: error = %s", Mtm->nodes[usrfctx->nodeId-1].con.connStr, usrfctx->nodeId, PQerrorMessage(conn));
 	}
 	result = PQexec(conn, "select * from mtm.get_cluster_state()");
 
@@ -3004,7 +3026,7 @@ static void MtmBroadcastUtilityStmt(char const* sql, bool ignoreError)
 	{ 
 		if (!BIT_CHECK(disabledNodeMask, i)) 
 		{
-			conns[i] = PQconnectdb(psprintf("%s application_name=%s", Mtm->nodes[i].con.connStr, MULTIMASTER_BROADCAST_SERVICE));
+			conns[i] = PQconnectdb_safe(psprintf("%s application_name=%s", Mtm->nodes[i].con.connStr, MULTIMASTER_BROADCAST_SERVICE));
 			if (PQstatus(conns[i]) != CONNECTION_OK)
 			{
 				if (ignoreError) 
@@ -3016,7 +3038,7 @@ static void MtmBroadcastUtilityStmt(char const* sql, bool ignoreError)
 					do { 
 						PQfinish(conns[i]);
 					} while (--i >= 0);                             
-					elog(ERROR, "Failed to establish connection '%s' to node %d", Mtm->nodes[failedNode].con.connStr, failedNode+1);
+					elog(ERROR, "Failed to establish connection '%s' to node %d, error = %s", Mtm->nodes[failedNode].con.connStr, failedNode+1, PQerrorMessage(conns[i]));
 				}
 			}
 			PQsetNoticeReceiver(conns[i], MtmNoticeReceiver, &i);
diff --git a/multimaster.h b/multimaster.h
@@ -7,6 +7,7 @@
 
 #include "access/clog.h"
 #include "pglogical_output/hooks.h"
+#include "libpq-fe.h"
 
 #define  DEBUG_LEVEL 0
 
@@ -283,7 +284,7 @@ extern void  MtmUpdateLsnMapping(int nodeId, XLogRecPtr endLsn);
 extern XLogRecPtr MtmGetFlushPosition(int nodeId);
 extern bool MtmWatchdog(timestamp_t now);
 extern void MtmCheckHeartbeat(void);
-
+extern PGconn *PQconnectdb_safe(const char *conninfo);
 
 
 #endif
diff --git a/pglogical_receiver.c b/pglogical_receiver.c
@@ -217,8 +217,9 @@ pglogical_receiver_main(Datum main_arg)
 	char	*copybuf = NULL;
 	int spill_file = -1;
 	StringInfoData spill_info;
+	char *slotName;
 	char* connString = psprintf("replication=database %s", Mtm->nodes[nodeId-1].con.connStr);
-	char* slotName = psprintf(MULTIMASTER_SLOT_PATTERN, MtmNodeId);
+	slotName = psprintf(MULTIMASTER_SLOT_PATTERN, MtmNodeId);
 
 	initStringInfo(&spill_info);
 
@@ -261,7 +262,7 @@ pglogical_receiver_main(Datum main_arg)
 		count = Mtm->recoveryCount;
 		
 		/* Establish connection to remote server */
-		conn = PQconnectdb(connString);
+		conn = PQconnectdb_safe(connString);
 		status = PQstatus(conn);
 		if (status != CONNECTION_OK)
 		{
