When performing a lookup, the Path (if present) is evaluated and if the first hop is local we skip it to avoid unnecessary looping.
It is useful if multiple servers are sharing a common database, each saving contacts with their local address as the Path.
You can merge this Pull Request by running:
git pull https://github.com/kamailio/kamailio cchance/registrar
Or you can view, comment on it, or merge it online at:
https://github.com/kamailio/kamailio/pull/23
-- Commit Summary --
* registrar: add optional check for local path during lookup()
* registrar: update documentation to include path_check_local parameter
-- File Changes --
M modules/registrar/doc/registrar_admin.xml (26)
M modules/registrar/lookup.c (74)
M modules/registrar/reg_mod.c (2)
M modules/registrar/reg_mod.h (1)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/23.patchhttps://github.com/kamailio/kamailio/pull/23.diff
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/23
Module: kamailio
Branch: master
Commit: 2fe8749ab7bd7622d46a57ab88f90e35bb7b0164
URL: https://github.com/kamailio/kamailio/commit/2fe8749ab7bd7622d46a57ab88f90e3…
Author: alezzandro <alezzandro(a)gmail.com>
Committer: alezzandro <alezzandro(a)gmail.com>
Date: 2015-01-09T16:59:11+01:00
dispatcher: Corrected comments indentation
---
Modified: modules/dispatcher/dispatch.h
---
Diff: https://github.com/kamailio/kamailio/commit/2fe8749ab7bd7622d46a57ab88f90e3…
Patch: https://github.com/kamailio/kamailio/commit/2fe8749ab7bd7622d46a57ab88f90e3…
---
diff --git a/modules/dispatcher/dispatch.h b/modules/dispatcher/dispatch.h
index 29f2ec7..d991fd7 100644
--- a/modules/dispatcher/dispatch.h
+++ b/modules/dispatcher/dispatch.h
@@ -102,8 +102,10 @@ extern pv_spec_t ds_attrs_pv;
extern struct tm_binds tmb;
extern str ds_ping_method;
extern str ds_ping_from;
-extern int probing_threshold; /*!< number of failed requests, before a destination is taken into probing */
-extern int inactive_threshold; /*!< number of successful requests, before a destination is taken into active */
+extern int probing_threshold; /*!< number of failed requests,
+ before a destination is taken into probing */
+extern int inactive_threshold; /*!< number of successful requests,
+ before a destination is taken into active */
extern int ds_probing_mode;
extern str ds_outbound_proxy;
extern str ds_default_socket;
First of all excuse me if in the previous pull request I didn't follow guidelines by devel-git-commit guide :)
As anticipated in the previous request I made some changes to dispatcher module for counting successful ping requests before moving a destination from inactive to active state.
While doing this changes I noticed some misspelled variables (in code and docs too) and I fixed them. So I've used similar variables (with the corrected words) also for the new feature I developed.
In the case we leave the old variables introducing the new one we will have in the code:
"ds_probing_threshhold" vs "ds_inactive_threshold"
This behaviour could be confusing while editing the configuration file and also while reading the code.
As you requested I've separated the misspelled fixes in two commits one for the code and one for the docs.
To this pull request will follow the one with the changes developing the new feature.
You can merge this Pull Request by running:
git pull https://github.com/alezzandro/kamailio master
Or you can view, comment on it, or merge it online at:
https://github.com/kamailio/kamailio/pull/13
-- Commit Summary --
* dispatcher: Fixing misspelled words in some code's variables
* dispatcher: Fixing misspelled words in some docs
-- File Changes --
M modules/dispatcher/README (12)
M modules/dispatcher/config.c (4)
M modules/dispatcher/config.h (2)
M modules/dispatcher/dispatch.c (5)
M modules/dispatcher/dispatch.h (3)
M modules/dispatcher/dispatcher.c (11)
M modules/dispatcher/doc/dispatcher_admin.xml (4)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/13.patchhttps://github.com/kamailio/kamailio/pull/13.diff
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/13