- tls_list() add PROTO_WSS to TLS_LIST RPC call to include WSS connections in tls.list
- tls_kill() add PROTO_WSS to handle WSS connections
Co-authored-by: Andreas Tarp <tarp(a)sipgate.de>
#### 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:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue #4167
#### Description
WSS connection is not shown in tls.list RPC command like all other TLS connections.
Also tls_kill does not handle WSS connections.
This is related to the issue 4167.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4259
-- Commit Summary --
* http_client: add information about parameter loading
* http_client: docs - typos from previous commit
* tls: add WSS to RPC funtions
-- File Changes --
M src/modules/http_client/doc/http_client_admin.xml (7)
M src/modules/tls/tls_rpc.c (4)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4259.patchhttps://github.com/kamailio/kamailio/pull/4259.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4259
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4259(a)github.com>
<!-- Kamailio Pull Request Template -->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [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:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [x] PR should be backported to stable branches
- [x] Tested changes locally
#### Description
sca was generating a crash since it is trying to shm_free a part of a bigger block at:
https://github.com/kamailio/kamailio/blob/master/src/modules/sca/sca_subscr…
So, let's reserve memory that field separately.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4251
-- Commit Summary --
* sca: reserve subscription.rr.s separately since it could be updated
-- File Changes --
M src/modules/sca/sca_subscribe.c (24)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4251.patchhttps://github.com/kamailio/kamailio/pull/4251.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4251
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4251(a)github.com>
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [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:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [x] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
This adds RPC command to view and/or change the timeout value for PDB queries. The documentation is also updated accordingly.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4260
-- Commit Summary --
* pdb: add new RPC command to view/change PDB query timeout
* pdb: add documentation for new RPC command to view/change PDB query timeout
* pdb: add new code contributor credits
-- File Changes --
M src/modules/pdb/doc/pdb.xml (6)
M src/modules/pdb/doc/pdb_admin.xml (18)
M src/modules/pdb/pdb.c (34)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4260.patchhttps://github.com/kamailio/kamailio/pull/4260.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4260
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4260(a)github.com>
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [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:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [x] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
This adds RPC command to view and/or change the timeout value for PDB queries. The documentation is also updated accordingly.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4254
-- Commit Summary --
* pdb: added rpc command to view/update the pdb query timeout
* pdb: add documentation for rpc command to view/update the pdb query timeout
-- File Changes --
M src/modules/pdb/doc/pdb.xml (55)
M src/modules/pdb/doc/pdb_admin.xml (334)
M src/modules/pdb/pdb.c (49)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4254.patchhttps://github.com/kamailio/kamailio/pull/4254.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4254
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4254(a)github.com>
- [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)
- [ ] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
Added missing kemi exports for RTP Engine module play_dtmf event_callback functions and Dialog module dlg_req_within function
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4245
-- Commit Summary --
* Add support for playing DTMF tones in RTP engine module using Kemi
* Add support for KEMI callbacks in RTP engine events
* Add `ki_dlg_req_within4` handling for dialog requests
-- File Changes --
M src/modules/dialog/dialog.c (24)
M src/modules/rtpengine/rtpengine.c (215)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4245.patchhttps://github.com/kamailio/kamailio/pull/4245.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4245
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4245(a)github.com>
Module: kamailio
Branch: master
Commit: df8e38958a3fa0a2e356bec727bb8d7ce19bec43
URL: https://github.com/kamailio/kamailio/commit/df8e38958a3fa0a2e356bec727bb8d7…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2025-05-27T08:58:17+02:00
secfilter: remove unnecessary check on clean up
---
Modified: src/modules/secfilter/secfilter_db.c
---
Diff: https://github.com/kamailio/kamailio/commit/df8e38958a3fa0a2e356bec727bb8d7…
Patch: https://github.com/kamailio/kamailio/commit/df8e38958a3fa0a2e356bec727bb8d7…
---
diff --git a/src/modules/secfilter/secfilter_db.c b/src/modules/secfilter/secfilter_db.c
index 1c0138b0193..24884b231c9 100644
--- a/src/modules/secfilter/secfilter_db.c
+++ b/src/modules/secfilter/secfilter_db.c
@@ -185,9 +185,9 @@ int secf_append_rule(int action, int type, str *value)
/**
* @brief Removes entries from a whitelist or blacklist based on action, type, and value.
*
- * The function validates the action and type, locates the relevant list, and iterates
- * through its nodes to find and remove all matches for the given value. If matches are found,
- * the corresponding memory is freed, and the list pointers are updated. Logs are generated
+ * The function validates the action and type, locates the relevant list, and iterates
+ * through its nodes to find and remove all matches for the given value. If matches are found,
+ * the corresponding memory is freed, and the list pointers are updated. Logs are generated
* for each removal and at the end of the process to summarize the result.
* @param action: Specifies the target list (0 = blacklist, 1 = whitelist, 2 = destination blacklist).
* @param type: Indicates the type of rule to be removed (e.g., domain, IP, user, etc.).
@@ -358,10 +358,8 @@ int secf_load_db(void)
lock_release(&(*secf_data)->lock);
clean:
- if(db_res) {
- if(db_funcs.free_result(db_handle, db_res) < 0) {
- LM_DBG("Failed to free the result\n");
- }
+ if(db_funcs.free_result(db_handle, db_res) < 0) {
+ LM_DBG("Failed to free the result\n");
}
db_funcs.close(db_handle);
return res;
sergey-safarov created an issue (kamailio/kamailio#4241)
### Description
In the GitHub Actions present Alpine Docker image build error
```
tar (child): kamailio_min-without_os_files.tar.gz: Cannot open: No such file or directory
tar (child): Error is not recoverable: exiting now
tar: Child returned status 2
tar: Error is not recoverable: exiting now
```
https://github.com/kamailio/kamailio/actions/runs/14722992691/job/413201110…
With high probability, this does not work
I will work to fix this, but if possible, it will be fine to switch to use `cmake` for Alpine Docker image.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/4241
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/4241(a)github.com>
sergey-safarov created an issue (kamailio/kamailio#4255)
### Description
I now make testing `cmake` packaging and see this error
```
RPM build errors:
Directory not found: /root/rpmbuild/BUILDROOT/kamailio-6.1.0-dev1.0.el10.centos.x86_64/usr/share/kamailio/postgres
File not found: /root/rpmbuild/BUILDROOT/kamailio-6.1.0-dev1.0.el10.centos.x86_64/usr/share/kamailio/postgres/*
```
### Troubleshooting
#### Reproduction
I use this module list for build
https://github.com/sergey-safarov/kamailio/blob/master/pkg/kamailio/obs/kam…
@xkaraman could you look?
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/4255
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/4255(a)github.com>
Module: kamailio
Branch: master
Commit: c7fcdfdb13aeec9aeae726a9cc1ccaeda557e7bb
URL: https://github.com/kamailio/kamailio/commit/c7fcdfdb13aeec9aeae726a9cc1ccae…
Author: Xenofon Karamanos <xk(a)gilawa.com>
Committer: Xenofon Karamanos <xk(a)gilawa.com>
Date: 2025-05-23T11:10:13Z
cmake: Add some check for finding group name if using non K-variants of group names
---
Modified: cmake/groups.cmake
---
Diff: https://github.com/kamailio/kamailio/commit/c7fcdfdb13aeec9aeae726a9cc1ccae…
Patch: https://github.com/kamailio/kamailio/commit/c7fcdfdb13aeec9aeae726a9cc1ccae…
---
diff --git a/cmake/groups.cmake b/cmake/groups.cmake
index a41560f17fc..c629ee49f93 100644
--- a/cmake/groups.cmake
+++ b/cmake/groups.cmake
@@ -764,6 +764,20 @@ function(find_group_name module)
""
PARENT_SCOPE
)
+ # This was need due to the dbschema.cmake
+ # If one select one of these option as group
+ # we want the group name to match instead of the actual group it belongs to
+ if(MODULE_GROUP_NAME STREQUAL "ALL"
+ OR MODULE_GROUP_NAME STREQUAL "DEFAULT"
+ OR MODULE_GROUP_NAME STREQUAL "STANDARD"
+ OR MODULE_GROUP_NAME STREQUAL "COMMON"
+ )
+ set(group_name
+ "${MODULE_GROUP_NAME}"
+ PARENT_SCOPE
+ )
+ return()
+ endif()
# message(WARNING "groups to search in" ${MODULE_GROUP_PACKAGE_GROUPS})
# Get all variable names in the current CMake context
foreach(group IN LISTS MODULE_GROUP_PACKAGE_GROUPS)
@@ -776,6 +790,8 @@ function(find_group_name module)
"${group}"
PARENT_SCOPE
)
+ return()
endif()
endforeach()
+ message((STATUS "module ${module} not found in any group"))
endfunction()
Module: kamailio
Branch: master
Commit: 0fb182720428b276793899debb210279a0b5ec97
URL: https://github.com/kamailio/kamailio/commit/0fb182720428b276793899debb21027…
Author: Xenofon Karamanos <xk(a)gilawa.com>
Committer: Xenofon Karamanos <xk(a)gilawa.com>
Date: 2025-05-23T11:10:13Z
cmake: Add missing package KSTANDARD
- When `ALL_PACKAGED` is used, KSTANDARD should be built as well, but not KMINI which is a subset of it.
- Fix RTP_MEDIA group name
---
Modified: cmake/groups.cmake
---
Diff: https://github.com/kamailio/kamailio/commit/0fb182720428b276793899debb21027…
Patch: https://github.com/kamailio/kamailio/commit/0fb182720428b276793899debb21027…
---
diff --git a/cmake/groups.cmake b/cmake/groups.cmake
index 90c47413d22..a41560f17fc 100644
--- a/cmake/groups.cmake
+++ b/cmake/groups.cmake
@@ -688,6 +688,7 @@ set(MODULE_GROUP_KRTP_MEDIA_SERVER ${MOD_LIST_RTP_MEDIA_SERVER})
set(STATIC_MODULES "")
set(MODULE_GROUP_PACKAGE_GROUPS
+ KSTANDARD
KPCRE
KMYSQL
KPOSTGRES
@@ -748,7 +749,7 @@ set(MODULE_GROUP_PACKAGE_GROUPS
KNGHTTP2
KGCRYPT
KSECSIPID
- KRT_MEDIA_SERVER
+ KRTP_MEDIA_SERVER
)
# Add group names to available group and provide "ALL_PACKAGED" as well
Hello,
I have noticed that the git tag for release 5.7.7 was missing, I've just set it.
The 5.7.7-bullseye docker image is also missing from https://github.com/kamailio/kamailio-docker/pkgs/container/kamailio
Not sure if this is done automatically, if not - please can somebody please trigger the workflow to create it?
Thank you,
Henning