From: Alan T. DeKok <aland@freeradius.org>
Date: Thu, 10 Aug 2017 09:15:53 +0000 (+0200)
Subject: it doesn't make sense to replicate and have status checks
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=ca8d859fb346b93a6f783cf01b00623c13cbe0fa;p=thirdparty%2Ffreeradius-server.git

it doesn't make sense to replicate and have status checks
---

diff --git a/src/modules/rlm_radius/rlm_radius.c b/src/modules/rlm_radius/rlm_radius.c
index 376b03f52ce..5dd65e3611f 100644
--- a/src/modules/rlm_radius/rlm_radius.c
+++ b/src/modules/rlm_radius/rlm_radius.c
@@ -517,11 +517,21 @@ static int mod_bootstrap(void *instance, CONF_SECTION *conf)
 	 */
 	if (inst->status_check && (inst->status_check != FR_CODE_STATUS_SERVER) &&
 	    !inst->allowed[inst->status_check]) {
-		cf_log_err(inst->io_conf, "Using 'status_check = %s' requires also 'type = %s'",
+		cf_log_err(conf, "Using 'status_check = %s' requires also 'type = %s'",
 			   fr_packet_codes[inst->status_check], fr_packet_codes[inst->status_check]);
 		return -1;
 	}
 
+	/*
+	 *	If we're replicating, we don't care if the other end
+	 *	is alive.
+	 */
+	if (inst->replicate && inst->status_check) {
+		cf_log_warn(conf, "Ignoring 'status_check = %s' due to 'replicate = true'",
+			    fr_packet_codes[inst->status_check]);
+		inst->status_check = 0;
+	}
+
 	/*
 	 *	Set limits on retransmission timers
 	 */