sr-dev July 2018

sr-dev@lists.kamailio.org

17 participants
229 discussions

git:master:a1f5fbe2: dmq: release resources instead of just doing continue to next job
by Daniel-Constantin Mierla 31 Jul '18

31 Jul '18

Module: kamailio Branch: master Commit: a1f5fbe2c18246d4afefa44fd8a52612a5182a46 URL: https://github.com/kamailio/kamailio/commit/a1f5fbe2c18246d4afefa44fd8a5261… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-07-31T13:59:44+02:00 dmq: release resources instead of just doing continue to next job - for cases when processing of the job is not fully completed --- Modified: src/modules/dmq/worker.c --- Diff: https://github.com/kamailio/kamailio/commit/a1f5fbe2c18246d4afefa44fd8a5261… Patch: https://github.com/kamailio/kamailio/commit/a1f5fbe2c18246d4afefa44fd8a5261… --- diff --git a/src/modules/dmq/worker.c b/src/modules/dmq/worker.c index 9c7873560c..14aadfc998 100644 --- a/src/modules/dmq/worker.c +++ b/src/modules/dmq/worker.c @@ -114,7 +114,7 @@ void worker_loop(int id) current_job->msg, &peer_response, dmq_node); if(ret_value < 0) { LM_ERR("running job failed\n"); - continue; + goto nextjob; } /* add the body to the reply */ if(peer_response.body.s) { @@ -122,7 +122,7 @@ void worker_loop(int id) &peer_response.content_type) < 0) { LM_ERR("error adding lumps\n"); - continue; + goto nextjob; } } /* send the reply */ @@ -130,8 +130,12 @@ void worker_loop(int id) &peer_response.reason) < 0) { LM_ERR("error sending reply\n"); + } else { + LM_DBG("done sending reply\n"); } + worker->jobs_processed++; +nextjob: /* if body given, free the lumps and free the body */ if(peer_response.body.s) { del_nonshm_lump_rpl(&current_job->msg->reply_lump); @@ -141,10 +145,8 @@ void worker_loop(int id) free_to(current_job->msg->from->parsed); } - LM_DBG("sent reply\n"); shm_free(current_job->msg); shm_free(current_job); - worker->jobs_processed++; } } }

1 0

git:master:4a64fb95: carrierroute: docs - removed mi commands section
by Daniel-Constantin Mierla 31 Jul '18

31 Jul '18

Module: kamailio Branch: master Commit: 4a64fb95680b9efac47c79888f2a68bdcfb29ad6 URL: https://github.com/kamailio/kamailio/commit/4a64fb95680b9efac47c79888f2a68b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-07-31T13:58:29+02:00 carrierroute: docs - removed mi commands section --- Modified: src/modules/carrierroute/doc/carrierroute_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/4a64fb95680b9efac47c79888f2a68b… Patch: https://github.com/kamailio/kamailio/commit/4a64fb95680b9efac47c79888f2a68b…

1 0

git:master:de429e10: topos: docs - ids for sections with event routes
by Daniel-Constantin Mierla 29 Jul '18

29 Jul '18

Module: kamailio Branch: master Commit: de429e10eab9154eb1935d08537db6757dcd2b70 URL: https://github.com/kamailio/kamailio/commit/de429e10eab9154eb1935d08537db67… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-07-29T23:32:45+02:00 topos: docs - ids for sections with event routes --- Modified: src/modules/topos/doc/topos_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/de429e10eab9154eb1935d08537db67… Patch: https://github.com/kamailio/kamailio/commit/de429e10eab9154eb1935d08537db67… --- diff --git a/src/modules/topos/doc/topos_admin.xml b/src/modules/topos/doc/topos_admin.xml index 8a423fdc84..4c87d371c6 100644 --- a/src/modules/topos/doc/topos_admin.xml +++ b/src/modules/topos/doc/topos_admin.xml @@ -281,7 +281,7 @@ modparam("topos", "event_mode", 2) </section> <section> <title>Event Routes</title> - <section> + <section id="topos.e.msg_outgoing"> <title>event_route[topos:msg-outgoing]</title> <para> It is executed before doing topology stripping processing for an outgoing @@ -308,7 +308,7 @@ event_route[topos:msg-outgoing] { </programlisting> </example> </section> - <section> + <section id="topos.e.msg_sending"> <title>event_route[topos:msg-sending]</title> <para> It is executed before doing topology stripping processing for a SIP

