sr-dev June 2025

sr-dev@lists.kamailio.org

20 participants
221 discussions

[kamailio/kamailio] extra bytes should be ignored when Content-Length header present (Issue #4290)
by sergey-safarov 22 Jun '25

22 Jun '25

sergey-safarov created an issue (kamailio/kamailio#4290) ### Description According to [RFC3261](https://www.ietf.org/rfc/rfc3261.html#section-18.3) Section 18.3 ``` In the case of message-oriented transports (such as UDP), if the message has a Content-Length header field, the message body is assumed to contain that many bytes. If there are additional bytes in the transport packet beyond the end of the body, they MUST be discarded. ``` In the current Kamailio implementation, extra bytes are analyzed, and this triggers a failed sanity check. ### Troubleshooting #### Reproduction To reproduce required prepare config file ``` listen=udp:127.0.0.1:5060 loadmodule "xlog.so" loadmodule "sl.so" loadmodule "sanity.so" modparam("sanity", "autodrop", 0) modparam("sanity", "noreply", 1) route{ if(sanity_check()) { sl_send_reply("200", "OK"); exit; } xerr("sanity checks failed\n"); sanity_reply(); drop; } ``` And start using a command like ``` kamailio -DD -E -f kamailio.cfg ``` Then need to send options using a command like ``` cat options-extra.txt | nc -u 127.0.0.1 5060 ``` Where `options-extra.txt` is a file from the attached archive [options.tar.gz](https://github.com/user-attachments/files/20838024/options.… If you remove extra bytes after the OPTIONS message, then Kamailio does not generate an error. This can be tested using the command ``` cat options-normal.txt | nc -u 127.0.0.1 5060 ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4290 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4290(a)github.com>

3 6

git:6.0:4e221917: pkg/kamailio/alpine: add missing dependency for jwt module [skip ci]
by Sergey Safarov 21 Jun '25

21 Jun '25

Module: kamailio Branch: 6.0 Commit: 4e221917fc18463e8a7a1a5f6db13e8da6785ed0 URL: https://github.com/kamailio/kamailio/commit/4e221917fc18463e8a7a1a5f6db13e8… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Sergey Safarov <s.safarov(a)anycast-lb.net> Date: 2025-06-21T21:17:31+03:00 pkg/kamailio/alpine: add missing dependency for jwt module [skip ci] related #4132 (cherry picked from commit 95d0c1fc38da00dc6bbda19dba1ac4b20535abf5) --- Modified: pkg/kamailio/alpine/APKBUILD --- Diff: https://github.com/kamailio/kamailio/commit/4e221917fc18463e8a7a1a5f6db13e8… Patch: https://github.com/kamailio/kamailio/commit/4e221917fc18463e8a7a1a5f6db13e8… --- diff --git a/pkg/kamailio/alpine/APKBUILD b/pkg/kamailio/alpine/APKBUILD index fe941ec28e5..2215305313f 100644 --- a/pkg/kamailio/alpine/APKBUILD +++ b/pkg/kamailio/alpine/APKBUILD @@ -32,7 +32,7 @@ makedepends="bison db-dev flex freeradius-client-dev expat-dev net-snmp-dev libuuid libev-dev jansson-dev json-c-dev libevent-dev linux-headers libmemcached-dev rabbitmq-c-dev hiredis-dev ruby-dev libmaxminddb-dev libunistring-dev mongo-c-driver-dev - libwebsockets-dev mosquitto-dev librdkafka-dev nghttp2-dev" + libwebsockets-dev mosquitto-dev librdkafka-dev nghttp2-dev libjwt-dev" install="$pkgname.pre-install $pkgname.pre-upgrade"

1 0

git:6.0:6edde708: pkg/kamailio: for dispatcher work required working DNS resolution daemon
by Sergey Safarov 21 Jun '25

21 Jun '25

Module: kamailio Branch: 6.0 Commit: 6edde7080fd5f7fb6a547f6de5c5b7c104cb493a URL: https://github.com/kamailio/kamailio/commit/6edde7080fd5f7fb6a547f6de5c5b7c… Author: Sergey Safarov <s.safarov(a)anycast-lb.net> Committer: Sergey Safarov <s.safarov(a)anycast-lb.net> Date: 2025-06-21T21:17:12+03:00 pkg/kamailio: for dispatcher work required working DNS resolution daemon (cherry picked from commit aa7b2460ff0d8085ddc8ae9232a9fea0fe0db5f8) --- Modified: pkg/kamailio/obs/kamailio.service --- Diff: https://github.com/kamailio/kamailio/commit/6edde7080fd5f7fb6a547f6de5c5b7c… Patch: https://github.com/kamailio/kamailio/commit/6edde7080fd5f7fb6a547f6de5c5b7c… --- diff --git a/pkg/kamailio/obs/kamailio.service b/pkg/kamailio/obs/kamailio.service index 0c4fbc49caf..0a527056f91 100644 --- a/pkg/kamailio/obs/kamailio.service +++ b/pkg/kamailio/obs/kamailio.service @@ -1,8 +1,8 @@ [Unit] Description=Kamailio - the Open Source SIP Server Documentation=man:kamailio(8) -Wants=network-online.target -After=network-online.target +Wants=network-online.target nss-lookup.target +After=network-online.target nss-lookup.target [Service] Type=simple

1 0

git:6.0:d4159f45: pkg/kamailio: updated alpine build rules
by Sergey Safarov 21 Jun '25

21 Jun '25

Module: kamailio Branch: 6.0 Commit: d4159f45972bad6151649d190f4e6c35c3259e5f URL: https://github.com/kamailio/kamailio/commit/d4159f45972bad6151649d190f4e6c3… Author: Sergey Safarov <s.safarov(a)anycast-lb.net> Committer: Sergey Safarov <s.safarov(a)anycast-lb.net> Date: 2025-06-21T17:21:54+03:00 pkg/kamailio: updated alpine build rules (cherry picked from commit 9ddd70a45168c618a3ef2356689890daee8c8415) --- Modified: pkg/kamailio/Makefile --- Diff: https://github.com/kamailio/kamailio/commit/d4159f45972bad6151649d190f4e6c3… Patch: https://github.com/kamailio/kamailio/commit/d4159f45972bad6151649d190f4e6c3… --- diff --git a/pkg/kamailio/Makefile b/pkg/kamailio/Makefile index ce2e3ae7327..6f1a18ec3c6 100644 --- a/pkg/kamailio/Makefile +++ b/pkg/kamailio/Makefile @@ -32,8 +32,8 @@ apk: sed -e "s/^_gitcommit=.*/_gitcommit=${GIT_TAG}/" \ -e "s/^pkgver=.*/pkgver=${APK_VERSION}/" \ -i alpine/APKBUILD - cd alpine && abuild snapshot - @echo \"abuild\" cannot be started from Makefile please start manually \"cd alpine \&\& abuild -r\" + abuild -C alpine snapshot + @echo \"abuild\" cannot be started from Makefile please start manually \"abuild -C alpine -r\" # build source rpm package .PHONY: src.rpm

1 0

git:6.0:26bd6104: github: refactored alpine docker image generation
by Sergey Safarov 21 Jun '25

21 Jun '25

Module: kamailio Branch: 6.0 Commit: 26bd61047c446eb509691765ad3fa60292a0b25d URL: https://github.com/kamailio/kamailio/commit/26bd61047c446eb509691765ad3fa60… Author: Sergey Safarov <s.safarov(a)anycast-lb.net> Committer: Sergey Safarov <s.safarov(a)anycast-lb.net> Date: 2025-06-21T17:00:04+03:00 github: refactored alpine docker image generation - GH #4241 (cherry picked from commit de4e36779ec0d01e4ba749318677e29472de5b55) Signed-off-by: Sergey Safarov <s.safarov(a)anycast-lb.net> --- Modified: .github/workflows/alpine.yml --- Diff: https://github.com/kamailio/kamailio/commit/26bd61047c446eb509691765ad3fa60… Patch: https://github.com/kamailio/kamailio/commit/26bd61047c446eb509691765ad3fa60…

1 0

git:6.0:7022d949: pkg/docker: submodule update
by Sergey Safarov 21 Jun '25

21 Jun '25

Module: kamailio Branch: 6.0 Commit: 7022d9493d4db047d5fa2d67e589c05f43aec5e9 URL: https://github.com/kamailio/kamailio/commit/7022d9493d4db047d5fa2d67e589c05… Author: Sergey Safarov <s.safarov(a)anycast-lb.net> Committer: Sergey Safarov <s.safarov(a)anycast-lb.net> Date: 2025-06-21T16:59:26+03:00 pkg/docker: submodule update --- Modified: pkg/docker --- Diff: https://github.com/kamailio/kamailio/commit/7022d9493d4db047d5fa2d67e589c05… Patch: https://github.com/kamailio/kamailio/commit/7022d9493d4db047d5fa2d67e589c05… --- diff --git a/pkg/docker b/pkg/docker index cf48a82c5f9..75be900d83b 160000 --- a/pkg/docker +++ b/pkg/docker @@ -1 +1 @@ -Subproject commit cf48a82c5f9df67813d14e6619cc9a41ec9a683b +Subproject commit 75be900d83bb70b3ec77443fdbb309b9ef1a6d80

1 0

[kamailio/kamailio] prefix_route - prefix longer than 12 characters ignored (Issue #4218)
by Barry Flanagan 21 Jun '25

21 Jun '25

flantel created an issue (kamailio/kamailio#4218) ### Description If I add a prefix of greater than 12 characters in to prefix_route it does not return the correct route but instead acts as if no record was found. ### Troubleshooting #### Reproduction Create a prefix_route prefix of more than 12 characters and the route will not be chosen. #### Debugging Data There is none. #### Log Messages prefix_route does not seem to produce DEBUG level messages. ### Additional Information ``` kamailio 5.8.6 (x86_64/linux) ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4218 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4218(a)github.com>

3 4

[kamailio/kamailio] topoh: fix bug in hiding record routes in reply messages (PR #4170)
by Farzaneh Soltanzadeh 21 Jun '25

21 Jun '25

#### Pre-Submission Checklist    - [ ] Commit message has the format required by CONTRIBUTING guide - [ ] Commits are split per component (core, individual modules, libs, utils, ...) - [ ] Each component has a single commit (if not, squash them into one commit) - [ ] 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/4170 -- Commit Summary -- * hide all record routes except last one in reply messages -- File Changes -- M src/modules/topoh/th_msg.c (16) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4170.patch https://github.com/kamailio/kamailio/pull/4170.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4170 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4170(a)github.com>

3 13

Re: [kamailio/kamailio] aaaa (2698244)
by Henning Westerholt 20 Jun '25

20 Jun '25

The commits were reverted (details at https://kamailio.org/mailman3/hyperkitty/list/sr-dev@lists.kamailio.org/mes… ) - but I was reluctant to do a force-push on a public repository, so there were still visible. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/commit/26982449b7dabe0413258a7602d8810… You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/commit/26982449b7dabe0413258a7602d88107c8f7e7d9/160481195(a)github.com>

1 0

Re: [kamailio/kamailio] aaaa (2698244)
by sergey-safarov 20 Jun '25

20 Jun '25

Hello @miconda By mistake, I pushed my private branch to the Kamailio repo and immediately sent a message to the dev maillist to force revert to the correct commit. For some reason, it hasn't happened. yes, Daiel I understand this should not happen. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/commit/26982449b7dabe0413258a7602d8810… You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/commit/26982449b7dabe0413258a7602d88107c8f7e7d9/160477300(a)github.com>

1 0

← Newer
1
...
4
5
6
7
8
9
10
...
23
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev June 2025