sr-dev October 2016

sr-dev@lists.kamailio.org

28 participants
234 discussions

[kamailio/kamailio] core: added mcast to select interface for multicast (#815)

by Timmo Verlaan

You can now define a mcast parameter before a listen parameter that contains a multicast address. After each listen parameter mcast gets reset. mcast must contain the name of an interface, eg `eth1`. Issue GH#813 You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/815 -- Commit Summary -- * core: added mcast to select interface for multicast -- File Changes -- M cfg.lex (3) M cfg.y (16) M globals.h (1) M ip_addr.h (3) M main.c (1) M socket_info.c (8) M udp_server.c (24) -- Patch Links -- https://github.com/kamailio/kamailio/pull/815.patch https://github.com/kamailio/kamailio/pull/815.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/815

7 years, 6 months

[kamailio/kamailio] Interface selection for multicast listen socket (#813)

by Timmo Verlaan

If you configure kamailio to listen on a multicast address (eg. `224.0.1.75`) it _auto_ selects the interface that joins the multicast group. The selection happens on what the kernel thinks the multicast IP address is active on. This is decided by the kernel route table since you cannot configure a multicast address on your interface. In the default scenario this would mean the default route interface will be selected to join the multicast group. In case you want to select another interface this is possible by altering the route for that particular multicast address to the interface you desire. This is a workable solution but it has the following downsides: - need to add route manually to specify interface - multicast for 1 specific address only possible on 1 interface I would like to propose a solution by adding a config parameter to kamailio (eg. `mcast_interface`) which could then be defined before each `listen` directive that is a multicast address. This would allow for selecting an interface per listen socket in the same way as you can specify the amount of workers for an interface. In this scenario there is no need for manually adding a route to specify the interface. Please let me know if this is a feature that is of interest to kamailio. If so I'd be willing to work on this myself, but somebody else can also do it. If something isn't clear or needs more explanation don't hesitate to ask. -- 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/813

7 years, 6 months

git:master:8b39cb6a: core: added mcast to select interface for multicast

by Timmo Verlaan

Module: kamailio Branch: master Commit: 8b39cb6a3aa37d26da1ccf8f55f25aaba8fc4613 URL: https://github.com/kamailio/kamailio/commit/8b39cb6a3aa37d26da1ccf8f55f25aa… Author: Timmo Verlaan <tverlaan(a)gmail.com> Committer: Timmo Verlaan <tverlaan(a)gmail.com> Date: 2016-10-07T12:13:24+02:00 core: added mcast to select interface for multicast - You can now define a mcast parameter before a listen parameter that contains a multicast address. After each listen parameter mcast gets reset. mcast must contain the name of an interface, eg `eth1`. Issue GH#813 --- Modified: cfg.lex Modified: cfg.y Modified: globals.h Modified: ip_addr.h Modified: main.c Modified: socket_info.c Modified: udp_server.c --- Diff: https://github.com/kamailio/kamailio/commit/8b39cb6a3aa37d26da1ccf8f55f25aa… Patch: https://github.com/kamailio/kamailio/commit/8b39cb6a3aa37d26da1ccf8f55f25aa…

7 years, 6 months

git:master:eb19b914: Merge pull request #815 from tverlaan/add_mcast_interface_directive

by GitHub

Module: kamailio Branch: master Commit: eb19b9149ee23ca7381b28dcab3108ac960c789a URL: https://github.com/kamailio/kamailio/commit/eb19b9149ee23ca7381b28dcab3108a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2016-10-10T09:20:43+02:00 Merge pull request #815 from tverlaan/add_mcast_interface_directive core: added mcast to select interface for multicast --- Modified: cfg.lex Modified: cfg.y Modified: globals.h Modified: ip_addr.h Modified: main.c Modified: socket_info.c Modified: udp_server.c --- Diff: https://github.com/kamailio/kamailio/commit/eb19b9149ee23ca7381b28dcab3108a… Patch: https://github.com/kamailio/kamailio/commit/eb19b9149ee23ca7381b28dcab3108a…

7 years, 6 months

git:master:fa724d2c: Merge pull request #817 from snen/presentity_dialog_id_null_fix

by GitHub

Module: kamailio Branch: master Commit: fa724d2c77295f8f8d78ed3253c30b21df3935e5 URL: https://github.com/kamailio/kamailio/commit/fa724d2c77295f8f8d78ed3253c30b2… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2016-10-10T09:13:35+02:00 Merge pull request #817 from snen/presentity_dialog_id_null_fix presence: fix crush if dialog id is NULL --- Modified: modules/presence/presentity.c --- Diff: https://github.com/kamailio/kamailio/commit/fa724d2c77295f8f8d78ed3253c30b2… Patch: https://github.com/kamailio/kamailio/commit/fa724d2c77295f8f8d78ed3253c30b2… --- diff --git a/modules/presence/presentity.c b/modules/presence/presentity.c index 00fdbfd..cb819cf 100644 --- a/modules/presence/presentity.c +++ b/modules/presence/presentity.c @@ -691,7 +691,7 @@ int update_presentity(struct sip_msg* msg, presentity_t* presentity, str* body, } check_if_dialog(*body, &is_dialog, &dialog_id); - if (is_dialog == 1) { + if ( dialog_id ) { if (delete_presentity_if_dialog_id_exists(presentity, dialog_id) < 0) { goto error; }

7 years, 6 months

git:master:d68d0126: presence: fix crush if dialog id is NULL

by Dmitri Savolainen

Module: kamailio Branch: master Commit: d68d01262e59593e7976afea1b1c458815448e7a URL: https://github.com/kamailio/kamailio/commit/d68d01262e59593e7976afea1b1c458… Author: Dmitri Savolainen <savolainen(a)erinaco.ru> Committer: Dmitri Savolainen <savolainen(a)erinaco.ru> Date: 2016-10-07T19:32:53+03:00 presence: fix crush if dialog id is NULL - check dialog_id is not NULL before call delete_presentity_if_dialog_id_exists --- Modified: modules/presence/presentity.c --- Diff: https://github.com/kamailio/kamailio/commit/d68d01262e59593e7976afea1b1c458… Patch: https://github.com/kamailio/kamailio/commit/d68d01262e59593e7976afea1b1c458… --- diff --git a/modules/presence/presentity.c b/modules/presence/presentity.c index 00fdbfd..cb819cf 100644 --- a/modules/presence/presentity.c +++ b/modules/presence/presentity.c @@ -691,7 +691,7 @@ int update_presentity(struct sip_msg* msg, presentity_t* presentity, str* body, } check_if_dialog(*body, &is_dialog, &dialog_id); - if (is_dialog == 1) { + if ( dialog_id ) { if (delete_presentity_if_dialog_id_exists(presentity, dialog_id) < 0) { goto error; }

7 years, 6 months

[kamailio/kamailio] presence: fix crush if dialog id is NULL (#817)

by Dmitri Savolainen

- check dialog_id is not NULL before call delete_presentity_if_dialog_id_exists sometimes after check_if_dialog(str body, int *is_dialog, char **dialog_id) it is possible is_dialog==1 dialog_id==NULL #0 __strcmp_sse42 () at ../sysdeps/x86_64/multiarch/strcmp.S:260 #1 0x00007feb2e446603 in delete_presentity_if_dialog_id_exists (presentity=0x7feb545c8ca8, dialog_id=0x0) at presentity.c:419 You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/817 -- Commit Summary -- * presence: fix crush if dialog id is NULL -- File Changes -- M modules/presence/presentity.c (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/817.patch https://github.com/kamailio/kamailio/pull/817.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/817

7 years, 6 months

[kamailio/kamailio] rabbitmq: Add new module (#786)

by Stefan Mititelu

Add new rabbitmq module. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/786 -- Commit Summary -- * rabbitmq: Add new module -- File Changes -- A modules/rabbitmq/Makefile (14) A modules/rabbitmq/README (244) A modules/rabbitmq/doc/Makefile (4) A modules/rabbitmq/doc/rabbitmq.xml (39) A modules/rabbitmq/doc/rabbitmq_admin.xml (338) A modules/rabbitmq/rabbitmq.c (579) A modules/rabbitmq/rabbitmq.h (15) A modules/rabbitmq/utils.c (192) A modules/rabbitmq/utils.h (50) -- Patch Links -- https://github.com/kamailio/kamailio/pull/786.patch https://github.com/kamailio/kamailio/pull/786.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/786

7 years, 6 months

[kamailio/kamailio] Problem with htable (#811)

by AlainDubois78

When I'm using a htable in Kamailio, I can get the htable:expired:<htable> route to be executed if I include in my htable name a _ character (meaning my_htable does not work whereas myHable works). Is it a known restriction or limitation the htable module or of Kamailio or is it a bug? I'm using Kamailio on Ubuntu. -- 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/811

7 years, 6 months

git:4.4:d546d5ea: core: allow '_' in event route name

by Daniel-Constantin Mierla

Module: kamailio Branch: 4.4 Commit: d546d5ea8a1ce442dcaad8592df55459666c7da5 URL: https://github.com/kamailio/kamailio/commit/d546d5ea8a1ce442dcaad8592df5545… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2016-10-07T10:09:53+02:00 core: allow '_' in event route name - reported by GH #811 (cherry picked from commit 5806703c1ff72243457ff55758f41c3f557d26fd) --- Modified: cfg.lex --- Diff: https://github.com/kamailio/kamailio/commit/d546d5ea8a1ce442dcaad8592df5545… Patch: https://github.com/kamailio/kamailio/commit/d546d5ea8a1ce442dcaad8592df5545… --- diff --git a/cfg.lex b/cfg.lex index 357ff93..e31b5a6 100644 --- a/cfg.lex +++ b/cfg.lex @@ -496,7 +496,7 @@ COLON ":" STAR \* DOT \. CR \n -EVENT_RT_NAME [a-zA-Z][0-9a-zA-Z-]*(":"[a-zA-Z][0-9a-zA-Z-]*)+ +EVENT_RT_NAME [a-zA-Z][0-9a-zA-Z-]*(":"[a-zA-Z][0-9a-zA-Z_-]*)+ COM_LINE "#"|"//"

7 years, 6 months

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev October 2016