1 0

git:master:674ab074: modules: readme files regenerated - uac ... [skip ci]
by Kamailio Dev 27 Jul '18

27 Jul '18

Module: kamailio Branch: master Commit: 674ab0749960398fdddfde3772376b1b2461c107 URL: https://github.com/kamailio/kamailio/commit/674ab0749960398fdddfde3772376b1… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2018-07-27T22:17:20+02:00 modules: readme files regenerated - uac ... [skip ci] --- Modified: src/modules/uac/README --- Diff: https://github.com/kamailio/kamailio/commit/674ab0749960398fdddfde3772376b1… Patch: https://github.com/kamailio/kamailio/commit/674ab0749960398fdddfde3772376b1… --- diff --git a/src/modules/uac/README b/src/modules/uac/README index dae255ee37..b833372b6a 100644 --- a/src/modules/uac/README +++ b/src/modules/uac/README @@ -840,8 +840,11 @@ failure_route[REMOTE_AUTH] { 6.1. event_route[uac:reply] - Event route executed for the final reply to the request set with - uac_req_send(). The associated $uac_req(evroute) has to be set to 1. + Event route executed for the final reply of the branch for the request + sent with uac_req_send(). The associated $uac_req(evroute) has to be + set to 1. If the request is challenged for authentication with 401/407, + then the event_route is executed twice, first for 401/407 and second + for final reply of the transaction. Example 1.33. event_route[uac:reply] usage ...

1 0

git:master:f8c46e63: modules: readme files regenerated - pipelimit ... [skip ci]
by The Root 27 Jul '18

27 Jul '18

Module: kamailio Branch: master Commit: f8c46e6395ed9cc4cbcf062a823a9530bdec62c9 URL: https://github.com/kamailio/kamailio/commit/f8c46e6395ed9cc4cbcf062a823a953… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: The Root <root(a)kamailio.org> Date: 2018-07-27T22:17:10+02:00 modules: readme files regenerated - pipelimit ... [skip ci] --- Modified: src/modules/pipelimit/README --- Diff: https://github.com/kamailio/kamailio/commit/f8c46e6395ed9cc4cbcf062a823a953… Patch: https://github.com/kamailio/kamailio/commit/f8c46e6395ed9cc4cbcf062a823a953… --- diff --git a/src/modules/pipelimit/README b/src/modules/pipelimit/README index 974eaed33b..3b48f2e0d6 100644 --- a/src/modules/pipelimit/README +++ b/src/modules/pipelimit/README @@ -51,12 +51,13 @@ Daniel-Constantin Mierla 5. RPC Commands - 5.1. pl.stats - 5.2. pl.set_pipe - 5.3. pl.get_pipes - 5.4. pl.set_pid - 5.5. pl.get_pid - 5.6. pl.push_load + 5.1. pl.list + 5.2. pl.stats + 5.3. pl.set_pipe + 5.4. pl.get_pipes + 5.5. pl.set_pid + 5.6. pl.get_pid + 5.7. pl.push_load List of Examples @@ -106,12 +107,13 @@ Chapter 1. Admin Guide 5. RPC Commands - 5.1. pl.stats - 5.2. pl.set_pipe - 5.3. pl.get_pipes - 5.4. pl.set_pid - 5.5. pl.get_pid - 5.6. pl.push_load + 5.1. pl.list + 5.2. pl.stats + 5.3. pl.set_pipe + 5.4. pl.get_pipes + 5.5. pl.set_pid + 5.6. pl.get_pid + 5.7. pl.push_load 1. Overview @@ -441,14 +443,30 @@ with unexpected retcode=$var(check_result)\n"); 5. RPC Commands - 5.1. pl.stats - 5.2. pl.set_pipe - 5.3. pl.get_pipes - 5.4. pl.set_pid - 5.5. pl.get_pid - 5.6. pl.push_load + 5.1. pl.list + 5.2. pl.stats + 5.3. pl.set_pipe + 5.4. pl.get_pipes + 5.5. pl.set_pid + 5.6. pl.get_pid + 5.7. pl.push_load -5.1. pl.stats +5.1. pl.list + + Lists the details of one or all pipes, respectively the attributes pipe + name (id), algorithm, limit and counter. + + Name: pl.list + + Parameters: name - (optional) pipe name + + RPC Command Format: +... +kamctl rpc pl.list +kamctl rpc pl.list testid +... + +5.2. pl.stats Lists the parameters and variables in the pipelimit module: pipe id, pipe load and pipe counter. @@ -458,9 +476,11 @@ with unexpected retcode=$var(check_result)\n"); Parameters: none RPC Command Format: - kamcmd pl.stats +... +kamcmd pl.stats +... -5.2. pl.set_pipe +5.3. pl.set_pipe Sets the pipe parameters for the given pipe id. @@ -472,9 +492,11 @@ with unexpected retcode=$var(check_result)\n"); * pipe_limit - the limit assigned to the given pipe id. RPC Command Format: - kamcmd pl.set_pipe 2 RED 10 +... +kamcmd pl.set_pipe 2 RED 10 +... -5.3. pl.get_pipes +5.4. pl.get_pipes Gets the list of in use pipes. @@ -483,9 +505,11 @@ with unexpected retcode=$var(check_result)\n"); Parameters: none RPC Command Format: - kamcmd pl.get_pipes +... +kamcmd pl.get_pipes +... -5.4. pl.set_pid +5.5. pl.set_pid Sets the PID Controller parameters for the Feedback Algorithm. @@ -497,9 +521,11 @@ with unexpected retcode=$var(check_result)\n"); * kd - the derivative parameter. RPC Command Format: - kamcmd pl.set_pid 0.5 0.5 0.5 +... +kamcmd pl.set_pid 0.5 0.5 0.5 +... -5.5. pl.get_pid +5.6. pl.get_pid Gets the list of in use PID Controller parameters. @@ -508,9 +534,11 @@ with unexpected retcode=$var(check_result)\n"); Parameters: none RPC Command Format: - kamcmd pl.get_pid +... +kamcmd pl.get_pid +... -5.6. pl.push_load +5.7. pl.push_load Force the value of the load parameter. This command is useful for testing the Feedback algorithm. @@ -522,4 +550,6 @@ with unexpected retcode=$var(check_result)\n"); smaller then 1.0). RPC Command Format: - kamcmd pl.push_load 0.85 +... +kamcmd pl.push_load 0.85 +...

