sr-dev

sr-dev@lists.kamailio.org

22 participants
45186 discussions

[kamailio/kamailio] dispatcher: don't reset load table when adding/removing a destination… (#2342)
by Federico Cabiddu 07 Jun '20

07 Jun '20

… or reloading the list   #### Pre-Submission Checklist    - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [x] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [x] Small bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist:  - [ ] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description When a destination is added/removed or the whole list is reloaded, the calls that are being routed to another destination will fail. This because ds_load_replace is returning an error not finding in the table the entry for the call being rerouted. Another side effect of this is that when adding/removing destinations the actual load of the endpoints will be wrong since the current calls will never be deleted. I've removed the calls to ds_clean_slots and modified ds_load_replace to not exit when the old destination is not found (the endpoint could have been removed after the failover list had been created) and to return a new error code (-2) when the new destination is not found (for the same reason). Finally I've modified ds_update_dst to skip the current selected destination and select the next if ds_load_replace returns -2. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2342 -- Commit Summary -- * dispatcher: don't reset load table when adding/removing a destination or reloading the list -- File Changes -- M src/modules/dispatcher/dispatch.c (37) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2342.patch https://github.com/kamailio/kamailio/pull/2342.diff -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/2342

3 4

git:5.3:88676c2f: dispatcher: don't reset load table when adding/removing a destination or reloading the list
by Henning Westerholt 07 Jun '20

07 Jun '20

Module: kamailio Branch: 5.3 Commit: 88676c2f083c2d7e52ae6b42f5cbe83f64357865 URL: https://github.com/kamailio/kamailio/commit/88676c2f083c2d7e52ae6b42f5cbe83… Author: Federico Cabiddu <federico.cabiddu(a)gmail.com> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-06-07T09:13:21Z dispatcher: don't reset load table when adding/removing a destination or reloading the list (cherry picked from commit 4099a6caf856c9af4d7d89aec73db143efbcd40b) --- Modified: src/modules/dispatcher/dispatch.c --- Diff: https://github.com/kamailio/kamailio/commit/88676c2f083c2d7e52ae6b42f5cbe83… Patch: https://github.com/kamailio/kamailio/commit/88676c2f083c2d7e52ae6b42f5cbe83… --- diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c index 36cf884b48..d66a48e919 100644 --- a/src/modules/dispatcher/dispatch.c +++ b/src/modules/dispatcher/dispatch.c @@ -870,7 +870,7 @@ int ds_load_list(char *lfile) /* Update list - should it be sync'ed? */ _ds_list_nr = setn; *crt_idx = *next_idx; - ds_ht_clear_slots(_dsht_load); + ds_log_sets(); return 0; @@ -1118,7 +1118,6 @@ int ds_load_db(void) /* update data - should it be sync'ed? */ _ds_list_nr = setn; *crt_idx = *next_idx; - ds_ht_clear_slots(_dsht_load); ds_log_sets(); @@ -1609,22 +1608,24 @@ int ds_load_replace(struct sip_msg *msg, str *duid) break; } } + /* old destination has not been found: has been removed meanwhile? */ if(olddst == -1) { - ds_unlock_cell(_dsht_load, &msg->callid->body); - LM_ERR("old destination address not found for [%d, %.*s]\n", set, + LM_WARN("old destination address not found for [%d, %.*s]\n", set, it->duid.len, it->duid.s); - return -1; - } + } if(newdst == -1) { + /* new destination has not been found: has been removed meanwhile? */ ds_unlock_cell(_dsht_load, &msg->callid->body); LM_ERR("new destination address not found for [%d, %.*s]\n", set, duid->len, duid->s); - return -1; + return -2; } ds_unlock_cell(_dsht_load, &msg->callid->body); ds_del_cell(_dsht_load, &msg->callid->body); - DS_LOAD_DEC(idx, olddst); + + if(olddst != -1) + DS_LOAD_DEC(idx, olddst); if(ds_load_add(msg, idx, set, newdst) < 0) { LM_ERR("unable to replace destination load [%.*s / %.*s]\n", duid->len, @@ -2317,6 +2318,7 @@ int ds_manage_routes(sip_msg_t *msg, ds_select_state_t *rstate) int ds_update_dst(struct sip_msg *msg, int upos, int mode) { + int ret; socket_info_t *sock = NULL; sr_xavp_t *rxavp = NULL; sr_xavp_t *lxavp = NULL; @@ -2329,6 +2331,7 @@ int ds_update_dst(struct sip_msg *msg, int upos, int mode) } } +next_dst: rxavp = xavp_get(&ds_xavp_dst, NULL); if(rxavp == NULL || rxavp->val.type != SR_XTYPE_XAVP) { LM_DBG("no xavp with previous destination record\n"); @@ -2376,12 +2379,18 @@ int ds_update_dst(struct sip_msg *msg, int upos, int mode) return 1; } if(upos == DS_USE_NEXT) { - if(ds_load_replace(msg, &lxavp->val.v.s) < 0) { - LM_ERR("cannot update load distribution\n"); - return -1; + ret = ds_load_replace(msg, &lxavp->val.v.s); + switch(ret) { + case 0: + break; + case -2: + LM_ERR("cannot update load with %.*s, skipping dst.\n", lxavp->val.v.s.len, lxavp->val.v.s.s); + goto next_dst; + default: + LM_ERR("cannot update load distribution\n"); + return -1; } } - return 1; } @@ -2432,7 +2441,7 @@ int ds_add_dst(int group, str *address, int flags) _ds_list_nr = setn; *crt_idx = *next_idx; - ds_ht_clear_slots(_dsht_load); + ds_log_sets(); return 0; @@ -2487,7 +2496,7 @@ int ds_remove_dst(int group, str *address) _ds_list_nr = setn; *crt_idx = *next_idx; - ds_ht_clear_slots(_dsht_load); + ds_log_sets(); return 0;

