sr-dev April 2022

sr-dev@lists.kamailio.org

17 participants
285 discussions

[kamailio/kamailio] Typos docs (PR #3078)
by Bastian Triller 11 Apr '22

11 Apr '22

#### 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) - [ ] 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  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3078 -- Commit Summary -- * core: Fix typos, reference to SER * auth: Fix missing dependency in docs -- File Changes -- M src/core/str.h (2) M src/core/ut.c (14) M src/core/ut.h (8) M src/modules/auth/doc/auth.xml (9) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3078.patch https://github.com/kamailio/kamailio/pull/3078.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3078 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3078(a)github.com>

2 3

git:master:841681e6: pua_dialoginfo: Fix typo
by Henning Westerholt 11 Apr '22

11 Apr '22

Module: kamailio Branch: master Commit: 841681e68a8fd913bc6944eed4f4d00c05f92359 URL: https://github.com/kamailio/kamailio/commit/841681e68a8fd913bc6944eed4f4d00… Author: Bastian Triller <bastian.triller(a)gmail.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2022-04-11T15:52:36+02:00 pua_dialoginfo: Fix typo --- Modified: src/modules/pua_dialoginfo/pua_dialoginfo.c --- Diff: https://github.com/kamailio/kamailio/commit/841681e68a8fd913bc6944eed4f4d00… Patch: https://github.com/kamailio/kamailio/commit/841681e68a8fd913bc6944eed4f4d00… --- diff --git a/src/modules/pua_dialoginfo/pua_dialoginfo.c b/src/modules/pua_dialoginfo/pua_dialoginfo.c index 0385ed90dd..7b27e666c5 100644 --- a/src/modules/pua_dialoginfo/pua_dialoginfo.c +++ b/src/modules/pua_dialoginfo/pua_dialoginfo.c @@ -643,7 +643,7 @@ struct dlginfo_cell* get_dialog_data(struct dlg_cell *dlg, int type, int disable } #ifdef PUA_DIALOGINFO_DEBUG - /* dialog callback testing (registered last to be executed frist) */ + /* dialog callback testing (registered last to be executed first) */ if (dlg_api.register_dlgcb(dlg, DLGCB_FAILED| DLGCB_CONFIRMED_NA | DLGCB_CONFIRMED | DLGCB_REQ_WITHIN | DLGCB_TERMINATED | DLGCB_EXPIRED

1 0

git:master:dd15436e: auth: Fix missing dependency in docs
by Henning Westerholt 11 Apr '22

11 Apr '22

Module: kamailio Branch: master Commit: dd15436e8697f70df6ef9d188da36f70e7116a74 URL: https://github.com/kamailio/kamailio/commit/dd15436e8697f70df6ef9d188da36f7… Author: Bastian Triller <bastian.triller(a)gmail.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2022-04-11T15:52:36+02:00 auth: Fix missing dependency in docs --- Modified: src/modules/auth/doc/auth.xml --- Diff: https://github.com/kamailio/kamailio/commit/dd15436e8697f70df6ef9d188da36f7… Patch: https://github.com/kamailio/kamailio/commit/dd15436e8697f70df6ef9d188da36f7… --- diff --git a/src/modules/auth/doc/auth.xml b/src/modules/auth/doc/auth.xml index 6f20e8c71d..137a5a5f56 100644 --- a/src/modules/auth/doc/auth.xml +++ b/src/modules/auth/doc/auth.xml @@ -66,7 +66,14 @@ <section id="auth.dep"> <title>Dependencies</title> <para> - The module does not depend on any other module. + The following modules must be loaded before this module: + <itemizedlist> + <listitem> + <para> + <emphasis>sl</emphasis> - stateless reply. + </para> + </listitem> + </itemizedlist> </para> </section>

1 0

git:master:8bab7115: core: Fix typos, reference to SER
by Henning Westerholt 11 Apr '22

11 Apr '22

Module: kamailio Branch: master Commit: 8bab711590ac8b4fd73efabbc7c22be3231e6214 URL: https://github.com/kamailio/kamailio/commit/8bab711590ac8b4fd73efabbc7c22be… Author: Bastian Triller <bastian.triller(a)gmail.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2022-04-11T15:52:36+02:00 core: Fix typos, reference to SER --- Modified: src/core/str.h Modified: src/core/ut.c Modified: src/core/ut.h --- Diff: https://github.com/kamailio/kamailio/commit/8bab711590ac8b4fd73efabbc7c22be… Patch: https://github.com/kamailio/kamailio/commit/8bab711590ac8b4fd73efabbc7c22be… --- diff --git a/src/core/str.h b/src/core/str.h index d7977652d9..063a9e9398 100644 --- a/src/core/str.h +++ b/src/core/str.h @@ -143,7 +143,7 @@ typedef struct _str str; } while(0) /** - * str with value: not null and not emptu + * str with value: not null and not empty */ #define STR_WITHVAL(_sp) ((_sp) && ((_sp)->s) && ((_sp)->len)) diff --git a/src/core/ut.c b/src/core/ut.c index 8dde51aaff..a581f98e9f 100644 --- a/src/core/ut.c +++ b/src/core/ut.c @@ -97,7 +97,7 @@ int group2gid(int* gid, char* group) /* - * Replacement of timegm (does not exists on all platforms + * Replacement of timegm (does not exists on all platforms) * Taken from * http://lists.samba.org/archive/samba-technical/2002-November/025737.html */ @@ -232,7 +232,7 @@ unsigned int get_sys_version(int* major, int* minor, int* minor2) /** transform a relative pathname into an absolute one. * @param base - base file, used to extract the absolute path prefix. - * Might be NULL, in which case the path of the ser.cfg is + * Might be NULL, in which case the path of the kamailio.cfg is * used. * @param file - file path to be transformed. If it's already absolute * (starts with '/') is left alone. If not the result will @@ -301,7 +301,7 @@ char* get_abs_pathname(str* base, str* file) /** - * @brief search for occurence of needle in text + * @brief search for occurrence of needle in text * @return pointer to start of needle in text or NULL if the needle * is not found */ @@ -323,7 +323,7 @@ char *str_search(str *text, str *needle) } /** - * @brief search for occurence of needlez starting from vstart and before vend + * @brief search for occurrence of needlez starting from vstart and before vend * @return pointer to start of needle in text or NULL if the needle * is not found */ @@ -346,7 +346,7 @@ char *stre_search_strz(char *vstart, char *vend, char *needlez) } /** - * @brief case insensitive search for occurence of needle in text + * @brief case insensitive search for occurrence of needle in text * @return pointer to start of needle in text or NULL if the needle * is not found */ @@ -393,7 +393,7 @@ char *str_casesearch_strz(str *text, char *needlez) } /** - * @brief search for last occurence of needle in text (reverse search) + * @brief search for last occurrence of needle in text (reverse search) * @return pointer to start of needle in text or NULL if the needle * is not found */ @@ -415,7 +415,7 @@ char *str_rsearch(str *text, str *needle) } /** - * @brief case insensitive search for last occurence of needle in text (reverse search) + * @brief case insensitive search for last occurrence of needle in text (reverse search) * @return pointer to start of needle in text or NULL if the needle * is not found */ diff --git a/src/core/ut.h b/src/core/ut.h index 44bd0e75c7..7792a865ef 100644 --- a/src/core/ut.h +++ b/src/core/ut.h @@ -729,7 +729,7 @@ static inline int str2int(str* _s, unsigned int* _r) /* - * Convert an str to signed long + * Convert a str to signed long */ static inline int str2slong(str* _s, long* _r) { @@ -738,7 +738,7 @@ static inline int str2slong(str* _s, long* _r) /* - * Convert an str to signed integer + * Convert a str to signed integer */ static inline int str2sint(str* _s, int* _r) { @@ -747,7 +747,7 @@ static inline int str2sint(str* _s, int* _r) /* - * Convert an strz to integer + * Convert a strz to integer */ static inline int strz2int(char* _s, unsigned int* _r) { @@ -770,7 +770,7 @@ static inline int strz2int(char* _s, unsigned int* _r) } /* - * Convert an strz to signed integer + * Convert a strz to signed integer */ static inline int strz2sint(char* _s, int* _r) {

1 0

git:master:abd41396: modules: readme files regenerated - pv_headers ... [skip ci]
by Kamailio Dev 11 Apr '22

11 Apr '22

Module: kamailio Branch: master Commit: abd41396a937155a978ff024f309b082459a057e URL: https://github.com/kamailio/kamailio/commit/abd41396a937155a978ff024f309b08… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2022-04-11T15:16:16+02:00 modules: readme files regenerated - pv_headers ... [skip ci] --- Modified: src/modules/pv_headers/README --- Diff: https://github.com/kamailio/kamailio/commit/abd41396a937155a978ff024f309b08… Patch: https://github.com/kamailio/kamailio/commit/abd41396a937155a978ff024f309b08… --- diff --git a/src/modules/pv_headers/README b/src/modules/pv_headers/README index 828c7d1a13..e714d3f666 100644 --- a/src/modules/pv_headers/README +++ b/src/modules/pv_headers/README @@ -12,6 +12,11 @@ Victor Seva Sipwise GmbH <vseva(a)sipwise.com> +Fabricio Santolin da Silva + + Alcatel-Lucent Enterprise + <fabricio.santolin-da-silva(a)al-enterprise.com> + Copyright © 2018 Sipwise GmbH __________________________________________________________________ @@ -44,6 +49,8 @@ Victor Seva 4.5. pvh_append_header(hname, hvalue) 4.6. pvh_modify_header(hname, hvalue, [idx]) 4.7. pvh_remove_header(hname, [idx]) + 4.8. pvh_value_exists(hname, hparameter) + 4.9. pvh_remove_header_param(hname, hparameter) 5. Exported Variables @@ -86,6 +93,8 @@ Chapter 1. Admin Guide 4.5. pvh_append_header(hname, hvalue) 4.6. pvh_modify_header(hname, hvalue, [idx]) 4.7. pvh_remove_header(hname, [idx]) + 4.8. pvh_value_exists(hname, hparameter) + 4.9. pvh_remove_header_param(hname, hparameter) 5. Exported Variables @@ -240,8 +249,8 @@ modparam("pv_headers", "auto_msg", 1) 4.5. pvh_append_header(hname, hvalue) 4.6. pvh_modify_header(hname, hvalue, [idx]) 4.7. pvh_remove_header(hname, [idx]) - - \ + 4.8. pvh_value_exists(hname, hparameter) + 4.9. pvh_remove_header_param(hname, hparameter) 4.1. pvh_collect_headers() @@ -317,6 +326,32 @@ modparam("pv_headers", "auto_msg", 1) This function can be used from ANY_ROUTE but only after pvh_collect_headers() or with “auto_msg” parameter enabled. +4.8. pvh_value_exists(hname, hparameter) + + Checks if the parameter “hparameter” is present in the header “hname” + from XAVP. + + This function can be used from ANY_ROUTE but only after + pvh_collect_headers() have been called or with “auto_msg” parameter + enabled. + +4.9. pvh_remove_header_param(hname, hparameter) + + Removes an existing parameter “hparameter” in the header “hname” from + the XAVP. + + If there are multiple headers, only the one containing the parameter + will be modified. + + The parameter can be located in any position (begining, middle or end) + of the list of parameters. + + If the parameter is the only one present, the header will be removed. + + This function can be used from ANY_ROUTE but only after + pvh_collect_headers() have been called or with “auto_msg” parameter + enabled. + 5. Exported Variables * $x_hdr

1 0

[kamailio/kamailio] pv_headers: Add support to remove a specific value from a header (PR #3070)
by Tomohare 11 Apr '22

11 Apr '22

- To be used with headers containing values separated by commas - pvh_header_param_exists() check if the value is present to a given header - pvh_remove_header_param() remove the value from the given header or the entire header if no other value is present   #### 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:  - [ ] PR should be backported to stable branches - [X] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  Previously, to remove a parameter from a header it was needed to create a complex method on the configuration file for each header/parameter. This modification moves this complexity to the 'pv_headers' module. Two new functions: * pvh_header_param_exists * pvh_remove_header_param Both functions take as parameters the name of the header and the name of the parameter. It returns '-1' if the combination header/parameter: - is not found (pvh_header_param_exists) - couldn't be removed (pvh_remove_header_param). Examples (proxy configuration): if (pvh_header_param_exists("Supported", "100rel")) { // "100rel" is present on the "Supported" header } if (pvh_remove_header_param("Supported", "100rel")) { // "100rel" removed from header "Supported" } You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3070 -- Commit Summary -- * pv_headers: Add support to remove a specific value from a header -- File Changes -- M src/modules/pv_headers/doc/functions.xml (35) M src/modules/pv_headers/doc/pv_headers.xml (6) M src/modules/pv_headers/pv_headers.c (61) M src/modules/pv_headers/pvh_func.c (87) M src/modules/pv_headers/pvh_func.h (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3070.patch https://github.com/kamailio/kamailio/pull/3070.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3070 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3070(a)github.com>

4 4

git:master:7a3f71e7: pv_headers: Add support to remove a specific value from a header
by Victor Seva 11 Apr '22

11 Apr '22

Module: kamailio Branch: master Commit: 7a3f71e7ce6b63fe1c32853c30d785bc9c1f2236 URL: https://github.com/kamailio/kamailio/commit/7a3f71e7ce6b63fe1c32853c30d785b… Author: Fabricio Santolin da Silva <fabricio.santolin-da-silva(a)al-enterprise.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2022-04-11T15:10:11+02:00 pv_headers: Add support to remove a specific value from a header - To be used with headers containing values separated by commas - pvh_header_param_exists() check if the value is present to a given header - pvh_remove_header_param() remove the value from the given header or the entire header if no other value is present --- Modified: src/modules/pv_headers/doc/functions.xml Modified: src/modules/pv_headers/doc/pv_headers.xml Modified: src/modules/pv_headers/pv_headers.c Modified: src/modules/pv_headers/pvh_func.c Modified: src/modules/pv_headers/pvh_func.h --- Diff: https://github.com/kamailio/kamailio/commit/7a3f71e7ce6b63fe1c32853c30d785b… Patch: https://github.com/kamailio/kamailio/commit/7a3f71e7ce6b63fe1c32853c30d785b…

1 0

git:master:322102e6: fix compilation warning for gcc 12.x (GH #3073)
by Henning Westerholt 11 Apr '22

11 Apr '22

Module: kamailio Branch: master Commit: 322102e6a6b9e775aaa5d9f738e549ba8dddf2cd URL: https://github.com/kamailio/kamailio/commit/322102e6a6b9e775aaa5d9f738e549b… Author: Henning Westerholt <hw(a)gilawa.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2022-04-11T07:40:09Z fix compilation warning for gcc 12.x (GH #3073) --- Modified: src/Makefile.defs --- Diff: https://github.com/kamailio/kamailio/commit/322102e6a6b9e775aaa5d9f738e549b… Patch: https://github.com/kamailio/kamailio/commit/322102e6a6b9e775aaa5d9f738e549b… --- diff --git a/src/Makefile.defs b/src/Makefile.defs index d2d7c60814..d3ca839c0d 100644 --- a/src/Makefile.defs +++ b/src/Makefile.defs @@ -283,7 +283,7 @@ ifneq (,$(findstring gcc, $(CC_LONGVER))) # sed with POSIX.1 regex doesn't support |, + or ? # (darwin, solaris ...) => this complicated expression MKDEP=$(CC) -MM -MG - #transform gcc version into 2.9x or 3.0 + #transform gcc version into some well-known versions, or 9 for newer versions CC_SHORTVER:=$(shell echo "$(CC_VER)" | cut -d" " -f 2| \ sed -e 's/[^0-9]*-$.*$/\1/'| \ sed -e 's/2\.9.*/2.9x/' \ @@ -310,7 +310,9 @@ ifneq (,$(findstring gcc, $(CC_LONGVER))) -e 's/10\.[0-9]\..*/9.0+/' \ -e 's/10\.[0-9]$$/9.0+/' \ -e 's/11\.[0-9]\..*/9.0+/' \ - -e 's/11\.[0-9]$$/9.0+/') + -e 's/11\.[0-9]$$/9.0+/' \ + -e 's/12\.[0-9]\..*/9.0+/' \ + -e 's/12\.[0-9]$$/9.0+/') ifeq (,$(strip $(filter-out 3.0 3.4 4.x 4.2+ 4.5+ 5.0+ 6.0+ 7.0+ 8.0+ 9.0+,$(CC_SHORTVER)))) # dependencies can be generated on-the-fly while compiling *.c CC_MKDEP_OPTS=-MMD -MP

1 0

[kamailio/kamailio] Does Kamailio process each SIP message in a separate process? (Issue #3075)
by jinhua luo 11 Apr '22

11 Apr '22

I check the codes and it seems that in `udp_rcv_loop()`, each incoming message got processed in the same process, so may it get blocked by some module function? For example, when the script calls `rtpengine_manage()`, it opens a new socket, sends a message, and receives the reply in blocking way, then it blocks the whole `udp_rcv_loop()`. Is it the expected design goal? why not consider new process or threading? -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3075 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3075(a)github.com>

2 2

git:master:4033b3be: modules: readme files regenerated - dispatcher ... [skip ci]
by Kamailio Dev 11 Apr '22

11 Apr '22

Module: kamailio Branch: master Commit: 4033b3be916eef91773fa8c3f04ee7e8f954b7f0 URL: https://github.com/kamailio/kamailio/commit/4033b3be916eef91773fa8c3f04ee7e… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2022-04-11T08:46:20+02:00 modules: readme files regenerated - dispatcher ... [skip ci] --- Modified: src/modules/dispatcher/README Modified: src/modules/ipops/README --- Diff: https://github.com/kamailio/kamailio/commit/4033b3be916eef91773fa8c3f04ee7e… Patch: https://github.com/kamailio/kamailio/commit/4033b3be916eef91773fa8c3f04ee7e…

1 0

← Newer
1
...
20
21
22
23
24
25
26
...
29
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev April 2022