From: Alan T. DeKok <aland@freeradius.org>
Date: Sat, 13 Dec 2025 21:10:46 +0000 (-0500)
Subject: Revert "cancel retries on failure"
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=21a3ccab561829b3658e45bf076945ece7653548;p=thirdparty%2Ffreeradius-server.git

Revert "cancel retries on failure"

This reverts commit 07d2b3c44030462f4ff3d28522e95ed169061848.

Nope.  the frame->retry is only for the "retry" keyword.
module.c and xlat.c do their own retries, and allocate their own
timers.
---

diff --git a/src/modules/rlm_radius/bio.c b/src/modules/rlm_radius/bio.c
index 4c4b25f3422..119c665f100 100644
--- a/src/modules/rlm_radius/bio.c
+++ b/src/modules/rlm_radius/bio.c
@@ -2403,7 +2403,6 @@ static void request_fail(request_t *request, NDEBUG_UNUSED void *preq, void *rct
 	u->rcode = RLM_MODULE_FAIL;
 	u->treq = NULL;
 
-	unlang_interpret_request_cancel_retry(request);
 	unlang_interpret_mark_runnable(request);
 }
 
@@ -2422,7 +2421,6 @@ static void request_complete(request_t *request, NDEBUG_UNUSED void *preq, void
 
 	u->treq = NULL;
 
-	unlang_interpret_request_cancel_retry(request);
 	unlang_interpret_mark_runnable(request);
 }
 
@@ -2478,7 +2476,6 @@ static void do_signal(rlm_radius_t const *inst, bio_request_t *u, UNUSED request
 	 */
 	case FR_SIGNAL_CANCEL:
 		trunk_request_signal_cancel(u->treq);
-		unlang_interpret_request_cancel_retry(request);
 		u->treq = NULL;
 		return;