1 0

git:5.3:ac47ce90: dispatcher: sync on load management operations
by Henning Westerholt 07 Jun '20

07 Jun '20

Module: kamailio Branch: 5.3 Commit: ac47ce9019616ffda2d206f42fbecb142a5fc55d URL: https://github.com/kamailio/kamailio/commit/ac47ce9019616ffda2d206f42fbecb1… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-06-07T09:13:14Z dispatcher: sync on load management operations - use locks to increment/decrement call load and get least loaded dst - GH #2322 (cherry picked from commit d14833d9d1ad6cc92c65e279b7dd89b3b922f129) --- Modified: src/modules/dispatcher/dispatch.c --- Diff: https://github.com/kamailio/kamailio/commit/ac47ce9019616ffda2d206f42fbecb1… Patch: https://github.com/kamailio/kamailio/commit/ac47ce9019616ffda2d206f42fbecb1… --- diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c index aaa7b4a2d5..36cf884b48 100644 --- a/src/modules/dispatcher/dispatch.c +++ b/src/modules/dispatcher/dispatch.c @@ -79,6 +79,22 @@ #define DS_ALG_RELWEIGHT 11 #define DS_ALG_PARALLEL 12 +/* increment call load */ +#define DS_LOAD_INC(dgrp, didx) do { \ + lock_get(&(dgrp)->lock); \ + (dgrp)->dlist[didx].dload++; \ + lock_release(&(dgrp)->lock); \ + } while(0) + +/* decrement call load */ +#define DS_LOAD_DEC(dgrp, didx) do { \ + lock_get(&(dgrp)->lock); \ + if(likely((dgrp)->dlist[didx].dload > 0)) { \ + (dgrp)->dlist[didx].dload--; \ + } \ + lock_release(&(dgrp)->lock); \ + } while(0) + static int _ds_table_version = DS_TABLE_VERSION; static ds_ht_t *_dsht_load = NULL; @@ -1506,6 +1522,7 @@ int ds_get_leastloaded(ds_set_t *dset) k = -1; t = 0x7fffffff; /* high load */ + lock_get(&dset->lock); \ for(j = 0; j < dset->nr; j++) { if(!ds_skip_dst(dset->dlist[j].flags) && (dset->dlist[j].attrs.maxload == 0 @@ -1517,6 +1534,7 @@ int ds_get_leastloaded(ds_set_t *dset) } } } + lock_release(&dset->lock); \ return k; } @@ -1538,7 +1556,7 @@ int ds_load_add(struct sip_msg *msg, ds_set_t *dset, int setid, int dst) msg->callid->body.s); return -1; } - dset->dlist[dst].dload++; + DS_LOAD_INC(dset, dst); return 0; } @@ -1606,8 +1624,7 @@ int ds_load_replace(struct sip_msg *msg, str *duid) ds_unlock_cell(_dsht_load, &msg->callid->body); ds_del_cell(_dsht_load, &msg->callid->body); - if(idx->dlist[olddst].dload > 0) - idx->dlist[olddst].dload--; + DS_LOAD_DEC(idx, olddst); if(ds_load_add(msg, idx, set, newdst) < 0) { LM_ERR("unable to replace destination load [%.*s / %.*s]\n", duid->len, @@ -1646,8 +1663,7 @@ int ds_load_remove_byid(int set, str *duid) return -1; } - if(idx->dlist[olddst].dload > 0) - idx->dlist[olddst].dload--; + DS_LOAD_DEC(idx, olddst); return 0; }

