sr-dev August 2023

sr-dev@lists.kamailio.org

15 participants
210 discussions

[kamailio/kamailio] [app_lua] clarify the file scripts can be in source or bytecode (PR #3550)
by Дилян Палаузов 12 Sep '23

12 Sep '23

This is valid both for Lua.org and LuaJIT. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3550 -- Commit Summary -- * [app_lua] clarify the file scripts can be in source or bytecode -- File Changes -- M src/modules/app_lua/doc/app_lua_admin.xml (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3550.patch https://github.com/kamailio/kamailio/pull/3550.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3550 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3550(a)github.com>

2 1

[kamailio/kamailio] kemi exports (PR #3544)
by Stefan Mititelu 12 Sep '23

12 Sep '23

#### 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 - [ ] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  some kemi exports for ims_qos and ims_dialog. Currently testing them, will add comment afterwards. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3544 -- Commit Summary -- * ims_qos: kemi export * ims_dialog: kemi export some functions -- File Changes -- M src/modules/ims_dialog/ims_dialog.c (96) M src/modules/ims_qos/ims_qos_mod.c (72) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3544.patch https://github.com/kamailio/kamailio/pull/3544.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3544 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3544(a)github.com>

2 9

[kamailio/kamailio] statsd: Add labels to metrics. (PR #3529)
by Eloy Coto 12 Sep '23

12 Sep '23

A user reached me to add a way to add the custom labels into the statsd module, so a better way to report metrics to the observability platform. I keep the same old functions and add a new parameter to the statsd modules to both interfaces cfg and kemi. The full documentation can be found here: https://docs.datadoghq.com/developers/dogstatsd/datagram_shell/?tab=metrics So, each function can be used like this: ``` statsd_set("fooo", 1, "inbound"); statsd_gauge("NotFound", "+1", "outbound,carrierFoo"); statsd_gauge("AuthFailed", "+1", "carrier=foo,priority=10"); ``` Signed-off-by: Eloy Coto <eloy.coto(a)acalustra.com> Tested-by: Alex Antonevych <alex.antonevych(a)replicant.ai>  #### 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 I'm waiting until first review on function arguments to create the docs changes. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3529 -- Commit Summary -- * statsd: Add labels to metrics. -- File Changes -- M src/modules/statsd/lib_statsd.c (102) M src/modules/statsd/lib_statsd.h (10) M src/modules/statsd/statsd.c (129) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3529.patch https://github.com/kamailio/kamailio/pull/3529.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3529 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3529(a)github.com>

3 8

[kamailio/kamailio] pdt: make add_to_tree() idempotent (PR #3485)
by vcill 12 Sep '23

12 Sep '23

- the function call will suceed even if the prefix being inserted already exists in the tree   #### 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  PDT module initialization currently fails if you try to insert a prefix multiple times. We use kamailio as a loadbalancer and have had this change out in the field for quite some time now as our database often contains a prefix multiple times (associated with the same domain). Would it make sense to change the behavior in this case ? Thanks, Vladimir. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3485 -- Commit Summary -- * pdt: make add_to_tree() idempotent -- File Changes -- M src/modules/pdt/pdtree.c (4) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3485.patch https://github.com/kamailio/kamailio/pull/3485.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3485 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3485(a)github.com>

2 5

[kamailio/kamailio] lcr: pcre2 migration (PR #3545)
by Victor Seva 12 Sep '23

12 Sep '23

#### 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:  - [ ] PR should be backported to stable branches - [ ] Tested changes locally - [x] Related to issue #2946 #### Description migrate lcr to pcre2 You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3545 -- Commit Summary -- * lcr: pcre2 migration -- File Changes -- M src/modules/lcr/Makefile (12) M src/modules/lcr/hash.c (24) M src/modules/lcr/hash.h (10) M src/modules/lcr/lcr_mod.c (187) M src/modules/lcr/lcr_mod.h (8) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3545.patch https://github.com/kamailio/kamailio/pull/3545.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3545 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3545(a)github.com>

2 2

[kamailio/kamailio] dialplan: pcre2 migration (PR #3543)
by Victor Seva 12 Sep '23

12 Sep '23

#### 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:  - [ ] PR should be backported to stable branches - [ ] Tested changes locally - [x] Related to issue #2946 #### Description migrate dialplan to pcre2 You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3543 -- Commit Summary -- * regex: migration to pcre2 * dialplan: migrate to pcre2 -- File Changes -- M src/modules/dialplan/Makefile (11) M src/modules/dialplan/dialplan.c (5) M src/modules/dialplan/dialplan.h (21) M src/modules/dialplan/dp_db.c (103) M src/modules/dialplan/dp_repl.c (56) M src/modules/regex/Makefile (11) M src/modules/regex/regex_mod.c (240) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3543.patch https://github.com/kamailio/kamailio/pull/3543.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3543 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3543(a)github.com>

2 4

[kamailio/kamailio] smsops: fixes as per static code analyzer report (PR #3551)
by Supreeth Herle 12 Sep '23

12 Sep '23

#### 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:  - [ ] PR should be backported to stable branches - [ ] 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/3551 -- Commit Summary -- * smsops: fixes as per static code analyzer report -- File Changes -- M src/modules/smsops/smsops_impl.c (12) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3551.patch https://github.com/kamailio/kamailio/pull/3551.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3551 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3551(a)github.com>

2 3

[kamailio/kamailio] Function `sdp_mangle_ip("IP/MASK","IP")` Malfunction (Issue #3554)
by teamwcp 06 Sep '23

06 Sep '23

I have been trying to replace the SDP connection IP with some other IP using module `mangler` (`Kamailio v5.7.0`). But It seem like its malfunctioning ``` if ($rs == 200) { xlog("L_INFO", "200 $rm Detected"); sdp_mangle_ip("192.168.0.132/24","8.8.8.8"); } ``` and results I received are like this below ``` v=0 o=- 107808851201429815 2 IN IP4 192.168.0.132 s=- t=0 0 m=audio 24794 RTP/AVP 0 8 101 c=IN IP4 192.168.0.1328.8.8.8 a=mid:audio a=sendrecv a=rtpmap:0 PCMU/8000 a=rtpmap:8 PCMA/8000 a=rtpmap:101 telephone-event/8000 a=ssrc:822878909 cname:0ZfUBhl8cMeHaeVl a=nortpproxy:yes ``` I first ask the same question in mailing list `sr-users(a)lists.kamailio.org` but after waiting for some time (>24hrs) i cant see my question on on forum, that why I open a ticket here. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3554 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3554(a)github.com>

2 2

Kamailio Patching Issues
by Bin LIU 03 Sep '23

03 Sep '23

Halo Experts, So sorry to disturb you for the kamailio start failure issue I faced recently when doing patching from version 5.5.4 to 5.5.6. Really appreciated if you could give me some assistance on it ! And attached is the start log. Thanks. Best regards, Louis LIU Senior Software Engineer (Cloud Platform && Audio/Video Intercom) M +86 18575580920 8F-A CES Building,No.3099 Global Offer Innovation E bin.liu2(a)se.com<mailto:bin.liu2@se.com> Keyuan South Road, Nanshan District Home & Distribution Division, Energy Management Shenzhen, China [cid:image001.png@01D9CAE5.A472D190] Download mySchneider app<https://www.se.com/ww/en/work/support/myschneider-app/> 24/7 support Mobile catalog. Acces to expert help. Internal

2 4

git:master:b56110ac: usrloc: check for enough space to build aor on udomain_contact_expired_cb()
by Daniel-Constantin Mierla 29 Aug '23

29 Aug '23

Module: kamailio Branch: master Commit: b56110aca34a5633029654ae457ff3ed4424e62f URL: https://github.com/kamailio/kamailio/commit/b56110aca34a5633029654ae457ff3e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2023-08-29T17:18:35+02:00 usrloc: check for enough space to build aor on udomain_contact_expired_cb() --- Modified: src/modules/usrloc/udomain.c --- Diff: https://github.com/kamailio/kamailio/commit/b56110aca34a5633029654ae457ff3e… Patch: https://github.com/kamailio/kamailio/commit/b56110aca34a5633029654ae457ff3e… --- diff --git a/src/modules/usrloc/udomain.c b/src/modules/usrloc/udomain.c index 578ae208aa4..43278fca004 100644 --- a/src/modules/usrloc/udomain.c +++ b/src/modules/usrloc/udomain.c @@ -1071,7 +1071,7 @@ int udomain_contact_expired_cb(db1_con_t *_c, udomain_t *_d) continue; } user.len = strlen(user.s); - if(user.len < AORBUF_SIZE) { + if(user.len + 1 < AORBUF_SIZE) { memcpy(aorbuf, user.s, user.len); aor.s = aorbuf; aor.len = user.len; @@ -1090,7 +1090,7 @@ int udomain_contact_expired_cb(db1_con_t *_c, udomain_t *_d) if(ul_use_domain) { domain.s = (char *)VAL_STRING(ROW_VALUES(row) + 20); domain.len = strlen(domain.s); - if(domain.len + aor.len < AORBUF_SIZE) { + if(domain.len + aor.len + 2 < AORBUF_SIZE) { aorbuf[aor.len] = '@'; memcpy(aorbuf + aor.len + 1, domain.s, domain.len); aor.len += domain.len + 1;

1 0

← Newer
1
...
11
12
13
14
15
16
17
...
21
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev August 2023