1 0

git:master:1cec5e80: uac: docs - more details about execution of event_route[uac:reply]
by Daniel-Constantin Mierla 27 Jul '18

27 Jul '18

Module: kamailio Branch: master Commit: 1cec5e80036808badf19c8c72ec243b504f56e28 URL: https://github.com/kamailio/kamailio/commit/1cec5e80036808badf19c8c72ec243b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-07-27T22:00:47+02:00 uac: docs - more details about execution of event_route[uac:reply] --- Modified: src/modules/uac/doc/uac_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/1cec5e80036808badf19c8c72ec243b… Patch: https://github.com/kamailio/kamailio/commit/1cec5e80036808badf19c8c72ec243b… --- diff --git a/src/modules/uac/doc/uac_admin.xml b/src/modules/uac/doc/uac_admin.xml index 162f491b31..8047c67c2b 100644 --- a/src/modules/uac/doc/uac_admin.xml +++ b/src/modules/uac/doc/uac_admin.xml @@ -985,9 +985,11 @@ failure_route[REMOTE_AUTH] { <function moreinfo="none">event_route[uac:reply]</function> </title> <para> - Event route executed for the final reply to the request set with - uac_req_send(). The associated $uac_req(evroute) has to be set to - 1. + Event route executed for the final reply of the branch for the + request sent with uac_req_send(). The associated $uac_req(evroute) + has to be set to 1. If the request is challenged for authentication + with 401/407, then the event_route is executed twice, first for + 401/407 and second for final reply of the transaction. </para> <example> <title><function>event_route[uac:reply]</function> usage</title>

1 0