1 0

git:5.2:c9ad77e7: dispatcher: don't reset load table when adding/removing a destination or reloading the list
by Henning Westerholt 07 Jun '20

07 Jun '20

Module: kamailio Branch: 5.2 Commit: c9ad77e7d0ff7fb6ff4b9763cdea9b84f5ba739e URL: https://github.com/kamailio/kamailio/commit/c9ad77e7d0ff7fb6ff4b9763cdea9b8… Author: Henning Westerholt <hw(a)skalatan.de> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-06-07T09:20:48Z dispatcher: don't reset load table when adding/removing a destination or reloading the list - manual backport from 4099a6caf856c9a --- Modified: src/modules/dispatcher/dispatch.c --- Diff: https://github.com/kamailio/kamailio/commit/c9ad77e7d0ff7fb6ff4b9763cdea9b8… Patch: https://github.com/kamailio/kamailio/commit/c9ad77e7d0ff7fb6ff4b9763cdea9b8… --- diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c index 4561649ebc..77294c7782 100644 --- a/src/modules/dispatcher/dispatch.c +++ b/src/modules/dispatcher/dispatch.c @@ -851,7 +851,7 @@ int ds_load_list(char *lfile) /* Update list - should it be sync'ed? */ _ds_list_nr = setn; *crt_idx = *next_idx; - ds_ht_clear_slots(_dsht_load); + ds_log_sets(); return 0; @@ -1096,7 +1096,6 @@ int ds_load_db(void) /* update data - should it be sync'ed? */ _ds_list_nr = setn; *crt_idx = *next_idx; - ds_ht_clear_slots(_dsht_load); ds_log_sets(); @@ -1587,22 +1586,24 @@ int ds_load_replace(struct sip_msg *msg, str *duid) break; } } + /* old destination has not been found: has been removed meanwhile? */ if(olddst == -1) { - ds_unlock_cell(_dsht_load, &msg->callid->body); - LM_ERR("old destination address not found for [%d, %.*s]\n", set, + LM_WARN("old destination address not found for [%d, %.*s]\n", set, it->duid.len, it->duid.s); - return -1; - } + } if(newdst == -1) { + /* new destination has not been found: has been removed meanwhile? */ ds_unlock_cell(_dsht_load, &msg->callid->body); LM_ERR("new destination address not found for [%d, %.*s]\n", set, duid->len, duid->s); - return -1; + return -2; } ds_unlock_cell(_dsht_load, &msg->callid->body); ds_del_cell(_dsht_load, &msg->callid->body); - DS_LOAD_DEC(idx, olddst); + + if(olddst != -1) + DS_LOAD_DEC(idx, olddst); if(ds_load_add(msg, idx, set, newdst) < 0) { LM_ERR("unable to replace destination load [%.*s / %.*s]\n", duid->len, @@ -2310,6 +2311,7 @@ int ds_manage_routes(sip_msg_t *msg, ds_select_state_t *rstate) int ds_update_dst(struct sip_msg *msg, int upos, int mode) { + int ret; socket_info_t *sock = NULL; sr_xavp_t *rxavp = NULL; sr_xavp_t *lxavp = NULL; @@ -2322,6 +2324,7 @@ int ds_update_dst(struct sip_msg *msg, int upos, int mode) } } +next_dst: rxavp = xavp_get(&ds_xavp_dst, NULL); if(rxavp == NULL || rxavp->val.type != SR_XTYPE_XAVP) { LM_DBG("no xavp with previous destination record\n"); @@ -2369,12 +2372,18 @@ int ds_update_dst(struct sip_msg *msg, int upos, int mode) return 1; } if(upos == DS_USE_NEXT) { - if(ds_load_replace(msg, &lxavp->val.v.s) < 0) { - LM_ERR("cannot update load distribution\n"); - return -1; + ret = ds_load_replace(msg, &lxavp->val.v.s); + switch(ret) { + case 0: + break; + case -2: + LM_ERR("cannot update load with %.*s, skipping dst.\n", lxavp->val.v.s.len, lxavp->val.v.s.s); + goto next_dst; + default: + LM_ERR("cannot update load distribution\n"); + return -1; } } - return 1; }

