[sr-dev] git:master:672b72d6: ims_qos: spelling fix for suspend_transaction docs, more sensible value for example docs

2023-10-16 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 672b72d65aba0b30fb363d60ab648f9705810ba7 URL: https://github.com/kamailio/kamailio/commit/672b72d65aba0b30fb363d60ab648f9705810ba7 Author: Henning Westerholt Committer: Henning Westerholt Date: 2023-10-16T13:36:42Z ims_qos: spelling fix for

[sr-dev] Re: [kamailio/kamailio] cdp: Add and apply cdp_openssl_clear_errors function (PR #3612)

2023-10-24 Thread Henning Westerholt via sr-dev
Merged #3612 into master. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3612#event-10757588772 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] Re: [kamailio/kamailio] cdp: Add and apply cdp_openssl_clear_errors function (PR #3612)

2023-10-24 Thread Henning Westerholt via sr-dev
Thanks, merged -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3612#issuecomment-161242 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) - Development

[sr-dev] Re: [kamailio/kamailio] cdp: Fix for undefined symbols when using older/unsupported OpenSSL (PR #3616)

2023-10-25 Thread Henning Westerholt via sr-dev
Thanks. The PR #3601 was merged already. I did looked to this change, but did not spotted why the error clearing added yesterday caused additional problems here? -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3616#issuecomment-1778649178 You are

[sr-dev] Re: [kamailio/kamailio] cdp: Fix for undefined symbols when using older/unsupported OpenSSL (PR #3616)

2023-10-25 Thread Henning Westerholt via sr-dev
Thanks for the clarification. Lets wait until the CI tests are done and then it will be merged, if there are no more comments. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3616#issuecomment-1778682246 You are receiving this because you are

[sr-dev] Re: Kamailio module http_client - setting timeout in ms - new parameter ?

2023-10-17 Thread Henning Westerholt via sr-dev
Hello Nicolas, sounds good, an additional parameter while keeping the existing functionality for the existing timeout would work. Just create a pull request, then developers can review it and maybe some smaller adjustments could be done before a merge. Thanks, Henning -- Henning Westerholt

[sr-dev] Re: Change in mailing list configuration related to DMARC spam filtering

2023-09-22 Thread Henning Westerholt via sr-dev
Hello, there has been no feedback regarding this change, so the setting was done some minutes ago. In case of unforeseen issues, just follow up on the sr-users lists for further discussions. Thanks, Henning -- Henning Westerholt - https://skalatan.de/blog/ Kamailio services -

[sr-dev] Re: [kamailio/kamailio] kamctl: Source only one kamctlrc (PR #3594)

2023-10-06 Thread Henning Westerholt via sr-dev
Thank you -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3594#issuecomment-1750064416 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) - Development

[sr-dev] Re: [kamailio/kamailio] kamctl: Source only one kamctlrc (PR #3594)

2023-10-05 Thread Henning Westerholt via sr-dev
Many linux software packages uses a different behaviour, if one configuration file is found, its using this and no merge of the values is done. Bash profile loading does it differently indeed, but I would argue that kamctl is more like a tool. But if people depend on this current behaviour,

[sr-dev] git:master:d2f0e1c3: topos: spelling fix in docs

2023-10-18 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: d2f0e1c31920e5b2b38ddbc823c6bc332fac86a5 URL: https://github.com/kamailio/kamailio/commit/d2f0e1c31920e5b2b38ddbc823c6bc332fac86a5 Author: Henning Westerholt Committer: Henning Westerholt Date: 2023-10-18T07:55:59Z topos: spelling fix in docs ---

[sr-dev] Re: [kamailio/kamailio] tls: Add and apply tls_openssl_clear_errors function. (PR #3607)

2023-10-19 Thread Henning Westerholt via sr-dev
I will later merge it manually -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3607#issuecomment-1770265188 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio

[sr-dev] git:master:110ebbaf: tls: Add and apply tls_openssl_clear_errors function

2023-10-19 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 110ebbafadcc225f4e88749287f06ae29a6cfa2e URL: https://github.com/kamailio/kamailio/commit/110ebbafadcc225f4e88749287f06ae29a6cfa2e Author: Xenofon Karamanos Committer: Henning Westerholt Date: 2023-10-19T12:18:38Z tls: Add and apply

[sr-dev] Re: [kamailio/kamailio] tls: Add and apply tls_openssl_clear_errors function. (PR #3607)

2023-10-19 Thread Henning Westerholt via sr-dev
Thanks, was merged manually, close it -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3607#issuecomment-1770847826 You are receiving this because you are subscribed to this thread. Message ID: ___

[sr-dev] Re: [kamailio/kamailio] tls: Add and apply tls_openssl_clear_errors function. (PR #3607)

2023-10-19 Thread Henning Westerholt via sr-dev
Closed #3607. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3607#event-10710873842 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) - Development Mailing

[sr-dev] Re: Kamailio module http_client - setting timeout in ms - new parameter ?

2023-10-19 Thread Henning Westerholt via sr-dev
Hello, thank you, will take a look! Regarding backporting it to stable branch, this will be unfortunately not possible as it's a new functionality. Generally speaking, its not necessary to create additional PRs for stable branches, this is usually done with cherry-picking. Cheers, Henning --

[sr-dev] git:master:7731fc3a: tls: add log line-break, increase buffer size (related to GH 3612 and GH 3607)

2023-10-24 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 7731fc3a1d793ffc7204e5c042a8bfc38d97e691 URL: https://github.com/kamailio/kamailio/commit/7731fc3a1d793ffc7204e5c042a8bfc38d97e691 Author: Henning Westerholt Committer: Henning Westerholt Date: 2023-10-24T11:13:36Z tls: add log line-break, increase

[sr-dev] git:master:2b966644: kamailio.cfg: use xalert instead of old format for xlog, similar as done in other cfg places

2023-11-10 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 2b9666445a9fd31f75bf86b9924658daffb7ac85 URL: https://github.com/kamailio/kamailio/commit/2b9666445a9fd31f75bf86b9924658daffb7ac85 Author: Henning Westerholt Committer: Henning Westerholt Date: 2023-11-10T14:01:47Z kamailio.cfg: use xalert instead of

[sr-dev] Re: [kamailio/kamailio] janssonrpcc: No failover to lower prio server (Issue #3378)

2023-11-13 Thread Henning Westerholt via sr-dev
For the record, the bot aparently removes the stale flag automatically now when there is activity. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3378#issuecomment-1808149045 You are receiving this because you are subscribed to this thread.

[sr-dev] Re: [kamailio/kamailio] pua_json module not compatible with library libjson-c-dev 0.15.2 (Issue #3376)

2023-11-13 Thread Henning Westerholt via sr-dev
@eschmidbauer anything I could do? Did you managed to have a look to it maybe? Thank you. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3376#issuecomment-1808203915 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] Re: [kamailio/kamailio] dialplan: implement Iterative dialplan matching (#764)

2023-11-13 Thread Henning Westerholt via sr-dev
Last activity was here in 2018, close this one, can be re-opened if people want to continue work on it. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/764#issuecomment-1808213719 You are receiving this because you are subscribed to this thread.

[sr-dev] Re: [kamailio/kamailio] dialplan: implement Iterative dialplan matching (#764)

2023-11-13 Thread Henning Westerholt via sr-dev
Closed #764 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/764#event-10942110162 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] Re: [kamailio/kamailio] topoh: uses socket IP when no mask_ip is defined (PR #3341)

2023-11-13 Thread Henning Westerholt via sr-dev
@henningw commented on this pull request. The stale flag got now removed from the bot. I think this is a useful addition, even if it changes the default behaviour of the module. For people want the previous behavior back, they can easily configure 127.0.0.1 manually. I did a quick review, just

[sr-dev] Re: [kamailio/kamailio] pua_json module not compatible with library libjson-c-dev 0.15.2 (Issue #3376)

2023-11-13 Thread Henning Westerholt via sr-dev
Thanks for the fast reply. We observed some time ago on an environment, but right now I don't have the means to reproduce it, unfortunately. I just looked into the report from the time, and it was easy to reproduce by just installing e.g. 5.6.3 from github and using the libjson-c-dev version

[sr-dev] Re: [kamailio/kamailio] branch_failure route in case 503 and dns failover (PR #3138)

2023-11-13 Thread Henning Westerholt via sr-dev
Thank you for providing the update. Right now there seems to be some conflicts probably after some refactoring changes that have been done. Could you maybe re-base your changes and then developers could have another close look. -- Reply to this email directly or view it on GitHub:

[sr-dev] Re: [kamailio/kamailio] tcp_send_timeout not used on reused socket (Issue #3443)

2023-11-13 Thread Henning Westerholt via sr-dev
@schoberw your pull request got merged already some time ago. Is there something missing in this issue? Otherwise it should be closed. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3443#issuecomment-1808139951 You are receiving this because

[sr-dev] Re: [kamailio/kamailio] core: Add TCP_USER_TIMEOUT socket option on listening socket. (PR #3528)

2023-11-13 Thread Henning Westerholt via sr-dev
I think it qualifies as bug fix, so it should be also included in the stable branches. Why it was not included in 5.7.2 I don't know, it might be that the patch was not easy to backport. The stable branches has been now also adapted to the same clang format to make future backports way easier.

[sr-dev] Re: [kamailio/kamailio] Lost selectors (#2705)

2023-11-13 Thread Henning Westerholt via sr-dev
@sergey-safarov the PR is still in draft status, not sure about the status. If it will be not completed for some reasons, just let us know. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/2705#issuecomment-1808159252 You are receiving this because

[sr-dev] Re: [kamailio/kamailio] kamailio 5.5 (commit 54c9df) SIGSEGV on run_dlg_callbacks (Issue #3106)

2023-11-13 Thread Henning Westerholt via sr-dev
Closed #3106 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3106#event-10941872309 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] Re: [kamailio/kamailio] kamailio 5.5 (commit 54c9df) SIGSEGV on run_dlg_callbacks (Issue #3106)

2023-11-13 Thread Henning Westerholt via sr-dev
The version 5.5. is now end of life. So if you only observed it once, you probably can try to update to 5.6.x or 5.7.x and see if it still happens. Close this issue for now. -- Reply to this email directly or view it on GitHub:

[sr-dev] Re: [Kamailio-Business] Roadmap to next major Kamailio release series v5.8.x

2024-01-23 Thread Henning Westerholt via sr-dev
Hello Daniel, we are working on a new module right now (related to special logging for high CPS setups), which would be nice to integrate if possible. Its isolated, so would not touch any other module and/or the core. There is also another core refactoring in progress (to replace the atomic

[sr-dev] git:master:3120d1df: ldap: option to start even when connecting to server fails

2024-02-09 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 3120d1dfcfa393b90272526e6fe2efb3aa32a19d URL: https://github.com/kamailio/kamailio/commit/3120d1dfcfa393b90272526e6fe2efb3aa32a19d Author: Xenofon Karamanos Committer: Henning Westerholt Date: 2024-02-09T09:46:25+01:00 ldap: option to start even when

[sr-dev] git:master:e7397acb: ldap/docs: Add connect_mode docs

2024-02-09 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: e7397acb86db5cf0a214fe7f2812e21130e7b2f6 URL: https://github.com/kamailio/kamailio/commit/e7397acb86db5cf0a214fe7f2812e21130e7b2f6 Author: Xenofon Karamanos Committer: Henning Westerholt Date: 2024-02-09T09:46:25+01:00 ldap/docs: Add connect_mode docs

[sr-dev] Re: [kamailio/kamailio] ldap: option to start even when connecting to ldap server fails (PR #3747)

2024-02-09 Thread Henning Westerholt via sr-dev
Merged #3747 into master. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3747#event-11755284442 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] Re: [kamailio/kamailio] ldap: option to start even when connecting to ldap server fails (PR #3747)

2024-02-09 Thread Henning Westerholt via sr-dev
Thanks for the feedback, merged -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3747#issuecomment-1935535310 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio

[sr-dev] Re: [kamailio/kamailio] DRAFT: new global config var to constrain thread wrapping (PR #3745)

2024-02-09 Thread Henning Westerholt via sr-dev
@space88man thanks for the work on this topic. It is great that there is then a way from version 5.7.4 and forward to restore the previous behaviour, especially in the stable branch. -- Reply to this email directly or view it on GitHub:

[sr-dev] Re: git:master:ba921b21: core/rthread.h: add prototype for db queries

2024-02-09 Thread Henning Westerholt via sr-dev
Hello Richard, thanks for working on this topic and providing a way to control this new protection mechanism depending on the requirements and age of systems etc.. Cheers, Henning From: Richard Chan via sr-dev Sent: Dienstag, 6. Februar 2024 12:20 To: mico...@gmail.com Cc: Kamailio (SER) -

[sr-dev] Re: Fwd: Reject TCP SYN

2024-02-13 Thread Henning Westerholt via sr-dev
Hello David, so far nobody responded to this e-mail. Maybe you can make a pull request from it, just to not loose it. Then developers could review and comment on it. Cheers, Henning From: David Villasmil via sr-dev Sent: Sonntag, 11. Februar 2024 12:55 To: Kamailio (SER) - Development

[sr-dev] git:master:22581f95: file_out: Free allocated memory

2024-02-19 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 22581f95690415681ea233c97923137e46fb55ae URL: https://github.com/kamailio/kamailio/commit/22581f95690415681ea233c97923137e46fb55ae Author: Xenofon Karamanos Committer: Henning Westerholt Date: 2024-02-19T21:28:19+01:00 file_out: Free allocated memory

[sr-dev] git:master:a722d4cb: file_out: Add a prefix paramater

2024-02-19 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: a722d4cba1ab18f481bfe8353cebc0c6fa1a7ebb URL: https://github.com/kamailio/kamailio/commit/a722d4cba1ab18f481bfe8353cebc0c6fa1a7ebb Author: Xenofon Karamanos Committer: Henning Westerholt Date: 2024-02-19T21:28:19+01:00 file_out: Add a prefix paramater

[sr-dev] Re: [kamailio/kamailio] file_out: Add prefix parameter and docs and fix leaks. (PR #3758)

2024-02-19 Thread Henning Westerholt via sr-dev
Merged #3758 into master. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3758#event-11853989687 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] Re: [kamailio/kamailio] file_out: Add prefix parameter and docs and fix leaks. (PR #3758)

2024-02-19 Thread Henning Westerholt via sr-dev
Thanks, merged. I will make a quick re-formatting of the readme example, to be consistent with other parts. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3758#issuecomment-1953122751 You are receiving this because you are subscribed to this

[sr-dev] [kamailio/kamailio] performance regression related to new TCP connection tracking feature with a high number of connections (Issue #3759)

2024-02-19 Thread Henning Westerholt via sr-dev
### Description On systems with a high number of TCP sessions there can be a significant performance regression observed, probably related to the newly added TCP connection tracking feature. ### Troubleshooting Reproduction No special configuration is necessary, just install the latest

[sr-dev] Re: [kamailio/kamailio] performance regression related to new TCP connection tracking feature with a high number of connections (Issue #3759)

2024-02-19 Thread Henning Westerholt via sr-dev
SVG version of flamegraph: https://github.com/kamailio/kamailio/assets/6481937/02aa91c9-fa72-4e7e-9bcd-732cdca454cb -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3759#issuecomment-1953174460 You are receiving this because you are subscribed

[sr-dev] Re: [kamailio/kamailio] performance regression related to new TCP connection tracking feature with a high number of connections (Issue #3759)

2024-02-19 Thread Henning Westerholt via sr-dev
PDF version of flamegraph: https://github.com/kamailio/kamailio/files/14336952/graph.pdf -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3759#issuecomment-1953173858 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] git:master:dad7f3ff: file_out: small extensions to the module docs

2024-02-19 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: dad7f3ff3af02d086c1cb3f915f1502bca127d58 URL: https://github.com/kamailio/kamailio/commit/dad7f3ff3af02d086c1cb3f915f1502bca127d58 Author: Henning Westerholt Committer: Henning Westerholt Date: 2024-02-19T20:30:45Z file_out: small extensions to the

[sr-dev] rpm kamailio git master version

2024-02-20 Thread Henning Westerholt via sr-dev
Hello, just a question about the https://rpm.kamailio.org/ repository, especially regarding git master builds. It seems that the git master build there was updated last autumn. It would be great if it could be updated, now as we want to test for the coming release. Thank you, Henning

[sr-dev] Re: [kamailio/kamailio] performance regression related to new TCP connection tracking feature with a high number of connections (Issue #3759)

2024-02-21 Thread Henning Westerholt via sr-dev
Thank you for the commit, will have it tested soon. Regarding the performance regression question, one one system where we observed it there are about 35.000 current TCP connections and a new connection rate about 85 connections per second. -- Reply to this email directly or view it on

[sr-dev] Re: [kamailio/kamailio] Dispatcher: added two new flags to mode parameter of ds_is_from_list function for more strictly matching (PR #3699)

2024-02-21 Thread Henning Westerholt via sr-dev
@Den4t @miconda Any more questions or things that needs to be discussed? Otherwise its probably a good idea to merge it in the next days before the window for 5.8.0 closes completely. -- Reply to this email directly or view it on GitHub:

[sr-dev] Re: [kamailio/kamailio] ldap: option to start even when connecting to ldap server fails (PR #3747)

2024-02-07 Thread Henning Westerholt via sr-dev
Its not strictly a bug fix, but without this change the ldap module (and kamailio) will not start when the ldap server is not available, leading to an unavailable service. The module already has logic in place to deal with the reconnect case, but the startup was missing. Therefore it would be

[sr-dev] Re: [kamailio/kamailio] topos: add compatiblity for early in-dialog NOTIFY (PR #3627)

2024-02-07 Thread Henning Westerholt via sr-dev
@mauritsderuiter95 Thanks for the feedback. So this PR was not merged to master in the proposed form, but Daniel pushed a bit different commits if I read it correctly. So if the current master is ok from your point of view, the PR should be closed. -- Reply to this email directly or view it

[sr-dev] Re: Working with arrays in MongoDB documents

2024-02-07 Thread Henning Westerholt via sr-dev
Hello Mario, if you want to use standard modules like usrloc and registrar, you need to use the db_mongodb module. This is naturally limited in its support of the mongodb specialities. If you want to get better support, you can of course use the ndb_mongodb module. You can find details about

[sr-dev] Re: [kamailio/kamailio] [pua] reginfo_increase_version does nothing (Issue #3234)

2024-02-09 Thread Henning Westerholt via sr-dev
Thanks for the feedback, close it then. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3234#issuecomment-1936227046 You are receiving this because you are subscribed to this thread. Message ID: ___

[sr-dev] Re: [kamailio/kamailio] [pua] reginfo_increase_version does nothing (Issue #3234)

2024-02-09 Thread Henning Westerholt via sr-dev
Closed #3234 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3234#event-11760099761 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] git:master:4c939854: pua: fix possible copy/paste error, we should use reginfo_increase_version for reginfo events (GH #3234)

2023-12-21 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 4c93985437122706bc68143b2f96a9d7f35be16d URL: https://github.com/kamailio/kamailio/commit/4c93985437122706bc68143b2f96a9d7f35be16d Author: Henning Westerholt Committer: Henning Westerholt Date: 2023-12-21T16:10:25Z pua: fix possible copy/paste error, we

[sr-dev] Re: [kamailio/kamailio] topos: add compatiblity for early in-dialog NOTIFY (PR #3627)

2024-01-03 Thread Henning Westerholt via sr-dev
@mauritsderuiter95 Did you managed to have another look to this topic regarding the test of git master? -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3627#issuecomment-1874988230 You are receiving this because you are subscribed to this thread.

[sr-dev] Re: [kamailio/kamailio] dmq_usrloc not replicating contact attributes (#1968)

2024-01-03 Thread Henning Westerholt via sr-dev
Should be fixed be (not yet merged) PR #3679 -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1968#issuecomment-1874986540 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] Re: [kamailio/kamailio] dmq_usrloc: DMQ also copies and sends attributes related to contacts (PR #3679)

2024-01-05 Thread Henning Westerholt via sr-dev
Are there more comments or objections? If not I would merge it in the next days. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3679#issuecomment-1878352646 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] Re: [kamailio/kamailio] dmq_usrloc: DMQ also copies and sends attributes related to contacts (PR #3679)

2024-01-05 Thread Henning Westerholt via sr-dev
Thanks, yes this was basically the question also @xkaraman had earlier. The XAVPs will be used in the cfg script context later on I think. He will do another review. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3679#issuecomment-1878409382 You

[sr-dev] Re: [kamailio/kamailio] http_async_client: exit mod_init if tm is not loaded (PR #3706)

2024-01-10 Thread Henning Westerholt via sr-dev
Thanks for the PR, merged -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3706#issuecomment-1884446598 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] git:master:5a40d1ef: http_async_client: exit mod_init if tm is not loaded

2024-01-10 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 5a40d1ef1b484c4fc437a3f0183a101241e80313 URL: https://github.com/kamailio/kamailio/commit/5a40d1ef1b484c4fc437a3f0183a101241e80313 Author: Federico Cabiddu Committer: Henning Westerholt Date: 2024-01-10T10:06:05+01:00 http_async_client: exit mod_init if

[sr-dev] Re: [kamailio/kamailio] http_async_client: exit mod_init if tm is not loaded (PR #3706)

2024-01-10 Thread Henning Westerholt via sr-dev
Merged #3706 into master. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3706#event-11440393557 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] Re: [kamailio/kamailio] http_async_client: exit mod_init if tm is not loaded (PR #3706)

2024-01-11 Thread Henning Westerholt via sr-dev
@dilyanpalauzov You are right, done in the commit above. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3706#issuecomment-1886815392 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] git:master:72595fe4: http_async_client: use error log level (GH #3706)

2024-01-11 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 72595fe42d28d3e4a12341b405af63d41f827dc4 URL: https://github.com/kamailio/kamailio/commit/72595fe42d28d3e4a12341b405af63d41f827dc4 Author: Henning Westerholt Committer: Henning Westerholt Date: 2024-01-11T10:30:40Z http_async_client: use error log level

[sr-dev] Re: [kamailio/kamailio] rtpengine: Add support for codec-accept and codec-consume flags (PR #3710)

2024-01-11 Thread Henning Westerholt via sr-dev
Unfortunately it does not seems to work at least with "codec-accept=XXX": kam02 rtpengine[570041]: WARNING: [jKOBctrRd-]: [core] Unknown dictionary key encountered: 'codec-accept' -- Reply to this email directly or view it on GitHub:

[sr-dev] Re: [kamailio/kamailio] rtpengine: Add support for codec-accept and codec-consume flags (PR #3710)

2024-01-11 Thread Henning Westerholt via sr-dev
Thanks, if there is a workaround there its of course not necessary to backport. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3710#issuecomment-1887564694 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] git:master:555530c7: doc/tutorial: remove unused define WITH_XAVP, its not used anymore from the code

2024-01-15 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 30c70f8f7034046e0b81f4a8469ed76bb405 URL: https://github.com/kamailio/kamailio/commit/30c70f8f7034046e0b81f4a8469ed76bb405 Author: Henning Westerholt Committer: Henning Westerholt Date: 2024-01-15T12:06:54Z doc/tutorial: remove unused define

[sr-dev] git:master:6f962070: docs/scripts: remove unused define WITH_XAVP, its not used anymore from the code

2024-01-15 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 6f9620704c661151415ad723e5f0b31f34b70c1d URL: https://github.com/kamailio/kamailio/commit/6f9620704c661151415ad723e5f0b31f34b70c1d Author: Henning Westerholt Committer: Henning Westerholt Date: 2024-01-15T12:06:25Z docs/scripts: remove unused define

[sr-dev] git:master:47303fc3: Makefile.defs: remove unused define WITH_XAVP, its not used anymore from the code

2024-01-15 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 47303fc34d4795edc7b58569fb97e7b0532b0367 URL: https://github.com/kamailio/kamailio/commit/47303fc34d4795edc7b58569fb97e7b0532b0367 Author: Henning Westerholt Committer: Henning Westerholt Date: 2024-01-15T12:07:16Z Makefile.defs: remove unused define

[sr-dev] Re: auto build failure

2024-01-16 Thread Henning Westerholt via sr-dev
failure > > Hello, > > On 16/1/24 8:38, "Henning Westerholt via sr-dev" d...@lists.kamailio.org> wrote: > > Hello, > > > > the kamailio auto build is failing, I noticed after my last commit. But it > > seems > to be related to some docker is

[sr-dev] auto build failure

2024-01-15 Thread Henning Westerholt via sr-dev
Hello, the kamailio auto build is failing, I noticed after my last commit. But it seems to be related to some docker issue: [...] Starting job container /usr/bin/docker --config /home/runner/work/_temp/.docker_0ae486ca-dcaa-4edb-a953-36dca2bac9d9 login ghcr.io -u henningw --password-stdin

[sr-dev] Re: [kamailio/kamailio] dmq_usrloc not replicating contact attributes (#1968)

2024-01-16 Thread Henning Westerholt via sr-dev
PR was merged, close -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1968#issuecomment-1893503466 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] Re: [kamailio/kamailio] dmq_usrloc not replicating contact attributes (#1968)

2024-01-16 Thread Henning Westerholt via sr-dev
Closed #1968 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1968#event-11497276431 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] Re: Help needed with GH CI/CD after pushes to master

2024-01-18 Thread Henning Westerholt via sr-dev
Hello Richard, I had the same issue some days ago, but there were some fixes from Victor. Could you maybe check if this is still the case? Cheers, Henning From: Richard Chan via sr-dev Sent: Donnerstag, 18. Januar 2024 00:13 To: Kamailio (SER) - Development Mailing List Cc: Richard Chan

[sr-dev] Re: [kamailio/kamailio] error on make cdp module (Issue #3701)

2024-01-08 Thread Henning Westerholt via sr-dev
Hello, thanks for the report. Which Kamailio version are you using exactly? -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3701#issuecomment-1880534915 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] Re: git:master:52e82aa7: gcrypt: new module for crypto functions

2024-01-09 Thread Henning Westerholt via sr-dev
Hello, thanks for the interesting module addition. Just a quick note, as maybe developers are not aware of it, there are AES ECB and CBC cryptographic functions also available in the core: /kamailio/src/core/crypto$ egrep "ecb_|cbc_" rijndael.h void aes_ecb_encrypt(rijndael_ctx *ctx, u_int8_t

[sr-dev] Re: [kamailio/kamailio] tls: OpenSSL 3/1.1.1 is using shared ERR_STATE in all workers (Issue #3695)

2024-01-04 Thread Henning Westerholt via sr-dev
Thank you for the detailed analysis and also the POC. I remember from a custom module that I maintained for some year that using threads in Kamailio can bring some challenges due to its multi-process architectures. So its probably need to be discussed more thoroughly. The other option you

[sr-dev] Re: [kamailio/kamailio] error on make cdp module (Issue #3701)

2024-01-08 Thread Henning Westerholt via sr-dev
Hm, this is quite old and not maintained anymore. If you are also using the "https://github.com/herlesupreeth/kamailio; repository, you should probably address your question to this project. Please re-open if you can reproduce this with the current Kamailio git master or release branch 5.6/5.7

[sr-dev] Re: [kamailio/kamailio] error on make cdp module (Issue #3701)

2024-01-08 Thread Henning Westerholt via sr-dev
Closed #3701 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3701#event-11417359598 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] git:master:4df92312: parser/contact: Change addition order to the end of list

2024-01-08 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 4df92312166d8aad77ef0898d0a7c6ba40b7 URL: https://github.com/kamailio/kamailio/commit/4df92312166d8aad77ef0898d0a7c6ba40b7 Author: Xenofon Karamanos Committer: Henning Westerholt Date: 2024-01-08T15:55:43+01:00 parser/contact: Change addition

[sr-dev] Re: [kamailio/kamailio] parser/contact: Change addition order to the end of list (PR #3698)

2024-01-08 Thread Henning Westerholt via sr-dev
Thanks for the PR and the review, merged. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3698#issuecomment-1881168665 You are receiving this because you are subscribed to this thread. Message ID: ___

[sr-dev] Re: [kamailio/kamailio] parser/contact: Change addition order to the end of list (PR #3698)

2024-01-08 Thread Henning Westerholt via sr-dev
Merged #3698 into master. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3698#event-11418690902 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] Re: [Kamailio-Business] Roadmap to next major Kamailio release series v5.8.x

2024-01-11 Thread Henning Westerholt via sr-dev
Hello, early February as a freezing date sounds good to me, just to have one full month after the holiday season/vacation period to allow other people to finish eventual contributions. Cheers, Henning -- Henning Westerholt - https://skalatan.de/blog/ Kamailio services - https://gilawa.com

[sr-dev] Re: [kamailio/kamailio] rtpengine: Add support for codec-accept and codec-consume flags (PR #3710)

2024-01-11 Thread Henning Westerholt via sr-dev
Thanks for the fast review and merge. I would like to cherry-pick this to 5.7 stable branch, as it should be available according to the module readme (at least codec-accept). If necessary I can also create a dedicated commit only for the codec-accept. -- Reply to this email directly or view

[sr-dev] Re: [kamailio/kamailio] [pua] reginfo_increase_version does nothing (Issue #3234)

2023-12-21 Thread Henning Westerholt via sr-dev
@xkaraman -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3234#issuecomment-1866055685 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) - Development

[sr-dev] git:master:8ed4165f: pv: remove redunant variable definition (warning: unused variable ??????innerIndex??????)

2023-12-21 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 8ed4165fd3b0a331a4580894e2e7f83456b474fa URL: https://github.com/kamailio/kamailio/commit/8ed4165fd3b0a331a4580894e2e7f83456b474fa Author: Henning Westerholt Committer: Henning Westerholt Date: 2023-12-21T11:23:51Z pv: remove redunant variable

[sr-dev] git:master:0547a4be: pv: remove another redunant variable definition (warning: unused variable ??????innerIndex??????)

2023-12-21 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: 0547a4becbc6a48fdc473cfd26cdeedfa2a9317d URL: https://github.com/kamailio/kamailio/commit/0547a4becbc6a48fdc473cfd26cdeedfa2a9317d Author: Henning Westerholt Committer: Henning Westerholt Date: 2023-12-21T11:34:14Z pv: remove another redunant variable

[sr-dev] git:master:b2045d55: pv: remove another redunant variable definition related commit 8ed4165fd3b0a33

2023-12-21 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: b2045d559ed720789df80f543fc00923a07ada83 URL: https://github.com/kamailio/kamailio/commit/b2045d559ed720789df80f543fc00923a07ada83 Author: Henning Westerholt Committer: Henning Westerholt Date: 2023-12-21T11:54:52Z pv: remove another redunant variable

[sr-dev] Re: [kamailio/kamailio] topos: add compatiblity for early in-dialog NOTIFY (PR #3627)

2023-12-06 Thread Henning Westerholt via sr-dev
@mauritsderuiter95 So just to summarize, the problems you've seen were due to an issue in your test setup? So its now fine and this PR could be closed? -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3627#issuecomment-1842439805 You are receiving

[sr-dev] Re: [kamailio/kamailio] pv: Add support to $hfl $hflc for Diversion header (PR #3651)

2023-12-06 Thread Henning Westerholt via sr-dev
Xenofon did a review of the modules that uses diversion (diversion, sipcapture). The diversion module was tested with the PR. The sipcapture does only seems to store the header content in a string database field, so it should be fine. Regarding the SIP msg cloning, I reviewed the

[sr-dev] Re: [kamailio/kamailio] nathelper: Add Max-Forwards to keep-alive (PR #3657)

2023-11-29 Thread Henning Westerholt via sr-dev
Closed #3657. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3657#event-11104692347 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) - Development Mailing

[sr-dev] git:master:e1d9bec0: nathelper: add Max-Forwards headers which is mandatory for OPTIONS according to RFC 3261 (GH 3657)

2023-11-29 Thread Henning Westerholt via sr-dev
Module: kamailio Branch: master Commit: e1d9bec0e16727325330c822c21ae29237c7812a URL: https://github.com/kamailio/kamailio/commit/e1d9bec0e16727325330c822c21ae29237c7812a Author: Kristian Hogh Committer: Henning Westerholt Date: 2023-11-30T06:21:25Z nathelper: add Max-Forwards headers which

[sr-dev] Re: [kamailio/kamailio] nathelper: Add Max-Forwards to keep-alive (PR #3657)

2023-11-29 Thread Henning Westerholt via sr-dev
Thanks! I merged it manually due to clang-format issues. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3657#issuecomment-1833177050 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] Re: [kamailio/kamailio] Dialog DMQ sync: Timer needs fixing, firing on wrong node! (Issue #3656)

2023-11-23 Thread Henning Westerholt via sr-dev
Just for reference, there is also another issue with is related to the DMQ dialog handling #2080 -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3656#issuecomment-1825244374 You are receiving this because you are subscribed to this thread.

[sr-dev] Re: [kamailio/kamailio] TLS related crash in Kamailio 5.7.2 (double free) (Issue #3635)

2023-12-07 Thread Henning Westerholt via sr-dev
Just a generic note, it seems that some projects like haproxy don't recomment openssl 3.x for production use anymore, due to similar issues and also multiple major performance regressions with a performance of 1/10 to 1/4 comparing to openssl 1.1.x

[sr-dev] Re: Roadmap to next major Kamailio release series v5.8.x

2024-01-25 Thread Henning Westerholt via sr-dev
Hello Daniel, we will manage to commit an initial version of the module which some work in progress still open to the repository by the end of this month. Thanks, Henning -- Henning Westerholt – https://skalatan.de/blog/ Kamailio services – https://gilawa.com > -Original Message- >

[sr-dev] Re: git:5.7:8e915d44: tls_wolfssl: clean-up; continue to remove OpenSSL-isms

2024-01-27 Thread Henning Westerholt via sr-dev
Hello, as mentioned already (I think from Victor), the stable branches should only receive bugfixes, documentation extensions and similar code changes. This looks more like a substantial refactoring to me, maybe you can provide a bit background why this was necessary to be introduced into this

[sr-dev] Re: Make dlg_dmq_replicate_action() available from config

2024-01-27 Thread Henning Westerholt via sr-dev
Hello Stefan, Alex already provided some background why its not that easy and probably not a good idea. If you want to access lower level DMQ functions, there are dmq_send_message(..) and similar functions available already. There is also always the possibility to implement a more simple

[sr-dev] Re: [kamailio/kamailio] 5.7 db modules: init libssl in a thread (PR #3740)

2024-01-31 Thread Henning Westerholt via sr-dev
Thanks for the PR. It adds a new core implementation, which are usually not introduced in stable releases. As most people are not using databases with SSL, I am not sure if we really need to introduce this in the stable branch. We already got some regressions (e.g. #3737) in the TLS module in

[sr-dev] Re: [kamailio/kamailio] New module/file_out: Module to log custom strings to file (PR #3741)

2024-02-01 Thread Henning Westerholt via sr-dev
@miconda any suggestions regarding the name? Maybe going for the existing "log_" prefix, like log_file, log_file_out or similar? -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3741#issuecomment-1921450975 You are receiving this because you are

[sr-dev] Re: [kamailio/kamailio] New module/file_out: Module to log custom strings to file (PR #3741)

2024-02-01 Thread Henning Westerholt via sr-dev
Thanks for the feedback, the copyright will be added uniformly, and also extend the prefix, sure. Then lets keep the file_out name for now until somebody else has a better idea. -- Reply to this email directly or view it on GitHub:

  1   2   3   >