sr-dev December 2021

sr-dev@lists.kamailio.org

11 participants
269 discussions

git:master:090f1f74: siputils: docs - note about restrictions of ring_insert_callid()

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 090f1f74603de6113f3dd41095ae41b4f03f8171 URL: https://github.com/kamailio/kamailio/commit/090f1f74603de6113f3dd41095ae41b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2021-12-28T20:00:17+01:00 siputils: docs - note about restrictions of ring_insert_callid() - GH #2989 --- Modified: src/modules/siputils/doc/siputils_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/090f1f74603de6113f3dd41095ae41b… Patch: https://github.com/kamailio/kamailio/commit/090f1f74603de6113f3dd41095ae41b… --- diff --git a/src/modules/siputils/doc/siputils_admin.xml b/src/modules/siputils/doc/siputils_admin.xml index 2b7ac27951..db66d40ac6 100644 --- a/src/modules/siputils/doc/siputils_admin.xml +++ b/src/modules/siputils/doc/siputils_admin.xml @@ -286,6 +286,12 @@ modparam("auth", "rpid_avp", "$avp(myrpid)") to use this function. </para> <para> + Note: the change of sip reply can be done only in cases when it is not + parsed too much before config execution, otherwise it fails. For + example, setting global parameter "log_prefix" can cause this function + to fail changing the replies. + </para> + <para> The function returns TRUE on success, and FALSE during processing failures. </para> <para>

2 years, 6 months