1 0

git:5.2:75fdcec6: dispatcher: sync on load management operations
by Henning Westerholt 07 Jun '20

07 Jun '20

Module: kamailio Branch: 5.2 Commit: 75fdcec6241fc4439805f7781114d1f44cbe2528 URL: https://github.com/kamailio/kamailio/commit/75fdcec6241fc4439805f7781114d1f… Author: Henning Westerholt <hw(a)skalatan.de> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-06-07T09:16:04Z dispatcher: sync on load management operations - use locks to increment/decrement call load and get least loaded dst - GH #2322 - manual backport of d14833d9d1a --- Modified: src/modules/dispatcher/dispatch.c --- Diff: https://github.com/kamailio/kamailio/commit/75fdcec6241fc4439805f7781114d1f… Patch: https://github.com/kamailio/kamailio/commit/75fdcec6241fc4439805f7781114d1f… --- diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c index 71bbb6239f..4561649ebc 100644 --- a/src/modules/dispatcher/dispatch.c +++ b/src/modules/dispatcher/dispatch.c @@ -79,6 +79,22 @@ #define DS_ALG_RELWEIGHT 11 #define DS_ALG_PARALLEL 12 +/* increment call load */ +#define DS_LOAD_INC(dgrp, didx) do { \ + lock_get(&(dgrp)->lock); \ + (dgrp)->dlist[didx].dload++; \ + lock_release(&(dgrp)->lock); \ + } while(0) + +/* decrement call load */ +#define DS_LOAD_DEC(dgrp, didx) do { \ + lock_get(&(dgrp)->lock); \ + if(likely((dgrp)->dlist[didx].dload > 0)) { \ + (dgrp)->dlist[didx].dload--; \ + } \ + lock_release(&(dgrp)->lock); \ + } while(0) + static int _ds_table_version = DS_TABLE_VERSION; static ds_ht_t *_dsht_load = NULL; @@ -1484,6 +1500,7 @@ int ds_get_leastloaded(ds_set_t *dset) k = -1; t = 0x7fffffff; /* high load */ + lock_get(&dset->lock); \ for(j = 0; j < dset->nr; j++) { if(!ds_skip_dst(dset->dlist[j].flags) && (dset->dlist[j].attrs.maxload == 0 @@ -1495,6 +1512,7 @@ int ds_get_leastloaded(ds_set_t *dset) } } } + lock_release(&dset->lock); \ return k; } @@ -1516,7 +1534,7 @@ int ds_load_add(struct sip_msg *msg, ds_set_t *dset, int setid, int dst) msg->callid->body.s); return -1; } - dset->dlist[dst].dload++; + DS_LOAD_INC(dset, dst); return 0; } @@ -1584,8 +1602,7 @@ int ds_load_replace(struct sip_msg *msg, str *duid) ds_unlock_cell(_dsht_load, &msg->callid->body); ds_del_cell(_dsht_load, &msg->callid->body); - if(idx->dlist[olddst].dload > 0) - idx->dlist[olddst].dload--; + DS_LOAD_DEC(idx, olddst); if(ds_load_add(msg, idx, set, newdst) < 0) { LM_ERR("unable to replace destination load [%.*s / %.*s]\n", duid->len, @@ -1673,8 +1690,7 @@ int ds_load_remove_byid(int set, str *duid) return -1; } - if(idx->dlist[olddst].dload > 0) - idx->dlist[olddst].dload--; + DS_LOAD_DEC(idx, olddst); return 0; }

1 0

git:5.2:50f60972: modules: readme files regenerated - modules ... [skip ci]
by Kamailio Dev 05 Jun '20

05 Jun '20

