From: Nick Porter <nick@portercomputing.co.uk>
Date: Wed, 18 Jan 2023 14:27:48 +0000 (+0000)
Subject: No need for [*] on debug policies
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=ae5e2d9a09f8e454abcb4a2ea54fcfb36d541381;p=thirdparty%2Ffreeradius-server.git

No need for [*] on debug policies
---

diff --git a/raddb/policy.d/debug b/raddb/policy.d/debug
index 08b4491890f..8b5ffaca64e 100644
--- a/raddb/policy.d/debug
+++ b/raddb/policy.d/debug
@@ -2,7 +2,7 @@
 #  Outputs the contents of the control list in debugging (-X) mode
 #
 debug_control {
-	if ("%(debug_attr:control[*])" == '') {
+	if ("%(debug_attr:control)" == '') {
 		noop
 	}
 }
@@ -11,7 +11,7 @@ debug_control {
 #  Outputs the contents of the request list in debugging (-X) mode
 #
 debug_request {
-	if ("%(debug_attr:request[*])" == '') {
+	if ("%(debug_attr:request)" == '') {
 		noop
 	}
 }
@@ -20,7 +20,7 @@ debug_request {
 #  Outputs the contents of the coa list in debugging (-X) mode
 #
 debug_coa {
-	if ("%(debug_attr:coa[*])" == '') {
+	if ("%(debug_attr:coa)" == '') {
 		noop
 	}
 }
@@ -29,7 +29,7 @@ debug_coa {
 #  Outputs the contents of the reply list in debugging (-X) mode
 #
 debug_reply {
-	if ("%(debug_attr:reply[*])" == '') {
+	if ("%(debug_attr:reply)" == '') {
 		noop
 	}
 }
@@ -38,7 +38,7 @@ debug_reply {
 #  Outputs the contents of the session state list in debugging (-X) mode
 #
 debug_session_state {
-	if ("%(debug_attr:session-state[*])" == '') {
+	if ("%(debug_attr:session-state)" == '') {
 		noop
 	}
 }
diff --git a/src/tests/keywords/policy.conf b/src/tests/keywords/policy.conf
index 06b075d7cc8..e6fa5de2b32 100644
--- a/src/tests/keywords/policy.conf
+++ b/src/tests/keywords/policy.conf
@@ -6,7 +6,7 @@
 #  Outputs the contents of the control list in debugging (-X) mode
 #
 debug_control {
-	if("%(debug_attr:control[*])" == '') {
+	if("%(debug_attr:control)" == '') {
 		noop
 	}
 }
@@ -15,7 +15,7 @@ debug_control {
 #  Outputs the contents of the request list in debugging (-X) mode
 #
 debug_request {
-	if("%(debug_attr:request[*])" == '') {
+	if("%(debug_attr:request)" == '') {
 		noop
 	}
 }
@@ -24,7 +24,7 @@ debug_request {
 #  Outputs the contents of the reply list in debugging (-X) mode
 #
 debug_reply {
-	if("%(debug_attr:reply[*])" == '') {
+	if("%(debug_attr:reply)" == '') {
 		noop
 	}
 }
diff --git a/src/tests/modules/unit_test_module.conf b/src/tests/modules/unit_test_module.conf
index c8983aca4dd..dfd169b415e 100644
--- a/src/tests/modules/unit_test_module.conf
+++ b/src/tests/modules/unit_test_module.conf
@@ -70,7 +70,7 @@ policy {
 	#  Outputs the contents of the control list in debugging (-X) mode
 	#
 	debug_control {
-		if("%(debug_attr:control[*])" == '') {
+		if("%(debug_attr:control)" == '') {
 			noop
 		}
 	}
@@ -79,7 +79,7 @@ policy {
 	#  Outputs the contents of the request list in debugging (-X) mode
 	#
 	debug_request {
-		if("%(debug_attr:request[*])" == '') {
+		if("%(debug_attr:request)" == '') {
 			noop
 		}
 	}
@@ -88,7 +88,7 @@ policy {
 	#  Outputs the contents of the reply list in debugging (-X) mode
 	#
 	debug_reply {
-		if("%(debug_attr:reply[*])" == '') {
+		if("%(debug_attr:reply)" == '') {
 			noop
 		}
 	}
@@ -97,7 +97,7 @@ policy {
 	#  Outputs the contents of the session state list in debugging (-X) mode
 	#
 	debug_session_state {
-		if("%(debug_attr:session-state[*])" == '') {
+		if("%(debug_attr:session-state)" == '') {
 			noop
 		}
 	}