[kamailio/kamailio] topos_redis not included in debian packages for kamailio-redis-modules (#1603)
by William King 27 Jul '18

27 Jul '18

### Description Installing new instance on Debian Stretch, using 5.1 release repo the module topos_redis is not built in the available packages. #### Reproduction root@sbc1-1:~# apt-file update Hit:1 http://security.debian.org stretch/updates InRelease Ign:2 http://deb.debian.org/debian stretch InRelease Hit:3 http://deb.debian.org/debian stretch-updates InRelease Hit:4 http://deb.debian.org/debian stretch Release Hit:5 http://packages.treasuredata.com/3/debian/stretch stretch InRelease Hit:7 http://deb.kamailio.org/kamailiodev-nightly stretch InRelease Hit:8 http://deb.kamailio.org/kamailio51 stretch InRelease Reading package lists... Done Building dependency tree Reading state information... Done root@sbc1-1:~# apt-file search topos_redis root@sbc1-1:~# -- 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/issues/1603

3 3

topos is masking the contact in 302 redirect response from outside
by Andrew Pogrebennyk 27 Jul '18

27 Jul '18

Hello, using kamailio 5.1.4: the topos is changing the Contact in 302 response from the UA. Has anyone else observed it so it's a known issue or should I file a proper bugreport? At a glance, I don't see any related fixes in master, so i guess i will do it anyway this week.. In meantime I'll try to workaround it for my purpose using new even route [topos:msg-sending] Thanks in advance for your hints. Regards, Andrew

2 3

git:5.1:e2eda137: dialog: unset DMQ flag on replicated dialog removal
by Charles Chance 26 Jul '18

26 Jul '18

Module: kamailio Branch: 5.1 Commit: e2eda1376def48ab1c78db7a766c1811d19205be URL: https://github.com/kamailio/kamailio/commit/e2eda1376def48ab1c78db7a766c181… Author: Charles Chance <charles.chance(a)sipcentric.com> Committer: Charles Chance <charles.chance(a)sipcentric.com> Date: 2018-07-26T13:54:31+01:00 dialog: unset DMQ flag on replicated dialog removal - prevents looping back to other nodes - reported by Patrick Murphy on sr-users list (cherry picked from commit a84a3ea618f0e602a8892c37fce7f4e72ab7371c) --- Modified: src/modules/dialog/dlg_dmq.c --- Diff: https://github.com/kamailio/kamailio/commit/e2eda1376def48ab1c78db7a766c181… Patch: https://github.com/kamailio/kamailio/commit/e2eda1376def48ab1c78db7a766c181… --- diff --git a/src/modules/dialog/dlg_dmq.c b/src/modules/dialog/dlg_dmq.c index 8ed9dce1d3..0cfd8a522e 100644 --- a/src/modules/dialog/dlg_dmq.c +++ b/src/modules/dialog/dlg_dmq.c @@ -365,6 +365,7 @@ int dlg_dmq_handle_msg(struct sip_msg* msg, peer_reponse_t* resp, dmq_node_t* no } /* prevent DB sync */ dlg->dflags |= DLG_FLAG_NEW; + dlg->iflags &= ~DLG_IFLAG_DMQ_SYNC; unref++; break;

1 0

git:master:487fbf45: Merge pull request #1601 from kamailio/cchance/dlg_dmq
by GitHub 26 Jul '18

26 Jul '18

Module: kamailio Branch: master Commit: 487fbf45cea288e743cbb87e352e76bb1bc05b87 URL: https://github.com/kamailio/kamailio/commit/487fbf45cea288e743cbb87e352e76b… Author: Charles Chance <charles.chance(a)sipcentric.com> Committer: GitHub <noreply(a)github.com> Date: 2018-07-26T13:51:21+01:00 Merge pull request #1601 from kamailio/cchance/dlg_dmq dialog: unset DMQ flag on replicated dialog removal --- Modified: src/modules/dialog/dlg_dmq.c --- Diff: https://github.com/kamailio/kamailio/commit/487fbf45cea288e743cbb87e352e76b… Patch: https://github.com/kamailio/kamailio/commit/487fbf45cea288e743cbb87e352e76b… --- diff --git a/src/modules/dialog/dlg_dmq.c b/src/modules/dialog/dlg_dmq.c index 8ed9dce1d3..0cfd8a522e 100644 --- a/src/modules/dialog/dlg_dmq.c +++ b/src/modules/dialog/dlg_dmq.c @@ -365,6 +365,7 @@ int dlg_dmq_handle_msg(struct sip_msg* msg, peer_reponse_t* resp, dmq_node_t* no } /* prevent DB sync */ dlg->dflags |= DLG_FLAG_NEW; + dlg->iflags &= ~DLG_IFLAG_DMQ_SYNC; unref++; break;

1 0

← Newer
1
...
6
7
8
9
10
11
12
...
23
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev July 2018