Module: kamailio Branch: 5.2 Commit: 50f60972abbde849c3e637cece7b08777ff3a515 URL: https://github.com/kamailio/kamailio/commit/50f60972abbde849c3e637cece7b087… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2020-06-05T19:46:32+02:00 modules: readme files regenerated - modules ... [skip ci] --- Modified: src/modules/siptrace/README --- Diff: https://github.com/kamailio/kamailio/commit/50f60972abbde849c3e637cece7b087… Patch: https://github.com/kamailio/kamailio/commit/50f60972abbde849c3e637cece7b087… --- diff --git a/src/modules/siptrace/README b/src/modules/siptrace/README index 69b4fb247d..d7c1a25a5f 100644 --- a/src/modules/siptrace/README +++ b/src/modules/siptrace/README @@ -424,7 +424,8 @@ modparam("siptrace", "trace_delayed", 1) The local interface in the form of SIP URI from where to send the duplicated traffic. In the absence of this parameter Kamailio - automatically picks an interface. + automatically picks an interface. This parameter is only used for HEP + forwarding, not for SIP forwarding mode. Example 1.17. Set force_send_sock parameter ...

1 0

git:5.2:144aeaa5: siptrace: extend docs that force_send_socket parameter is only used for HEP mode
by Henning Westerholt 05 Jun '20

05 Jun '20

Module: kamailio Branch: 5.2 Commit: 144aeaa5fd7392f5ebcb4e601c3a23d5f7ba86fe URL: https://github.com/kamailio/kamailio/commit/144aeaa5fd7392f5ebcb4e601c3a23d… Author: Henning Westerholt <hw(a)skalatan.de> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-06-05T17:35:44Z siptrace: extend docs that force_send_socket parameter is only used for HEP mode (cherry picked from commit 1b27855539f42f5dece0232c90ee0857ca012ebd) --- Modified: src/modules/siptrace/doc/siptrace_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/144aeaa5fd7392f5ebcb4e601c3a23d… Patch: https://github.com/kamailio/kamailio/commit/144aeaa5fd7392f5ebcb4e601c3a23d… --- diff --git a/src/modules/siptrace/doc/siptrace_admin.xml b/src/modules/siptrace/doc/siptrace_admin.xml index 0359b0c1c8..f81cf2cc74 100644 --- a/src/modules/siptrace/doc/siptrace_admin.xml +++ b/src/modules/siptrace/doc/siptrace_admin.xml @@ -462,7 +462,8 @@ modparam("siptrace", "trace_delayed", 1) <para> The local interface in the form of SIP URI from where to send the duplicated traffic. In the absence of this parameter - &kamailio; automatically picks an interface. + &kamailio; automatically picks an interface. This parameter is + only used for HEP forwarding, not for SIP forwarding mode. </para> <example> <title>Set <varname>force_send_sock</varname> parameter</title>

1 0

git:5.3:0dd7f9e9: modules: readme files regenerated - modules ... [skip ci]
by Kamailio Dev 05 Jun '20

05 Jun '20

Module: kamailio Branch: 5.3 Commit: 0dd7f9e939c2cc54b338a1432847622f7c4e385f URL: https://github.com/kamailio/kamailio/commit/0dd7f9e939c2cc54b338a1432847622… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2020-06-05T19:31:20+02:00 modules: readme files regenerated - modules ... [skip ci] --- Modified: src/modules/siptrace/README --- Diff: https://github.com/kamailio/kamailio/commit/0dd7f9e939c2cc54b338a1432847622… Patch: https://github.com/kamailio/kamailio/commit/0dd7f9e939c2cc54b338a1432847622… --- diff --git a/src/modules/siptrace/README b/src/modules/siptrace/README index df5b99f7cc..65a3226c89 100644 --- a/src/modules/siptrace/README +++ b/src/modules/siptrace/README @@ -427,7 +427,8 @@ modparam("siptrace", "trace_delayed", 1) The local interface in the form of SIP URI from where to send the duplicated traffic. In the absence of this parameter Kamailio - automatically picks an interface. + automatically picks an interface. This parameter is only used for HEP + forwarding, not for SIP forwarding mode. Example 1.17. Set force_send_sock parameter ...

1 0

git:5.3:1b278555: siptrace: extend docs that force_send_socket parameter is only used for HEP mode
by Henning Westerholt 05 Jun '20

05 Jun '20

