[sr-dev] git:master: - In active probing mode (ds_probing_mode=1) disable gateways not only on "408" but also on any non-successful reply ( as per ds_reply_codes)

[Carsten Bock]

Module: sip-router
Branch: master
Commit: 2a9b1cdc98d322032598d0c48fc4d7db085a83a0
URL:    http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=2a9b1cdc98d322032598d0c48fc4d7db085a83a0

Author: Carsten Bock <carsten at bock.info>
Committer: Carsten Bock <carsten at bock.info>
Date:   Tue Oct  4 17:14:44 2011 +0200

- In active probing mode (ds_probing_mode=1) disable gateways not only on "408" but also on any non-successful reply (as per ds_reply_codes)
(e.g. SEMS replies with "503" when in shutdown mode)

---

 modules_k/dispatcher/dispatch.c               |    4 +---
 modules_k/dispatcher/doc/dispatcher_admin.xml |    2 +-
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/modules_k/dispatcher/dispatch.c b/modules_k/dispatcher/dispatch.c
index f5f5242..2c28fb6 100644
--- a/modules_k/dispatcher/dispatch.c
+++ b/modules_k/dispatcher/dispatch.c
@@ -2342,9 +2342,7 @@ static void ds_options_callback( struct cell *t, int type,
 			LM_ERR("Setting the state failed (%.*s, group %d)\n", uri.len,
 					uri.s, group);
 		}
-	}
-	if(ds_probing_mode==1 && ps->code == 408)
-	{
+	} else if (ds_probing_mode==1) {
 
 		if (faked_msg_init() < 0)
 		{
diff --git a/modules_k/dispatcher/doc/dispatcher_admin.xml b/modules_k/dispatcher/doc/dispatcher_admin.xml
index d6d961a..3aeb737 100644
--- a/modules_k/dispatcher/doc/dispatcher_admin.xml
+++ b/modules_k/dispatcher/doc/dispatcher_admin.xml
@@ -586,7 +586,7 @@ modparam("dispatcher", "force_dst", 1)
  		<para>
 		Controls what gateways are tested to see if they are reachable. If set
 		to 0, only the gateways with state PROBING are tested, if set to 1, all
-		gateways are tested. If set to 1 and the response is 408 (timeout),
+		gateways are tested. If set to 1 and the is failure to the above list,
 		an active gateway is set to PROBING state.
  		</para>
  		<para>