sr-dev

sr-dev@lists.kamailio.org

4 participants
44691 discussions

[kamailio/kamailio] kafka_send in event_route not work (Issue #4198)
by wangduanduan 02 Apr '25

02 Apr '25

wangduanduan created an issue (kamailio/kamailio#4198)  ### Description kafka_send in event_route like tcp:timeout, tcp:closed not work ``` event_route[tcp:closed] { if (kafka_send("my_topic", "tcp:closed")) { xlog("L_INFO", "send success tcp:closed"); } } event_route[tcp:timeout] { if (kafka_send("my_topic", "tcp:timeout")) { xlog("L_INFO", "send success tcp:timeout"); } } ``` in log, its show "send success tcp:closed", "send success tcp:timeout", but in kafka web ui, in the **my_topic**, can't find the "tcp:closed" or " tcp:timeout" message. --- but if the kafka_send in onreply_route, message will be send ok, in the kafka web ui, the "some_route" can be find. ``` onreplay_route[some_route]{ if (kafka_send("my_topic", "some_routet")) { xlog("L_INFO", "send success some_routet"); } } ```  ### Troubleshooting #### Reproduction  #### Debugging Data  ``` (paste your debugging data here) ``` #### Log Messages  ``` (paste your log messages here) ``` #### SIP Traffic i use tcpdump to capture the traffic between kamailio and kafka. - when use kafka_send in event_route, no packages captured.  ``` ``` ### Possible Solutions  ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` 5.6.1 ``` * **Operating System**: centos7  ``` (paste your output here) ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4198 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4198(a)github.com>

4 8

git:master:c532a67d: parser/sdp: check if value is exceeded when looking for o= version field
by Daniel-Constantin Mierla 02 Apr '25

02 Apr '25

Module: kamailio Branch: master Commit: c532a67db654efb21f81e058b7e3a86229766a40 URL: https://github.com/kamailio/kamailio/commit/c532a67db654efb21f81e058b7e3a86… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-04-02T09:38:03+02:00 parser/sdp: check if value is exceeded when looking for o= version field --- Modified: src/core/parser/sdp/sdp_helpr_funcs.c --- Diff: https://github.com/kamailio/kamailio/commit/c532a67db654efb21f81e058b7e3a86… Patch: https://github.com/kamailio/kamailio/commit/c532a67db654efb21f81e058b7e3a86… --- diff --git a/src/core/parser/sdp/sdp_helpr_funcs.c b/src/core/parser/sdp/sdp_helpr_funcs.c index 893ef0e58a4..5e9b3f121ab 100644 --- a/src/core/parser/sdp/sdp_helpr_funcs.c +++ b/src/core/parser/sdp/sdp_helpr_funcs.c @@ -735,7 +735,12 @@ int extract_sess_version(str *oline, str *sess_version) } i++; - } while(len < oline->len && i < 3); + } while((cp < oline->s + oline->len) && i < 3); + + if(cp >= oline->s + oline->len) { + LM_ERR("broken o= line - version field not found\n"); + return -1; + } len = cp - cp0 - 1; LM_DBG("end %d: >%.*s<\n", len, len, cp0);

1 0

[kamailio/kamailio] kex: add KDMQ to request statistic (PR #4197)
by vlitvinov 02 Apr '25

02 Apr '25

#### 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 KDMQ messages were added as a separate received request method in the statistic. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4197 -- Commit Summary -- * kex: add KDMQ to request statistic -- File Changes -- M src/modules/kex/core_stats.c (5) M src/modules/kex/core_stats.h (1) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4197.patch https://github.com/kamailio/kamailio/pull/4197.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4197 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4197(a)github.com>

2 3

git:master:7314fbcc: kex: add KDMQ to request statistic
by Victor Seva 02 Apr '25

02 Apr '25

Module: kamailio Branch: master Commit: 7314fbcc6fb7c211817d1660b98deaf9db1ea504 URL: https://github.com/kamailio/kamailio/commit/7314fbcc6fb7c211817d1660b98deaf… Author: vlitvinov <litvvitya92(a)gmail.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2025-04-02T07:42:38+02:00 kex: add KDMQ to request statistic --- Modified: src/modules/kex/core_stats.c Modified: src/modules/kex/core_stats.h --- Diff: https://github.com/kamailio/kamailio/commit/7314fbcc6fb7c211817d1660b98deaf… Patch: https://github.com/kamailio/kamailio/commit/7314fbcc6fb7c211817d1660b98deaf… --- diff --git a/src/modules/kex/core_stats.c b/src/modules/kex/core_stats.c index cded47ffc94..ac0c177bf83 100644 --- a/src/modules/kex/core_stats.c +++ b/src/modules/kex/core_stats.c @@ -70,6 +70,7 @@ stat_var *rcv_reqs_prack; stat_var *rcv_reqs_update; stat_var *rcv_reqs_refer; stat_var *rcv_reqs_publish; +stat_var *rcv_reqs_kdmq; /*! extended received replies */ stat_var *rcv_rpls_1xx; stat_var *rcv_rpls_18x; @@ -148,7 +149,8 @@ stat_export_t core_stats[] = {{"rcv_requests", 0, &rcv_reqs}, {"rcv_requests_update", 0, &rcv_reqs_update}, {"rcv_requests_refer", 0, &rcv_reqs_refer}, {"rcv_requests_publish", 0, &rcv_reqs_publish}, - {"rcv_replies", 0, &rcv_rpls}, {"rcv_replies_1xx", 0, &rcv_rpls_1xx}, + {"rcv_requests_kdmq", 0, &rcv_reqs_kdmq}, {"rcv_replies", 0, &rcv_rpls}, + {"rcv_replies_1xx", 0, &rcv_rpls_1xx}, {"rcv_replies_18x", 0, &rcv_rpls_18x}, {"rcv_replies_2xx", 0, &rcv_rpls_2xx}, {"rcv_replies_3xx", 0, &rcv_rpls_3xx}, @@ -259,6 +261,9 @@ static int km_cb_req_stats(struct sip_msg *msg, unsigned int flags, void *param) case METHOD_PUBLISH: update_stat(rcv_reqs_publish, 1); break; + case METHOD_KDMQ: + update_stat(rcv_reqs_kdmq, 1); + break; case METHOD_OTHER: update_stat(unsupported_methods, 1); break; diff --git a/src/modules/kex/core_stats.h b/src/modules/kex/core_stats.h index a1ee018ba18..433747473d0 100644 --- a/src/modules/kex/core_stats.h +++ b/src/modules/kex/core_stats.h @@ -54,6 +54,7 @@ extern stat_var *rcv_reqs_prack; extern stat_var *rcv_reqs_update; extern stat_var *rcv_reqs_refer; extern stat_var *rcv_reqs_publish; +extern stat_var *rcv_reqs_kdmq; /*! \brief received replies */ extern stat_var *rcv_rpls;

1 0

[kamailio/kamailio] tcp_timer_check_connections() logs on CRITICAL and is not caught by event_route[tcp:timeout] (Issue #3995)
by Fritjof Höst 02 Apr '25

02 Apr '25

### Description Hi! We are in the progress of upgrading from Kamailio 5.5 to 5.8. During our testing we have noticed a new error being reported from Kamailio. We don’t see any other errors following it. ```jsx /usr/sbin/kamailio[201]: CRITICAL: <core> [core/tcp_main.c:5544]: tcp_timer_check_connections(): message processing timeout on connection id: 67896 (state: 3) - closing ``` It does seem to be [new code](https://github.com/kamailio/kamailio/blob/master/src/core/tcp_main.c#… in Kamailio reporting this issue. Given that this is a fairly expected thing, cleaning up a connection which receives no traffic within the given time, is there a need for it to be reported on CRITICAL? I’d also expect it to be caught by ``` event_route[tcp:timeout] { xlog("L_INFO","connection $conid timeouts (unanswered keepalives)"); } ``` given that [the description](https://www.kamailio.org/docs/modules/stable/modules/tcpops.ht… of this one is `Called for connection timeouts (unanswered keepalives).`. ### Troubleshooting We don't have any way to reproduce it, we are still investigating it to figure out the cause. It happens around every 2 hours, so we think there might some some scheduled test or job running in our test system causing this. ### Additional Information * **Kamailio Version** - 5.8.3 * **Operating System**: Debian 12 AMI from the AWS Marketplace. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3995 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3995(a)github.com>

5 7

[kamailio/kamailio] tls: Create one shared context and reuse it. (PR #3972)
by Xenofon Karamanos 01 Apr '25

01 Apr '25

#### 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 - [x] Related to issue #3823 #### Description  This PR aims to implement what was discussed in [mailing list](https://lists.kamailio.org/mailman3/hyperkitty/list/sr-dev@lists.kama… regarding some `tls.reload` and increasing memory usage. It adds a new parameter `enable_shared_ctx` in `tls` module that if set to 0, preserves the old behavior and if set to 1 (other than 0 tbh), it creates a single SSL context that is being shared. This have the effect of using way less memory when initialized as well, but also minimizes (can't say it fixes the problem) the `tls.reload` memory increase. I have also added a small markdown (comparison.md) file, where some comparisons where made between enabled/disabled shared context and with/without CA file (where the initial problem was occurring by the reporter). Feedback would be necessary to verify whether this patch, acts as expected and kamailio works as intented. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3972 -- Commit Summary -- * tls: Add parameter for shared contexts * tls: Comparison for enable_shared_ctx -- File Changes -- A comparison.md (15) M src/modules/tls/tls_domain.c (172) M src/modules/tls/tls_mod.c (11) M src/modules/tls/tls_mod.h (1) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3972.patch https://github.com/kamailio/kamailio/pull/3972.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3972 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3972(a)github.com>

4 9

[kamailio/kamailio] core: local TCP socket is bound on listening address (PR #3925)
by sergey-safarov 01 Apr '25

01 Apr '25

#### 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: - [x] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description As a local IP address for TCP sending operation the Kamailio service is taking the same network_interface/IP_address, which is used by the service for TCP listening. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3925 -- Commit Summary -- * core: local TCP socket is bound on listening address -- File Changes -- M src/core/tcp_main.c (24) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3925.patch https://github.com/kamailio/kamailio/pull/3925.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3925 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3925(a)github.com>

5 14

[kamailio/kamailio] github: [skip ci]: bump awalsh128/cache-apt-pkgs-action from 1.4.3 to 1.5.0 (PR #4196)
by dependabot[bot] 31 Mar '25

31 Mar '25

Bumps [awalsh128/cache-apt-pkgs-action](https://github.com/awalsh128/cache-apt-pkg… from 1.4.3 to 1.5.0. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/awalsh128/cache-apt-pkgs-action/releases&quot… releases</a>.</em></p> <blockquote> <h2>v1.5.0</h2> <h2>What's Changed</h2> <ul> <li>fix: remove timestamps from loglines by <a href="https://github.com/F2210"><code>@F2210</code&g… in <a href="https://redirect.github.com/awalsh128/cache-apt-pkgs-action/pull… <li>Update actions/checkout versions in README by <a href="https://github.com/ulgens"><code>@ulgens</code… in <a href="https://redirect.github.com/awalsh128/cache-apt-pkgs-action/pull… <li>Avoid using xargs when creating package tarballs by <a href="https://github.com/bscott-zebra"><code>@bscott-ze… in <a href="https://redirect.github.com/awalsh128/cache-apt-pkgs-action/pull… <li>Add support for virtual packages and break out APT logic into common by <a href="https://github.com/awalsh128"><code>@awalsh128&lt… in <a href="https://redirect.github.com/awalsh128/cache-apt-pkgs-action/pull… </ul> <h2>New Contributors</h2> <ul> <li><a href="https://github.com/F2210"><code>@F2210</code&g… made their first contribution in <a href="https://redirect.github.com/awalsh128/cache-apt-pkgs-action/pull… <li><a href="https://github.com/ulgens"><code>@ulgens</code… made their first contribution in <a href="https://redirect.github.com/awalsh128/cache-apt-pkgs-action/pull… <li><a href="https://github.com/bscott-zebra"><code>@bscott-ze… made their first contribution in <a href="https://redirect.github.com/awalsh128/cache-apt-pkgs-action/pull… </ul> <p><strong>Full Changelog</strong>: <a href="https://github.com/awalsh128/cache-apt-pkgs-action/compare/v1.4.… </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/awalsh128/cache-apt-pkgs-action/commit/7ca5f4… Fix exec error.</li> <li><a href="https://github.com/awalsh128/cache-apt-pkgs-action/commit/2330cb… Add support for virtual packages and break out APT logic into common library....</li> <li><a href="https://github.com/awalsh128/cache-apt-pkgs-action/commit/128de2… Add support for virtual packages and break out APT logic into common library....</li> <li><a href="https://github.com/awalsh128/cache-apt-pkgs-action/commit/13d222… Avoid using xargs when creating package tarballs (<a href="https://redirect.github.com/awalsh128/cache-apt-pkgs-action/issu… <li><a href="https://github.com/awalsh128/cache-apt-pkgs-action/commit/d1a184… Update actions/checkout versions in README (<a href="https://redirect.github.com/awalsh128/cache-apt-pkgs-action/issu… <li><a href="https://github.com/awalsh128/cache-apt-pkgs-action/commit/dfe9c8… fix: remove timestamps from loglines (<a href="https://redirect.github.com/awalsh128/cache-apt-pkgs-action/issu… <li>See full diff in <a href="https://github.com/awalsh128/cache-apt-pkgs-action/compare/v1.4.… view</a></li> </ul> </details> <br /> [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?d… Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) </details> You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4196 -- Commit Summary -- * github: [skip ci]: bump awalsh128/cache-apt-pkgs-action -- File Changes -- M .github/workflows/pull_request.yml (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4196.patch https://github.com/kamailio/kamailio/pull/4196.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4196 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4196(a)github.com>

2 1

git:master:f05e6e11: github: [skip ci]: bump awalsh128/cache-apt-pkgs-action
by Victor Seva 31 Mar '25

31 Mar '25

Module: kamailio Branch: master Commit: f05e6e11dcb13f971465db691b1b947bbdc13c34 URL: https://github.com/kamailio/kamailio/commit/f05e6e11dcb13f971465db691b1b947… Author: dependabot[bot] <49699333+dependabot[bot](a)users.noreply.github.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2025-04-01T00:45:47+02:00 github: [skip ci]: bump awalsh128/cache-apt-pkgs-action Bumps [awalsh128/cache-apt-pkgs-action](https://github.com/awalsh128/cache-apt-pkg… from 1.4.3 to 1.5.0. - [Release notes](https://github.com/awalsh128/cache-apt-pkgs-action/releases) - [Commits](https://github.com/awalsh128/cache-apt-pkgs-action/compare/v1.4.3.… --- updated-dependencies: - dependency-name: awalsh128/cache-apt-pkgs-action dependency-version: 1.5.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support(a)github.com> --- Modified: .github/workflows/pull_request.yml --- Diff: https://github.com/kamailio/kamailio/commit/f05e6e11dcb13f971465db691b1b947… Patch: https://github.com/kamailio/kamailio/commit/f05e6e11dcb13f971465db691b1b947… --- diff --git a/.github/workflows/pull_request.yml b/.github/workflows/pull_request.yml index 5be003dcea8..96f3ac6b864 100644 --- a/.github/workflows/pull_request.yml +++ b/.github/workflows/pull_request.yml @@ -27,7 +27,7 @@ jobs: uses: actions/checkout@v4 with: ref: ${{ github.event.pull_request.head.sha }} - - uses: awalsh128/cache-apt-pkgs-action(a)v1.4.3 + - uses: awalsh128/cache-apt-pkgs-action(a)v1.5.0 with: packages: cmake-format - name: Run scripts/check-cmake-format.sh

1 0

[kamailio/kamailio] generate rpm packages using github actions (PR #4194)
by sergey-safarov 31 Mar '25

31 Mar '25

#### 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: - [x] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description I have refactored the rpm packaging and am now using the GitHub actions. Now `tar.gz` archive with rpm packages, source rpm and deps rpm files attached to the release message like. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4194 -- Commit Summary -- * generate rpm packages using github actions -- File Changes -- A .github/workflows/rpm.yml (57) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4194.patch https://github.com/kamailio/kamailio/pull/4194.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4194 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4194(a)github.com>

2 2

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev