sr-dev September 2024

sr-dev@lists.kamailio.org

12 participants
138 discussions

git:master:81f96617: dialog/doc: Remove dlg_flag docs and suggest dlg_manage() instead.

by Henning Westerholt

Module: kamailio Branch: master Commit: 81f966174275b715608f50aae606d40a791dd67c URL: https://github.com/kamailio/kamailio/commit/81f966174275b715608f50aae606d40… Author: Xenofon Karamanos <22965395+xkaraman(a)users.noreply.github.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2024-09-16T13:39:06+02:00 dialog/doc: Remove dlg_flag docs and suggest dlg_manage() instead. --- Modified: src/modules/dialog/doc/dialog_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/81f966174275b715608f50aae606d40… Patch: https://github.com/kamailio/kamailio/commit/81f966174275b715608f50aae606d40… --- diff --git a/src/modules/dialog/doc/dialog_admin.xml b/src/modules/dialog/doc/dialog_admin.xml index 07116e70a63..d1547b367a6 100644 --- a/src/modules/dialog/doc/dialog_admin.xml +++ b/src/modules/dialog/doc/dialog_admin.xml @@ -46,9 +46,8 @@ <title>How it works</title> <para> 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. + the function <quote>dlg_manage()</quote> before creating the + corresponding transaction. </para> <para> The dialog is automatically destroyed when a <quote>BYE</quote> is @@ -253,28 +252,17 @@ modparam("dialog", "rr_param", "xyz") <section id="dialog.p.flag"> <title><varname>dlg_flag</varname> (integer)</title> <para> - Flag to be used for marking if a dialog should be constructed for the - current request (this makes sense only for initial requests). - </para> - <para> - Note: it is not needed to set this parameter and its corresponding flag + Note: This flag has beed deprecated and not used anyymore. + 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. + is used. + In other words, use <quote>dlg_manage()</quote>. </para> <para> <emphasis> Default value is <quote>none</quote>. </emphasis> </para> - <example> - <title>Set <varname>dlg_flag</varname> parameter</title> - <programlisting format="linespecific"> -... -modparam("dialog", "dlg_flag", 4) -... -</programlisting> - </example> </section> <section id="dialog.p.timeout_avp">

2 days, 11 hours

git:master:e60a0646: dialog: Drop support for dlg_flag parameter

by Henning Westerholt

Module: kamailio Branch: master Commit: e60a06469503c566d40437e69cf3374aad361251 URL: https://github.com/kamailio/kamailio/commit/e60a06469503c566d40437e69cf3374… Author: Xenofon Karamanos <22965395+xkaraman(a)users.noreply.github.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2024-09-16T13:39:06+02:00 dialog: Drop support for dlg_flag parameter --- Modified: src/modules/dialog/dialog.c Modified: src/modules/dialog/dlg_handlers.c Modified: src/modules/dialog/dlg_handlers.h --- Diff: https://github.com/kamailio/kamailio/commit/e60a06469503c566d40437e69cf3374… Patch: https://github.com/kamailio/kamailio/commit/e60a06469503c566d40437e69cf3374…

2 days, 11 hours

[kamailio/kamailio] cdp: export all functions in KEMI module (PR #3949)

by bazrafshan.m.h

- Added `cdp_has_app` function: - Accepts a single parameter: - `application`: The Application ID. - Added `cdp_has_app2` function: - Accepts two parameters: - `vendorid`: The Vendor ID associated with the application. - `application`: The Application ID. - Added cdp_check_peer function: - Accepts one parameter: - `fqdn`: the Fully qualified domain name of the peer, that should be checked. The parameter may contain pseudovariables. NOTE: All parameters are passed as standard strings.   #### 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 - [ ] Small bug fix (non-breaking change which fixes an issue) - [x] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist:  - [x] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  - The cdp functions are exported to KEMI You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3949 -- Commit Summary -- * cdp: export all functions in KEMI module -- File Changes -- M src/modules/cdp/cdp_mod.c (49) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3949.patch https://github.com/kamailio/kamailio/pull/3949.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3949 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3949(a)github.com>

2 days, 11 hours

git:master:11f20730: cdp: export all functions in KEMI module

by Henning Westerholt

Module: kamailio Branch: master Commit: 11f207306a1da400f4252a1fafd456cda015fb3b URL: https://github.com/kamailio/kamailio/commit/11f207306a1da400f4252a1fafd456c… Author: Moheb <bazrafshan.m.h(a)gmail.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2024-09-16T13:35:01+02:00 cdp: export all functions in KEMI module - Added `cdp_has_app` function: - Accepts a single parameter: - `application`: The Application ID. - Added `cdp_has_app2` function: - Accepts two parameters: - `vendorid`: The Vendor ID associated with the application. - `application`: The Application ID. - Added cdp_check_peer function: - Accepts one parameter: - `fqdn`: the Fully qualified domain name of the peer, that should be checked. The parameter may contain pseudovariables. NOTE: All parameters are passed as standard strings. --- Modified: src/modules/cdp/cdp_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/11f207306a1da400f4252a1fafd456c… Patch: https://github.com/kamailio/kamailio/commit/11f207306a1da400f4252a1fafd456c… --- diff --git a/src/modules/cdp/cdp_mod.c b/src/modules/cdp/cdp_mod.c index 4685d2af511..b32e4ea7bf4 100644 --- a/src/modules/cdp/cdp_mod.c +++ b/src/modules/cdp/cdp_mod.c @@ -57,6 +57,7 @@ #include "cdp_functions.h" #include "cdp_tls.h" #include "../../core/mod_fix.h" +#include "../../core/kemi.h" MODULE_VERSION @@ -301,7 +302,10 @@ int w_cdp_check_peer(sip_msg_t *msg, char *peer, char *p2) } return -1; } - +int ki_cdp_check_peer(sip_msg_t *msg, str *peer) +{ + return w_cdp_check_peer(msg, peer->s, "NULL"); +} static int w_cdp_has_app(sip_msg_t *msg, char *appid, char *param) { unsigned int app_flags; @@ -324,6 +328,10 @@ static int w_cdp_has_app(sip_msg_t *msg, char *appid, char *param) return check_application(-1, a); } +static int ki_cdp_has_app(sip_msg_t *msg, str *appid) +{ + return w_cdp_has_app(msg, appid->s, "NULL"); +} static int w_cdp_has_app2(sip_msg_t *msg, char *vendor, char *appid) { unsigned int vendor_flags, app_flags; @@ -359,3 +367,42 @@ static int w_cdp_has_app2(sip_msg_t *msg, char *vendor, char *appid) } return check_application(v, a); } +static int ki_cdp_has_app2(sip_msg_t *msg, str *vendor, str *appid) +{ + return w_cdp_has_app2(msg, vendor->s, appid->s); +} + +/** + * + */ +/* clang-format off */ +static sr_kemi_t sr_kemi_cdp_exports[] = { + { str_init("cdp"), str_init("cdp_check_peer"), + SR_KEMIP_INT, ki_cdp_check_peer, + { SR_KEMIP_STR, SR_KEMIP_NONE, SR_KEMIP_NONE, + SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } + }, + { str_init("cdp"), str_init("cdp_has_app"), + SR_KEMIP_INT, ki_cdp_has_app, + { SR_KEMIP_STR, SR_KEMIP_NONE, SR_KEMIP_NONE, + SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } + }, + { str_init("cdp"), str_init("cdp_has_app2"), + SR_KEMIP_INT, ki_cdp_has_app2, + { SR_KEMIP_STR, SR_KEMIP_STR, SR_KEMIP_NONE, + SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } + }, + + { {0, 0}, {0, 0}, 0, NULL, { 0, 0, 0, 0, 0, 0 } } +}; +/* clang-format on */ + +/** + * + */ + +int mod_register(char *path, int *dlflags, void *p1, void *p2) +{ + sr_kemi_modules_add(sr_kemi_cdp_exports); + return 0; +}

2 days, 11 hours

git:master:15d81a01: core: use macro to check send_port_str value

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 15d81a0151aa46a20f95fb8c6dc2665d03e741aa URL: https://github.com/kamailio/kamailio/commit/15d81a0151aa46a20f95fb8c6dc2665… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-09-16T12:18:54+02:00 core: use macro to check send_port_str value --- Modified: src/core/msg_translator.c --- Diff: https://github.com/kamailio/kamailio/commit/15d81a0151aa46a20f95fb8c6dc2665… Patch: https://github.com/kamailio/kamailio/commit/15d81a0151aa46a20f95fb8c6dc2665… --- diff --git a/src/core/msg_translator.c b/src/core/msg_translator.c index bc68a920ef6..4f88150e11b 100644 --- a/src/core/msg_translator.c +++ b/src/core/msg_translator.c @@ -728,7 +728,7 @@ static inline int lumps_len( }; \ break; \ case SUBST_SND_PORT: \ - if(send_port_str != NULL) { \ + if(STR_WITHVAL(send_port_str)) { \ new_len += send_port_str->len; \ } else { \ LM_CRIT("null send sock port\n"); \ @@ -772,7 +772,7 @@ static inline int lumps_len( send_address_str->len) \ != NULL)) \ new_len += 2; \ - if(send_port_str != NULL) { \ + if(STR_WITHVAL(send_port_str)) { \ /* add :port_no */ \ new_len += 1 + send_port_str->len; \ } \ @@ -1175,7 +1175,7 @@ void process_lumps(struct sip_msg *msg, struct lump *lumps, char *new_buf, }; \ break; \ case SUBST_SND_PORT: \ - if(send_port_str != NULL) { \ + if(STR_WITHVAL(send_port_str)) { \ memcpy(new_buf + offset, send_port_str->s, \ send_port_str->len); \ offset += send_port_str->len; \ @@ -1205,7 +1205,7 @@ void process_lumps(struct sip_msg *msg, struct lump *lumps, char *new_buf, offset++; \ } \ /* :port */ \ - if(send_port_str != NULL) { \ + if(STR_WITHVAL(send_port_str)) { \ new_buf[offset] = ':'; \ offset++; \ memcpy(new_buf + offset, send_port_str->s, \

2 days, 12 hours

git:master:c09a2936: core: use advertised port for recv socket only when set

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: c09a29369f03f6c5e621a94b5b853047490568aa URL: https://github.com/kamailio/kamailio/commit/c09a29369f03f6c5e621a94b5b85304… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-09-16T10:42:19+02:00 core: use advertised port for recv socket only when set --- Modified: src/core/msg_translator.c --- Diff: https://github.com/kamailio/kamailio/commit/c09a29369f03f6c5e621a94b5b85304… Patch: https://github.com/kamailio/kamailio/commit/c09a29369f03f6c5e621a94b5b85304… --- diff --git a/src/core/msg_translator.c b/src/core/msg_translator.c index 19713e1b1cc..bc68a920ef6 100644 --- a/src/core/msg_translator.c +++ b/src/core/msg_translator.c @@ -631,7 +631,7 @@ static inline int lumps_len( }; \ break; \ case SUBST_RCV_PORT: \ - if(msg->rcv.bind_address && STR_WITHVAL(recv_port_str)) { \ + if(STR_WITHVAL(recv_port_str)) { \ new_len += recv_port_str->len; \ } else { \ LM_CRIT(" rcv port - null bind_address\n"); \ @@ -864,9 +864,11 @@ static inline int lumps_len( recv_address_str = &(msg->rcv.bind_address->address_str); recv_af = msg->rcv.bind_address->address.af; } - if(msg->rcv.bind_address->useinfo.port_no > 0) { - recv_port_str = &(msg->rcv.bind_address->useinfo.port_no_str); - recv_port_no = msg->rcv.bind_address->useinfo.port_no; + if(msg->rcv.bind_address->useinfo.name.len > 0) { + if(msg->rcv.bind_address->useinfo.port_no > 0) { + recv_port_str = &(msg->rcv.bind_address->useinfo.port_no_str); + recv_port_no = msg->rcv.bind_address->useinfo.port_no; + } } else { recv_port_str = &(msg->rcv.bind_address->port_no_str); recv_port_no = msg->rcv.bind_address->port_no; @@ -1055,7 +1057,7 @@ void process_lumps(struct sip_msg *msg, struct lump *lumps, char *new_buf, }; \ break; \ case SUBST_RCV_PORT: \ - if(msg->rcv.bind_address && STR_WITHVAL(recv_port_str)) { \ + if(STR_WITHVAL(recv_port_str)) { \ memcpy(new_buf + offset, recv_port_str->s, \ recv_port_str->len); \ offset += recv_port_str->len; \ @@ -1396,9 +1398,11 @@ void process_lumps(struct sip_msg *msg, struct lump *lumps, char *new_buf, recv_address_str = &(msg->rcv.bind_address->address_str); recv_af = msg->rcv.bind_address->address.af; } - if(msg->rcv.bind_address->useinfo.port_no > 0) { - recv_port_str = &(msg->rcv.bind_address->useinfo.port_no_str); - recv_port_no = msg->rcv.bind_address->useinfo.port_no; + if(msg->rcv.bind_address->useinfo.name.len > 0) { + if(msg->rcv.bind_address->useinfo.port_no > 0) { + recv_port_str = &(msg->rcv.bind_address->useinfo.port_no_str); + recv_port_no = msg->rcv.bind_address->useinfo.port_no; + } } else { recv_port_str = &(msg->rcv.bind_address->port_no_str); recv_port_no = msg->rcv.bind_address->port_no;

2 days, 14 hours

git:master:0f423f97: core: use advertised port only when explicitely set

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 0f423f97607416edcff8d0f1612fad7af29b6064 URL: https://github.com/kamailio/kamailio/commit/0f423f97607416edcff8d0f1612fad7… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-09-16T09:17:09+02:00 core: use advertised port only when explicitely set - do not fallback to socket port --- Modified: src/core/msg_translator.c --- Diff: https://github.com/kamailio/kamailio/commit/0f423f97607416edcff8d0f1612fad7… Patch: https://github.com/kamailio/kamailio/commit/0f423f97607416edcff8d0f1612fad7…

2 days, 15 hours

[kamailio/kamailio] move app_mono to archive (PR #3964)

by Victor Seva

#### 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 - [ ] Small bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds new functionality) - [x] Breaking change (fix or feature that would change existing functionality) #### Checklist:  - [ ] PR should be backported to stable branches - [ ] Tested changes locally - [x] Related to issue #3963 #### Description steps to remove app_mono You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3964 -- Commit Summary -- * Makefile.groups: remove app_mono * app_mono: module relocated to archive -- File Changes -- M src/Makefile.groups (8) D src/modules/app_mono/Makefile (17) D src/modules/app_mono/README (223) D src/modules/app_mono/app_mono_api.c (907) D src/modules/app_mono/app_mono_api.h (63) D src/modules/app_mono/app_mono_mod.c (203) D src/modules/app_mono/doc/Makefile (4) D src/modules/app_mono/doc/app_mono.xml (41) D src/modules/app_mono/doc/app_mono_admin.xml (231) D src/modules/app_mono/lib/SR.cs (41) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3964.patch https://github.com/kamailio/kamailio/pull/3964.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3964 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3964(a)github.com>

2 days, 15 hours

[kamailio/kamailio] app_mono: maybe time to move it to archive repo? (Issue #3963)

by Victor Seva

### Description From https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1079882 ``` From: "Chris Hofstaedtler" <zeha(a)debian.org> Source: kamailio Priority: wishlist Control: block 1010445 by -1 Hi, kamailio currently (build-)depends on mono libraries. Please drop these relations, probably turning off mono support altogether. The mono stack in Debian is not maintained in any way, so we'd like to reduce packages relying on it. Chris ``` I'll remove app_mono from sid and >= trixie builds -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3963 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3963(a)github.com>

2 days, 15 hours

git:master:6cf255de: app_mono: module relocated to archive

by Victor Seva

Module: kamailio Branch: master Commit: 6cf255dee8620ffa1a5be1bb23a1b767015218fa URL: https://github.com/kamailio/kamailio/commit/6cf255dee8620ffa1a5be1bb23a1b76… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2024-09-16T09:13:02+02:00 app_mono: module relocated to archive fixes #3963 --- Removed: src/modules/app_mono/Makefile Removed: src/modules/app_mono/README Removed: src/modules/app_mono/app_mono_api.c Removed: src/modules/app_mono/app_mono_api.h Removed: src/modules/app_mono/app_mono_mod.c Removed: src/modules/app_mono/doc/Makefile Removed: src/modules/app_mono/doc/app_mono.xml Removed: src/modules/app_mono/doc/app_mono_admin.xml Removed: src/modules/app_mono/lib/SR.cs --- Diff: https://github.com/kamailio/kamailio/commit/6cf255dee8620ffa1a5be1bb23a1b76… Patch: https://github.com/kamailio/kamailio/commit/6cf255dee8620ffa1a5be1bb23a1b76…

2 days, 16 hours

← Newer
1
2
3
4
5
6
7
8
9
...
14
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev September 2024