sr-dev April 2022

sr-dev@lists.kamailio.org

17 participants
283 discussions

git:master:dd15436e: auth: Fix missing dependency in docs

by Henning Westerholt

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>

2 years

git:master:8bab7115: core: Fix typos, reference to SER

by Henning Westerholt

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) {

2 years

git:master:abd41396: modules: readme files regenerated - pv_headers ... [skip ci]

by Kamailio Dev

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

2 years

[kamailio/kamailio] pv_headers: Add support to remove a specific value from a header (PR #3070)

by Tomohare

- 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>

2 years

git:master:7a3f71e7: pv_headers: Add support to remove a specific value from a header

by Victor Seva

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…

2 years

git:master:322102e6: fix compilation warning for gcc 12.x (GH #3073)

by Henning Westerholt

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

2 years

[kamailio/kamailio] Does Kamailio process each SIP message in a separate process? (Issue #3075)

by jinhua luo

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 years

git:master:4033b3be: modules: readme files regenerated - dispatcher ... [skip ci]

by Kamailio Dev

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…

2 years

git:master:06aa21a9: ipops: docs for dns_set_local_ttl()

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 06aa21a975f76ce49a10925972ad8c239fc91216 URL: https://github.com/kamailio/kamailio/commit/06aa21a975f76ce49a10925972ad8c2… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-04-11T08:42:57+02:00 ipops: docs for dns_set_local_ttl() --- Modified: src/modules/ipops/doc/ipops_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/06aa21a975f76ce49a10925972ad8c2… Patch: https://github.com/kamailio/kamailio/commit/06aa21a975f76ce49a10925972ad8c2… --- diff --git a/src/modules/ipops/doc/ipops_admin.xml b/src/modules/ipops/doc/ipops_admin.xml index a04a42ab16..9b7da7a974 100644 --- a/src/modules/ipops/doc/ipops_admin.xml +++ b/src/modules/ipops/doc/ipops_admin.xml @@ -1064,6 +1064,46 @@ if (naptr_query ("example.com", "res") > 0) { </section> + <section id="ipops.f.dns_set_local_ttl"> + <title> + <function moreinfo="none">dns_set_local_ttl(vttl)</function> + </title> + + <para> + Set local ttl for DNS query results. If vttl is 0, then the value of + the result and the core parameters are used. + </para> + + <para>Parameters:</para> + + <itemizedlist> + <listitem> + <para> + <emphasis>vttl</emphasis> - TTL value in seconds. It can be static + integer or a variable holding an integer value. + </para> + </listitem> + </itemizedlist> + + <para> + This function can be used from ANY_ROUTE. + </para> + + <example> + <title> + <function>dns_set_local_ttl</function> usage + </title> + <programlisting format="linespecific"> +... + dns_set_local_ttl("7200"); + dns_query("test.com", "xyz") + dns_set_local_ttl("0"); +... + </programlisting> + </example> + + </section> + </section> - + </chapter>

2 years

git:master:ac376471: ipops: added function to set local ttl for dns queries

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: ac376471d95ab7a350d9d7cc124c06a157776dff URL: https://github.com/kamailio/kamailio/commit/ac376471d95ab7a350d9d7cc124c06a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-04-11T08:31:20+02:00 ipops: added function to set local ttl for dns queries --- Modified: src/modules/ipops/ipops_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/ac376471d95ab7a350d9d7cc124c06a… Patch: https://github.com/kamailio/kamailio/commit/ac376471d95ab7a350d9d7cc124c06a… --- diff --git a/src/modules/ipops/ipops_mod.c b/src/modules/ipops/ipops_mod.c index 273f232fa7..49b9ceb653 100644 --- a/src/modules/ipops/ipops_mod.c +++ b/src/modules/ipops/ipops_mod.c @@ -46,6 +46,7 @@ #include "../../core/mod_fix.h" #include "../../core/pvar.h" #include "../../core/resolve.h" +#include "../../core/dns_cache.h" #include "../../core/lvalue.h" #include "../../core/kemi.h" #include "api.h" @@ -110,6 +111,7 @@ static int fixup_free_detailed_ip_type(void** param, int param_no); static int w_dns_query(sip_msg_t* msg, char* str1, char* str2); static int w_srv_query(sip_msg_t* msg, char* str1, char* str2); static int w_naptr_query(sip_msg_t* msg, char* str1, char* str2); +static int w_dns_set_local_ttl(sip_msg_t*, char*, char*); static int mod_init(void); static pv_export_t mod_pvs[] = { @@ -165,6 +167,8 @@ static cmd_export_t cmds[] = ANY_ROUTE }, { "naptr_query", (cmd_function)w_naptr_query, 2, fixup_spve_spve, 0, ANY_ROUTE }, + { "dns_set_local_ttl", (cmd_function)w_dns_set_local_ttl, 1, fixup_igp_null, + fixup_free_igp_null, ANY_ROUTE }, { "bind_ipops", (cmd_function)bind_ipops, 0, 0, 0, 0}, { 0, 0, 0, 0, 0, 0 } }; @@ -1358,6 +1362,30 @@ static int ki_naptr_query(sip_msg_t* msg, str* naptrname, str* pvid) return naptr_update_pv(naptrname, pvid); } +/** + * + */ +static int ki_dns_set_local_ttl(sip_msg_t* msg, int vttl) +{ + dns_set_local_ttl(vttl); + return 1; +} + +/** + * + */ +static int w_dns_set_local_ttl(sip_msg_t *msg, char *pttl, char *p2) +{ + int vttl = 0; + + if(fixup_get_ivalue(msg, (gparam_t*)pttl, &vttl)<0) { + LM_ERR("cannot get the ttl value\n"); + return -1; + } + + return ki_dns_set_local_ttl(msg, vttl); +} + /** * */ @@ -1458,6 +1486,11 @@ static sr_kemi_t sr_kemi_ipops_exports[] = { { SR_KEMIP_STR, SR_KEMIP_STR, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } }, + { str_init("ipops"), str_init("dns_set_local_ttl"), + SR_KEMIP_INT, ki_dns_set_local_ttl, + { SR_KEMIP_INT, 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 } } };

2 years

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

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev April 2022