Module: kamailio Branch: 5.3 Commit: 1b27855539f42f5dece0232c90ee0857ca012ebd URL: https://github.com/kamailio/kamailio/commit/1b27855539f42f5dece0232c90ee085… Author: Henning Westerholt <hw(a)skalatan.de> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-06-05T17:29:59Z siptrace: extend docs that force_send_socket parameter is only used for HEP mode --- Modified: src/modules/siptrace/doc/siptrace_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/1b27855539f42f5dece0232c90ee085… Patch: https://github.com/kamailio/kamailio/commit/1b27855539f42f5dece0232c90ee085… --- diff --git a/src/modules/siptrace/doc/siptrace_admin.xml b/src/modules/siptrace/doc/siptrace_admin.xml index f5be1c663c..57c4b2a8ae 100644 --- a/src/modules/siptrace/doc/siptrace_admin.xml +++ b/src/modules/siptrace/doc/siptrace_admin.xml @@ -455,7 +455,8 @@ modparam("siptrace", "trace_delayed", 1) <para> The local interface in the form of SIP URI from where to send the duplicated traffic. In the absence of this parameter - &kamailio; automatically picks an interface. + &kamailio; automatically picks an interface. This parameter is + only used for HEP forwarding, not for SIP forwarding mode. </para> <example> <title>Set <varname>force_send_sock</varname> parameter</title>

1 0

git:master:c44ccd0e: modules: readme files regenerated - pv_headers ... [skip ci]
by Kamailio Dev 05 Jun '20

05 Jun '20

Module: kamailio Branch: master Commit: c44ccd0e66715f33620ddfc93cfeeb4dcc65a5fc URL: https://github.com/kamailio/kamailio/commit/c44ccd0e66715f33620ddfc93cfeeb4… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2020-06-05T19:01:14+02:00 modules: readme files regenerated - pv_headers ... [skip ci] --- Modified: src/modules/pv_headers/README --- Diff: https://github.com/kamailio/kamailio/commit/c44ccd0e66715f33620ddfc93cfeeb4… Patch: https://github.com/kamailio/kamailio/commit/c44ccd0e66715f33620ddfc93cfeeb4… --- diff --git a/src/modules/pv_headers/README b/src/modules/pv_headers/README index ec6e766ba5..828c7d1a13 100644 --- a/src/modules/pv_headers/README +++ b/src/modules/pv_headers/README @@ -27,7 +27,7 @@ Victor Seva 3. Parameters - 3.1. xavp_name (string) + 3.1. xavi_name (string) 3.2. header_value_size (int) 3.3. header_collect_flag (int) 3.4. header_apply_flag (int) @@ -49,7 +49,7 @@ Victor Seva List of Examples - 1.1. Set xavp_name parameter + 1.1. Set xavi_name parameter 1.2. Set header_value_size parameter 1.3. Set header_collect_flag parameter 1.4. Set header_apply_flag parameter @@ -69,7 +69,7 @@ Chapter 1. Admin Guide 3. Parameters - 3.1. xavp_name (string) + 3.1. xavi_name (string) 3.2. header_value_size (int) 3.3. header_collect_flag (int) 3.4. header_apply_flag (int) @@ -92,7 +92,7 @@ Chapter 1. Admin Guide 1. Overview The main goal of the module is to offload the intermediate header - processing into the XAVP dynamic container as well as provide with high + processing into the XAVI dynamic container as well as provide with high level methods and pseudovariables to simplify SIP message header modifications. @@ -116,7 +116,7 @@ Chapter 1. Admin Guide 3. Parameters - 3.1. xavp_name (string) + 3.1. xavi_name (string) 3.2. header_value_size (int) 3.3. header_collect_flag (int) 3.4. header_apply_flag (int) @@ -124,19 +124,19 @@ Chapter 1. Admin Guide 3.6. split_headers (string) 3.7. auto_msg (int) -3.1. xavp_name (string) +3.1. xavi_name (string) - Name of the XAVP there the collected headers are stored. + Name of the XAVI where the collected headers are stored. Default value is “headers”. - Example 1.1. Set xavp_name parameter + Example 1.1. Set xavi_name parameter ... -modparam("pv_headers", "xavp_name", "pvh") +modparam("pv_headers", "xavi_name", "headers") ... - Result: $xavp(headers[0]=>From) $xavp(headers[0]=>To) - $xavp(headers[0]=>Call-ID) + Result: $xavi(headers[0]=>From) $xavi(headers[0]=>To) + $xavi(headers[0]=>Call-ID) 3.2. header_value_size (int) @@ -228,7 +228,7 @@ modparam("pv_headers", "split_headers", "Diversion") Example 1.7. Set auto_msg parameter ... -modparam("pvh", "auto_msg", 1) +modparam("pv_headers", "auto_msg", 1) ... 4. Functions

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev