sr-dev January 2019

sr-dev@lists.kamailio.org

27 participants
301 discussions

git:5.0:9e006026: tm: Add fixup for PVS in fixup_t_check_status
by Hugh Waite 09 Jan '19

09 Jan '19

Module: kamailio Branch: 5.0 Commit: 9e00602623107ccba54a3a2ddf189c0e40432fc0 URL: https://github.com/kamailio/kamailio/commit/9e00602623107ccba54a3a2ddf189c0… Author: Hugh Waite <hughw273(a)gmail.com> Committer: Hugh Waite <hughw273(a)gmail.com> Date: 2019-01-09T10:19:59Z tm: Add fixup for PVS in fixup_t_check_status - Allows PVs including $avp(...) to be passed to t_check_status --- Modified: src/modules/tm/tm.c --- Diff: https://github.com/kamailio/kamailio/commit/9e00602623107ccba54a3a2ddf189c0… Patch: https://github.com/kamailio/kamailio/commit/9e00602623107ccba54a3a2ddf189c0… --- diff --git a/src/modules/tm/tm.c b/src/modules/tm/tm.c index 6c493a9860..e4874108d0 100644 --- a/src/modules/tm/tm.c +++ b/src/modules/tm/tm.c @@ -667,6 +667,9 @@ static int fixup_t_check_status(void** param, int param_no) { int ret; + ret = fix_param(FPARAM_PVS, param); + if (ret <= 0) return ret; + ret = fix_param(FPARAM_AVP, param); if (ret <= 0) return ret;

1 0

git:master:7a619c8f: modules: readme files regenerated - dialog ... [skip ci]
by Kamailio Dev 09 Jan '19

09 Jan '19

Module: kamailio Branch: master Commit: 7a619c8f014a3c402942ef478ee76ce237b7e21a URL: https://github.com/kamailio/kamailio/commit/7a619c8f014a3c402942ef478ee76ce… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2019-01-09T10:01:48+01:00 modules: readme files regenerated - dialog ... [skip ci] --- Modified: src/modules/dialog/README --- Diff: https://github.com/kamailio/kamailio/commit/7a619c8f014a3c402942ef478ee76ce… Patch: https://github.com/kamailio/kamailio/commit/7a619c8f014a3c402942ef478ee76ce… --- diff --git a/src/modules/dialog/README b/src/modules/dialog/README index d3bc615b53..4e3f43e99b 100644 --- a/src/modules/dialog/README +++ b/src/modules/dialog/README @@ -438,9 +438,10 @@ Chapter 1. Admin Guide 2. How it works - To create the dialog associated with an initial request, the flag - “dlg_flag” (Section 6.4, “dlg_flag (integer)”) must be set before - creating the corresponding transaction. + To create the dialog associated with an initial INVITE request, execute + the function “dlg_manage()” or set the flag specified by parameter + “dlg_flag” (Section 6.4, “dlg_flag (integer)”) before creating the + corresponding transaction. The dialog is automatically destroyed when a “BYE” is received. In case of no “BYE”, the dialog lifetime is controlled via the default timeout @@ -623,6 +624,10 @@ modparam("dialog", "rr_param", "xyz") Flag to be used for marking if a dialog should be constructed for the current request (this make sense only for initial requests). + Note: it is not needed to set this parameter and its corresponding flag + for initial request in case the “dlg_manage()” function is used. In + other words, using “dlg_manage()” alone is enough. + Default value is “none”. Example 1.4. Set dlg_flag parameter

1 0

git:master:e4dfe358: dialog: docs - more info about using the flag and dlg_manage()
by Daniel-Constantin Mierla 09 Jan '19

09 Jan '19

Module: kamailio Branch: master Commit: e4dfe3587943d69d556b15df3492cf38c3b21a5f URL: https://github.com/kamailio/kamailio/commit/e4dfe3587943d69d556b15df3492cf3… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2019-01-09T09:50:12+01:00 dialog: docs - more info about using the flag and dlg_manage() --- Modified: src/modules/dialog/doc/dialog_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/e4dfe3587943d69d556b15df3492cf3… Patch: https://github.com/kamailio/kamailio/commit/e4dfe3587943d69d556b15df3492cf3… --- diff --git a/src/modules/dialog/doc/dialog_admin.xml b/src/modules/dialog/doc/dialog_admin.xml index ca0109b31f..9adca735db 100644 --- a/src/modules/dialog/doc/dialog_admin.xml +++ b/src/modules/dialog/doc/dialog_admin.xml @@ -45,9 +45,10 @@ <section> <title>How it works</title> <para> - To create the dialog associated with an initial request, the flag - <quote>dlg_flag</quote> (<xref linkend="dialog.p.flag"/>) must be set before - creating the corresponding transaction. + To create the dialog associated with an initial INVITE request, execute + the function <quote>dlg_manage()</quote> or set the flag specified by + parameter <quote>dlg_flag</quote> (<xref linkend="dialog.p.flag"/>) + before creating the corresponding transaction. </para> <para> The dialog is automatically destroyed when a <quote>BYE</quote> is @@ -256,6 +257,12 @@ modparam("dialog", "rr_param", "xyz") current request (this make sense only for initial requests). </para> <para> + Note: it is not needed to set this parameter and its corresponding flag + for initial request in case the <quote>dlg_manage()</quote> function + is used. In other words, using <quote>dlg_manage()</quote> alone is + enough. + </para> + <para> <emphasis> Default value is <quote>none</quote>. </emphasis>

1 0

