I have a patch to suggest at the bottom of this email.
Thanks,
Chris
On Feb 20, 2004, at 1:51 PM, Alan DeKok wrote:
Chris Brotsos <[EMAIL PROTECTED]> wrote:Isn't this actually correct? According to the DOCS, if Synchronous is
set to Yes, then all of the other parameters should be set to 0. How
will the server 'know' what the retry_delay, retry_count, and dead_time
are?
The server should have some configuration to use, to decide that a home server hasn't responded in N seconds, and marks it dead. That time can probably be "retry_delay * retry_count".
Some code probably needs to be added to the server to support this.
Alan DeKok.
-
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html
Index: request_list.c
===================================================================
RCS file: /source/radiusd/src/main/request_list.c,v
retrieving revision 1.26
diff -u -r1.26 request_list.c
--- request_list.c 19 Feb 2004 21:40:58 -0000 1.26
+++ request_list.c 23 Feb 2004 17:03:19 -0000
@@ -916,6 +916,14 @@
* in the given retry window.
*/
if (mainconfig.proxy_synchronous) {
+ if (info->now > (request->timestamp + (mainconfig.proxy_retry_delay * mainconfig.proxy_retry_count))) {
+ rad_assert(request->child_pid == NO_SUCH_CHILD_PID);
+ request_reject(request);
+ realm_disable(request->proxy->dst_ipaddr,
+ request->proxy->dst_port);
+ request->finished = TRUE;
+ goto setup_timeout;
+ }
request->proxy_next_try = info->now + mainconfig.proxy_retry_delay;
}
difference = request->proxy_next_try - info->now;
- List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html