[kamailio/kamailio] Run kamailio in kubernates (Issue #2988)

by maryamb99

Hello everyone, I have a problem over running local docker images of Kamailio in Kubernate cluster.. Consider I build a local docker image from Kamailio and create a pod in one of my nodes . My process created successfully and Kamailio listen over port 5060 now I need to rebuild another docker image with new Kamailio configuration and run it as another pod with new name and listening over port 5061 but I have error "ImagePullBackOff" . I test this scenario many times but problem exists.. It is devastating for me and I need emergency help. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/2988 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/2988(a)github.com>

2 years, 6 months

[kamailio/kamailio] Remove RHEL/CentOS 6 packaging as EOL & other enhancements (PR #2986)

by istiak101

#### Pre-Submission Checklist    - [ ] 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:  - [ ] PR should be backported to stable branches - [ ] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description Removed RHEL/CentOS 6 packaging from spec file as those versions reached End of Life almost more than a year ago. Fixed a typo and did %autosetup instead of %setup -n which figures out name and version automatically from macros and also does a quiet decompression of source archive. Note: %autosetup has similar -n flag if archive name isn't standard like %{name}-%{version} You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2986 -- Commit Summary -- * Removed packaging for RHEL/CentOS 6 as End Of Life * Typo fix & %autosetup in prep section -- File Changes -- M pkg/kamailio/obs/kamailio.spec (104) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2986.patch https://github.com/kamailio/kamailio/pull/2986.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/2986 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/2986(a)github.com>

2 years, 6 months

Merry Christmas and Happy Holidays 2021-2022!

by Daniel-Constantin Mierla

Another challenging year is getting close to its end, pressuring even more everyone! Let's hope things will go back to a more usual life (as it used to be) in the near future! Kamailio project continued to develop, bringing out a new major release, preparing for the next one in a few months, relying on an amazing community of developers and users. We also trust it continued to help people and companies world wide to connect with the dear ones and build a better way of living. The season holidays are ahead, time to relax, spend time with family and friends! I take again this chance to express my thanks and greetings to all the friends, developers, supporting companies and community members that made 2021 another great year for Kamailio project. Enjoy the holidays! Merry Christmas! Daniel * Santa is flying Kamailio! * -- Daniel-Constantin Mierla -- www.asipto.com www.twitter.com/miconda -- www.linkedin.com/in/miconda

2 years, 6 months

[kamailio/kamailio] Add new parameter to $uac_req (Issue #2982)

by Neimar Avila

### Description If possible, add a new parameter, where you can choose between sending the password in ha1 or plain text. ### Expected behavior route[REGFWD] { if(!is_method("REGISTER")) { return; } $uac_req(method)="REGISTER"; $uac_req(auser)=$fU; $uac_req(apasswd)="4850730cadc0250e9fa951eae583ace9"; $uac_req(passtype)="ha1"; ... uac_req_send(); ... } #### Actual observed behavior It is currently not possible to send the password in ha1 format #### Debugging Data #### Log Messages #### SIP Traffic ### Possible Solutions Create an additional field. ### Additional Information version: kamailio 5.5.2 (x86_64/linux) 55e232 flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLOCKLIST, HAVE_RESOLV_RES ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB poll method support: poll, epoll_lt, epoll_et, sigio_rt, select. id: 55e232 compiled on 14:20:31 Aug 25 2021 with gcc 4.8.5 * **Operating System**: CentOS Linux release 7.9.2009 (Core) Linux vsphone-proxy-srv1.virtualsistemas.local 3.10.0-1160.45.1.el7.x86_64 #1 SMP Wed Oct 13 17:20:51 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/2982 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/2982(a)github.com>

2 years, 6 months

git:master:e809b241: uac: fix unknown type name

by Victor Seva

Module: kamailio Branch: master Commit: e809b2415b6074826bf641a0a85667a400f778ef URL: https://github.com/kamailio/kamailio/commit/e809b2415b6074826bf641a0a85667a… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2021-12-22T18:08:30+01:00 uac: fix unknown type name introduced at b6e264c9780fc0432e6e746f473024e41bc1126b > uac_send.c: In function 'uac_send_tm_callback': > uac_send.c:741:16: error: unknown type name 'authenticate_body_t' > 741 | static authenticate_body_t auth; > | ^~~~~~~~~~~~~~~~~~~ > uac_send.c:781:49: warning: passing argument 2 of 'parse_authenticate_body' from incompatible pointer type [-Wincompatible-pointer-types] > 781 | if (parse_authenticate_body(&hdr->body, &auth)<0) > | ^~~~~ > | | > | int * > In file included from uac_send.c:42: > auth_hdr.h:30:67: note: expected 'struct authenticate_body *' but argument is of type 'int *' > 30 | int parse_authenticate_body( str *body, struct authenticate_body *auth); > | ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~ > uac_send.c:788:27: error: request for member 'realm' in something not a structure or union > 788 | cred.realm = auth.realm; > | ^ > uac_send.c:795:56: warning: passing argument 4 of 'do_uac_auth' from incompatible pointer type [-Wincompatible-pointer-types] > 795 | do_uac_auth(&tp->s_method, &tp->s_ruri, &cred, &auth, response); > | ^~~~~ > | | > | int * > In file included from uac_send.c:41: > auth.h:77:43: note: expected 'struct authenticate_body *' but argument is of type 'int *' > 77 | struct authenticate_body *auth, > | ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~ > uac_send.c:797:49: warning: passing argument 4 of 'build_authorization_hdr' from incompatible pointer type [-Wincompatible-pointer-types] > 797 | &auth, response); > | ^~~~~ > | | > | int * > In file included from uac_send.c:42: > auth_hdr.h:33:71: note: expected 'struct authenticate_body *' but argument is of type 'int *' > 33 | struct uac_credential *crd, struct authenticate_body *auth, > | ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~ > make[2]: *** [../../Makefile.rules:100: uac_send.o] Error 1 --- Modified: src/modules/uac/uac_send.c --- Diff: https://github.com/kamailio/kamailio/commit/e809b2415b6074826bf641a0a85667a… Patch: https://github.com/kamailio/kamailio/commit/e809b2415b6074826bf641a0a85667a… --- diff --git a/src/modules/uac/uac_send.c b/src/modules/uac/uac_send.c index e5ea420d72..43767c7804 100644 --- a/src/modules/uac/uac_send.c +++ b/src/modules/uac/uac_send.c @@ -738,7 +738,7 @@ void uac_send_tm_callback(struct cell *t, int type, struct tmcb_params *ps) struct hdr_field *hdr; HASHHEX response; str *new_auth_hdr = NULL; - static authenticate_body_t auth; + static uac_authenticate_body_t auth; uac_credential_t cred; char b_hdrs[MAX_UACH_SIZE]; str s_hdrs;

2 years, 6 months

git:5.5:b42dfd30: pv_headers: restore pvh_real_hdr_del_by_name()

by Victor Seva

Module: kamailio Branch: 5.5 Commit: b42dfd30ba74562e3c7673b3de449760bb315d62 URL: https://github.com/kamailio/kamailio/commit/b42dfd30ba74562e3c7673b3de44976… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2021-12-22T11:21:15+01:00 pv_headers: restore pvh_real_hdr_del_by_name() it was wrongly removed at 09981c41cb6685fa18f61b9e7d66339917c4f097 (cherry picked from commit d8f0e71a0497e0bb98be60452914df51c84b56d8) --- Modified: src/modules/pv_headers/pvh_hdr.c --- Diff: https://github.com/kamailio/kamailio/commit/b42dfd30ba74562e3c7673b3de44976… Patch: https://github.com/kamailio/kamailio/commit/b42dfd30ba74562e3c7673b3de44976… --- diff --git a/src/modules/pv_headers/pvh_hdr.c b/src/modules/pv_headers/pvh_hdr.c index 31c09b4b5d..fbb471ccd5 100644 --- a/src/modules/pv_headers/pvh_hdr.c +++ b/src/modules/pv_headers/pvh_hdr.c @@ -131,6 +131,21 @@ int pvh_real_hdr_append(struct sip_msg *msg, str *hname, str *hvalue) return 1; } +int pvh_real_hdr_del_by_name(struct sip_msg *msg, str *hname) +{ + hdr_field_t *hf = NULL; + + for(hf = msg->headers; hf; hf = hf->next) { + if(hf->name.len == hname->len + && strncasecmp(hf->name.s, hname->s, hname->len) == 0) { + LM_DBG("remove header[%.*s]: %.*s\n", hf->name.len, hf->name.s, + hf->body.len, hf->body.s); + del_lump(msg, hf->name.s - msg->buf, hf->len, 0); + } + } + return 1; +} + int pvh_real_hdr_remove_display(struct sip_msg *msg, str *hname) { hdr_field_t *hf = NULL;

2 years, 6 months

git:master:d8f0e71a: pv_headers: restore pvh_real_hdr_del_by_name()

by Victor Seva

Module: kamailio Branch: master Commit: d8f0e71a0497e0bb98be60452914df51c84b56d8 URL: https://github.com/kamailio/kamailio/commit/d8f0e71a0497e0bb98be60452914df5… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2021-12-22T11:19:19+01:00 pv_headers: restore pvh_real_hdr_del_by_name() it was wrongly removed at 09981c41cb6685fa18f61b9e7d66339917c4f097 --- Modified: src/modules/pv_headers/pvh_hdr.c --- Diff: https://github.com/kamailio/kamailio/commit/d8f0e71a0497e0bb98be60452914df5… Patch: https://github.com/kamailio/kamailio/commit/d8f0e71a0497e0bb98be60452914df5… --- diff --git a/src/modules/pv_headers/pvh_hdr.c b/src/modules/pv_headers/pvh_hdr.c index 31c09b4b5d..fbb471ccd5 100644 --- a/src/modules/pv_headers/pvh_hdr.c +++ b/src/modules/pv_headers/pvh_hdr.c @@ -131,6 +131,21 @@ int pvh_real_hdr_append(struct sip_msg *msg, str *hname, str *hvalue) return 1; } +int pvh_real_hdr_del_by_name(struct sip_msg *msg, str *hname) +{ + hdr_field_t *hf = NULL; + + for(hf = msg->headers; hf; hf = hf->next) { + if(hf->name.len == hname->len + && strncasecmp(hf->name.s, hname->s, hname->len) == 0) { + LM_DBG("remove header[%.*s]: %.*s\n", hf->name.len, hf->name.s, + hf->body.len, hf->body.s); + del_lump(msg, hf->name.s - msg->buf, hf->len, 0); + } + } + return 1; +} + int pvh_real_hdr_remove_display(struct sip_msg *msg, str *hname) { hdr_field_t *hf = NULL;

2 years, 6 months

[kamailio/kamailio] nats: Add publish function (PR #2915)

by Stefan Mititelu

#### 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  Added a publish function to nats module. When the function is called, the routing process tries to connect to the url (param) and tries to send the payload (param) to the subject queue (param). No libuv is used for this. Probably a better approach is to make publisher workers (like subscriber workers) and pass payload from routing processes to publisher worker processes via shared memory. Right now I see this as an improvement and find the new function good enough, for now. @eschmidbauer what do you think of this? Thank you, Stefan P.S. Will update doc after review. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2915 -- Commit Summary -- * <a href="https://github.com/kamailio/kamailio/pull/2915/commits/121ae374e0c309027eeb…">nats: Add publish function</a> -- File Changes -- M src/modules/nats/nats_mod.c (71) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2915.patch https://github.com/kamailio/kamailio/pull/2915.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/2915

2 years, 6 months

[kamailio/kamailio] ldap: rename private function ldap_connect (PR #2975)

by micmac1

``` When compiling against openldap 2.6.0 kamailio's private ldap_connect() clashes with openldap's own. curl dealt with a similar issue earlier (see [1]). Simply rename the function to avoid the issue. In file included from ldap_api_fn.c:37: ldap_connect.h:34:12: error: conflicting types for 'ldap_connect'; have 'int(char *)' 34 | extern int ldap_connect(char *_ld_name); | ^~~~~~~~~~~~ In file included from ldap_api_fn.c:33: /home/sk/tmp/sdk/openwrt-sdk-ath79-generic_gcc-11.2.0_musl.Linux-x86_64/staging_dir/target-mips_24kc_musl/usr/include/ldap.h:1555:1: note: previous declaration of 'ldap_connect' with type 'int(LDAP *)' {aka 'int(struct ldap *)'} 1555 | ldap_connect( LDAP *ld ); | ^~~~~~~~~~~~ make[5]: *** [../../Makefile.rules:100: ldap_api_fn.o] Error 1 make[4]: *** [Makefile:511: modules] Error 1 ``` [1] https://github.com/curl/curl/commit/8bdde6b14ce3b5fd71c772a578fcbd4b6fa6df19 Signed-off-by: Sebastian Kemper <sebastian_ml(a)gmx.net>   #### 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 - [x] 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 - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  Hi all, Small compile fix. I saw kamailio is failing to build on OpenWrt build bots since a recent openldap bump from 2.4.58 to 2.6.0. I linked a curl commit in the commit message, they also renamed some of their functions, although they did rename all their ldap_* functions to oldap_. I didn't dare do that in kamailio :) Renaming a single function was enough to make it compile again. Maybe you want to do something different to address this issue, I don't know. I didn't runtest this. Kind regards, Seb You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2975 -- Commit Summary -- * ldap: rename private function ldap_connect -- File Changes -- M src/modules/ldap/ldap_connect.c (2) M src/modules/ldap/ldap_connect.h (2) M src/modules/ldap/ldap_mod.c (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2975.patch https://github.com/kamailio/kamailio/pull/2975.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/2975

2 years, 6 months

← Newer
1
...
9
10
11
12
13
14
15
...
27
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev December 2021