[kamailio/kamailio] Modules: rtpengine-rtpengine_query exported to ksr routes (#1792)
by Surendra Tiwari 09 Jan '19

09 Jan '19

#### Pre-Submission Checklist    - [ ] Commit message has the format required by CONTRIBUTING guide - [ ] Commits are split per component (core, individual modules, libs, utils, ...) - [ ] Each component has a single commit (if not, squash them into one commit) - [ ] 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 - [ ] 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 - [ ] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1792 -- Commit Summary -- * Modules: rtpengine-rtpengine_query exported to ksr routes -- File Changes -- M src/modules/rtpengine/rtpengine.c (20) -- Patch Links -- https://github.com/kamailio/kamailio/pull/1792.patch https://github.com/kamailio/kamailio/pull/1792.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/1792

2 3

git:master:026475aa: rtpengine: exported rtpengine_query() variants to KEMI framework
by Daniel-Constantin Mierla 09 Jan '19

09 Jan '19

Module: kamailio Branch: master Commit: 026475aa9d43307442d0e57acf8e11544d00dbed URL: https://github.com/kamailio/kamailio/commit/026475aa9d43307442d0e57acf8e115… Author: Surendra Tiwari <surendratiwari3(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2019-01-09T08:57:12+01:00 rtpengine: exported rtpengine_query() variants to KEMI framework - GH #1792 --- Modified: src/modules/rtpengine/rtpengine.c --- Diff: https://github.com/kamailio/kamailio/commit/026475aa9d43307442d0e57acf8e115… Patch: https://github.com/kamailio/kamailio/commit/026475aa9d43307442d0e57acf8e115… --- diff --git a/src/modules/rtpengine/rtpengine.c b/src/modules/rtpengine/rtpengine.c index 484ce05f35..6269a8c787 100644 --- a/src/modules/rtpengine/rtpengine.c +++ b/src/modules/rtpengine/rtpengine.c @@ -3843,6 +3843,16 @@ static int ki_rtpengine_delete(sip_msg_t *msg, str *flags) return rtpengine_rtpp_set_wrap(msg, rtpengine_delete_wrap, flags->s, 1); } +static int ki_rtpengine_query0(sip_msg_t *msg) +{ + return rtpengine_rtpp_set_wrap(msg, rtpengine_query_wrap, NULL, 1); +} + +static int ki_rtpengine_query(sip_msg_t *msg, str *flags) +{ + return rtpengine_rtpp_set_wrap(msg, rtpengine_query_wrap, flags->s, 1); +} + static int ki_start_recording(sip_msg_t *msg) { return rtpengine_rtpp_set_wrap(msg, rtpengine_start_recording_wrap, NULL, 1); @@ -3977,6 +3987,16 @@ static sr_kemi_t sr_kemi_rtpengine_exports[] = { { SR_KEMIP_INT, SR_KEMIP_INT, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } }, + { str_init("rtpengine"), str_init("rtpengine_query0"), + SR_KEMIP_INT, ki_rtpengine_query0, + { SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, + SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } + }, + { str_init("rtpengine"), str_init("rtpengine_query"), + SR_KEMIP_INT, ki_rtpengine_query, + { SR_KEMIP_STR, SR_KEMIP_NONE, SR_KEMIP_NONE, + SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } + }, { {0, 0}, {0, 0}, 0, NULL, { 0, 0, 0, 0, 0, 0 } } };

1 0

[kamailio/kamailio] Error while initializing module carrierroute (#1798)
by Gkpanda321 09 Jan '19

09 Jan '19

Ubuntu 18.04 & Kamailio V 5.1 I am unable to start kamailio services due the below error. ● kamailio.service - Kamailio (OpenSER) - the Open Source SIP Server Loaded: loaded (/lib/systemd/system/kamailio.service; enabled; vendor preset: enabled) Active: failed (Result: exit-code) since Tue 2019-01-08 22:36:49 UTC; 12s ago Process: 12933 ExecStart=/usr/sbin/kamailio -P /var/run/kamailio/kamailio.pid -f $CFGFILE -m $SHM_MEMORY -M $PKG_MEMORY -u $USER -g $GROUP (code=exited, status=255) Main PID: 12877 (code=exited, status=0/SUCCESS) Jan 08 22:36:49 kamatl03 kamailio[12933]: udp: kamatl03:5060 Jan 08 22:36:49 kamatl03 kamailio[12933]: udp: localhost:5060 Jan 08 22:36:49 kamatl03 kamailio[12933]: udp: localhost.localdomain:5060 Jan 08 22:36:49 kamatl03 /usr/sbin/kamailio[12955]: INFO: carrierroute [carrierroute.c:213]: mod_init(): use file as configuration source Jan 08 22:36:49 kamatl03 /usr/sbin/kamailio[12955]: ERROR: carrierroute [carrierroute.c:215]: mod_init(): can't stat config file Jan 08 22:36:49 kamatl03 /usr/sbin/kamailio[12955]: ERROR: <core> [core/sr_module.c:986]: init_mod(): Error while initializing module carrierroute (/usr/lib/x86_64-linu Jan 08 22:36:49 kamatl03 /usr/sbin/kamailio[12955]: INFO: <core> [core/sctp_core.c:53]: sctp_core_destroy(): SCTP API not initialized Jan 08 22:36:49 kamatl03 systemd[1]: kamailio.service: Control process exited, code=exited status=255 Jan 08 22:36:49 kamatl03 systemd[1]: kamailio.service: Failed with result 'exit-code'. Jan 08 22:36:49 kamatl03 systemd[1]: Failed to start Kamailio (OpenSER) - the Open Source SIP Server. ***********************Here is the Module Selection in configuraiton file*************** ####### Modules Section ######## # set paths to location of modules (to sources or installation folders) #!ifdef WITH_SRCPATH mpath="/usr/local/lib64/kamailio/modules" #!else mpath="/usr/lib/x86_64-linux-gnu/kamailio/modules/" #!endif -- 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/1798

3 3

No Media in SIP Incoming calls
by Prashant Gupta 09 Jan '19

09 Jan '19

Hi, I have the following architecture - SIP provider <-> Kamailio <-> Asterisk servers Currently I have everything setup and incoming calls from Sip are routed to my asterisk server. The issue is however that when I answer the call, there is no media in the call. I have tried connecting with a normal local extension(not SIP,eg 1001) and there is a normal flow of media. When i try to sniff my connection via Wireshark on the asterisk server, there is an outflow of RTP packets but the same RTP traffic does not appear on the Wireshark of my Kamailio server connection. I am not sure if this is an RTP engine issue and how to resolve this. I have - modparam("rtpproxy", "rtpproxy_sock", "udp:127.0.0.1:45038") this in my kamailio cfg but I don;t know which port to use here. Any suggestions?

2 1

[kamailio/kamailio] Cannot use AVPs with t_check_status (#1795)
by Hugh Waite 09 Jan '19

09 Jan '19

### Description If an AVP containing a regex is passed to the t_check_status function, it does not match. Example: ``` failure_route[DEFAULT] { $avp(FAILURE_CODES) = "5[0-9][0-9]"; xlog("L_WARN", "Status code: $T_reply_code\n"); if ($avp(FAILURE_CODES) != $null) { xlog("L_WARN", "resp codes regex: $avp(FAILURE_CODES)\n"); if (t_check_status("$avp(FAILURE_CODES)")) { xlog("L_WARN", "t_check_status matched\n"); } } } ``` #### Reproduction Using the code above in a failure_route should match a 500 response, but it does not. ### Possible Solutions The fixup_t_check_status in tm.c:663 attempts to match FPARAM_AVP/FPARAM_SELECT/FPARAM_REGEX parameter types. If another check for the FPARAM_PVS parameter type then the issue appears fixed -- 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/1795

3 2

Errored: lazedo/kamailio#41 (5.1 - 74b8fbf)
by Travis CI 09 Jan '19

09 Jan '19

Build Update for lazedo/kamailio ------------------------------------- Build: #41 Status: Errored Duration: 50 mins and 0 secs Commit: 74b8fbf (5.1) Author: Daniel-Constantin Mierla Message: topos_redis: check the branch value to avoid using uninitialized fields - GH #1784 (cherry picked from commit cfc89db53c087382c29d809dc0bc56f7351af2d1) (cherry picked from commit 7c551812960dd1cfc2901a7a46bbf3010df47cd7) View the changeset: https://github.com/lazedo/kamailio/compare/9d301849a145...74b8fbf0fc82 View the full build log and details: https://travis-ci.org/lazedo/kamailio/builds/477120826?utm_medium=notificat… -- You can unsubscribe from build emails from the lazedo/kamailio repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=11772075&u…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

git:master:c740e2f3: tm: Add fixup for PVS in fixup_t_check_status
by Hugh Waite 08 Jan '19

08 Jan '19

Module: kamailio Branch: master Commit: c740e2f3da746c2f8c6f488fa926dc5d348ea43a URL: https://github.com/kamailio/kamailio/commit/c740e2f3da746c2f8c6f488fa926dc5… Author: Hugh Waite <hugh.waite(a)newvoicemedia.com> Committer: Hugh Waite <hugh.waite(a)newvoicemedia.com> Date: 2019-01-04T16:02:09Z tm: Add fixup for PVS in fixup_t_check_status - Allows PVs including $avp(...) to be passed to t_check_status --- Modified: src/modules/tm/tm.c --- Diff: https://github.com/kamailio/kamailio/commit/c740e2f3da746c2f8c6f488fa926dc5… Patch: https://github.com/kamailio/kamailio/commit/c740e2f3da746c2f8c6f488fa926dc5… --- diff --git a/src/modules/tm/tm.c b/src/modules/tm/tm.c index 49db7d4507..c8afee9c4c 100644 --- a/src/modules/tm/tm.c +++ b/src/modules/tm/tm.c @@ -664,6 +664,9 @@ static int fixup_t_check_status(void** param, int param_no) { int ret; + ret = fix_param(FPARAM_PVS, param); + if (ret <= 0) return ret; + ret = fix_param(FPARAM_AVP, param); if (ret <= 0) return ret;

1 0

← Newer
1
...
23
24
25
26
27
28
29
30
31
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev January 2019