[sr-dev] [kamailio/kamailio] src/Makefile.defs: recognize GCC 14 as recent ⇔ not too old compiler (PR #3711)

2024-01-13 Thread Дилян Палаузов via sr-dev
similar to 87ed3d9c3ae3ecfabe6 GCC 14 will be released in April or May 2024. Adding this now will avoid backporting the change to the 5.8 branch. (Not that 87ed3d9c3ae3ecfabe6 was backported to 5.7.) You can view, comment on, or merge this pull request online at:

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

2024-01-10 Thread Дилян Палаузов via sr-dev
@dilyanpalauzov commented on this pull request. I think the line above LM_ERROR is bettet than LM_INFO. > @@ -295,7 +295,7 @@ static int mod_init(void) if(load_tm_api() < 0) { LM_INFO("cannot load the TM-functions - async relay disabled\n"); - memset(, 0,

[sr-dev] Re: [kamailio/kamailio] Crash in dns_cache.c with dns_cache_init=off (Issue #3350)

2023-12-02 Thread Дилян Палаузов via sr-dev
/notexpired -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3350#issuecomment-1837147025 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) - Development

[sr-dev] Re: [kamailio/kamailio] Silence “make: --libs: No such file or directory” warnings (PR #3475)

2023-11-26 Thread Дилян Палаузов via sr-dev
> I guess the option of the reporter to reopen the issue/pr is still available. I do not see such option. Currently the only way to prevent automatic closing is to write (probably periodically) any text. -- Reply to this email directly or view it on GitHub:

[sr-dev] Re: [kamailio/kamailio] Silence “make: --libs: No such file or directory” warnings (PR #3475)

2023-11-26 Thread Дилян Палаузов via sr-dev
> An issue or PR marked `stale` can have the label removed by reporter… This is not correct. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3475#issuecomment-1826889285 You are receiving this because you are subscribed to this thread. Message

[sr-dev] Re: [kamailio/kamailio] Silence “make: --libs: No such file or directory” warnings (PR #3475)

2023-11-26 Thread Дилян Палаузов via sr-dev
> From that point of view, I think it is fair that the reporter reacts on a > stale or close notification and indicate that the issue is still there or the > PR is still relevant. Writing repeatedly any text just for the sake of keeping a ticket open does not help anyhow. -- Reply to this

[sr-dev] Re: [kamailio/kamailio] Crash in dns_cache.c with dns_cache_init=off (Issue #3350)

2023-11-26 Thread Дилян Палаузов via sr-dev
Closing valid bugs, without fixing the real problem, results very few current reports for real problems in the software. This practice leads to the wrong impression, that Kamailio has no known problems, and therefore is good software. This is misleading and nobody has interest in having such

[sr-dev] Re: [kamailio/kamailio] Silence “make: --libs: No such file or directory” warnings (PR #3475)

2023-11-26 Thread Дилян Палаузов via sr-dev
The proposed here change does fix problems (warnings). I object that this PR is closed, instead of fixing the underlying warnings by the suggested or other ways. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3475#issuecomment-1826776355 You

[sr-dev] [kamailio/kamailio] Typos at https://www.kamailio.org/docs/modules/devel/ (Issue #3596)

2023-10-07 Thread Дилян Палаузов via sr-dev
At https://www.kamailio.org/docs/modules/devel/ is spelled CALL_CONTROL | Conector to call_control application -- | -- EVREXEC | Execut event routes at startup on dedicated processes It shall be co**NN**ector and execut**E**. Likewise for https://www.kamailio.org/docs/modules/5.8.x/ ,

[sr-dev] [kamailio/kamailio] examples/kamailio-basic-kemi-lua.lua: Add example for rtpengine (PR #3587)

2023-10-01 Thread Дилян Палаузов via sr-dev
A while ago I wanted to adjust my Lua configuration to work with rtpengine and I have adjusted this example accordingly. I have not tried the example, but if there are no obvious bugs with it, I think it will be useful. You can view, comment on, or merge this pull request online at:

[sr-dev] [kamailio/kamailio] avp: typo assing → assign (PR #3576)

2023-09-16 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3576 -- Commit Summary -- * avp: typo assing → assign -- File Changes -- M src/modules/avp/avp.xml (6) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3576.patch

[sr-dev] [kamailio/kamailio] Delete tag `orig` (Issue #3575)

2023-09-16 Thread Дилян Палаузов
I use command line completion when I work in git. In all repositories I have, the upsteam is always called `origin`, as this is the default from git. So typing `or` does complete the things correctly. Since Kamailio has a tag `orig` the completion completes to `orig`, not to `origin`. This

[sr-dev] Re: [kamailio/kamailio] No warning generated when UDP and no Content-Length header (PR #3359)

2023-09-12 Thread Дилян Палаузов
Implementing this is a really good idea and I am looking forward to see progress on this improvement. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3359#issuecomment-1716278343 You are receiving this because you are subscribed to this thread.

[sr-dev] Re: [kamailio/kamailio] misc/ typos (PR #3559)

2023-09-10 Thread Дилян Палаузов
@dilyanpalauzov pushed 1 commit. 4aae32d0eca0bba40f6a222c6bebe87c284cd265 misc/ typos -- View it on GitHub: https://github.com/kamailio/kamailio/pull/3559/files/a72d00424e7c1d396c0f1c36ccc3a7f0ca8b3d06..4aae32d0eca0bba40f6a222c6bebe87c284cd265 You are receiving this because you are subscribed

[sr-dev] Re: [kamailio/kamailio] misc/ typos (PR #3559)

2023-09-10 Thread Дилян Палаузов
@dilyanpalauzov pushed 1 commit. a72d00424e7c1d396c0f1c36ccc3a7f0ca8b3d06 misc/ typos -- View it on GitHub: https://github.com/kamailio/kamailio/pull/3559/files/4b938a1914e1eff437a6083580d20ecd6e0608e1..a72d00424e7c1d396c0f1c36ccc3a7f0ca8b3d06 You are receiving this because you are subscribed

[sr-dev] [kamailio/kamailio] misc/ typos (PR #3559)

2023-09-10 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3559 -- Commit Summary -- * misc/ typos -- File Changes -- M misc/examples/exec/exec_s5b.cfg (2) M misc/examples/ims/icscf/README.md (2) M

[sr-dev] [kamailio/kamailio] [app_lua] clarify the file scripts can be in source or bytecode (PR #3550)

2023-08-25 Thread Дилян Палаузов
This is valid both for Lua.org and LuaJIT. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3550 -- Commit Summary -- * [app_lua] clarify the file scripts can be in source or bytecode -- File Changes -- M

[sr-dev] [kamailio/kamailio] tm: clarify t_relay() can create a transaction (PR #3538)

2023-08-12 Thread Дилян Палаузов
I removed the word atomic before transaction, since it implies that there are in addition non-atomic ways to create a transaction. Per https://lists.kamailio.org/mailman3/hyperkitty/list/sr-us...@lists.kamailio.org/message/V7ZJEQZMXH7BNGRDI32BTYX742GVAWDB/ t_relay() creates a transaction. -

[sr-dev] [kamailio/kamailio] Typos (PR #3520)

2023-07-22 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3520 -- Commit Summary -- * docs/ : typos * etc/ typos -- File Changes -- M INSTALL (2) M doc/docbook/catalog.xml (2) M doc/docbook/dep.xsl (4) M

[sr-dev] [kamailio/kamailio] tls: remove openssl_mutex_shared (PR #3519)

2023-07-21 Thread Дилян Палаузов
The code from src/modules/tls/utils/openssl_mutex_shared/openssl_mutex_shared.c is in src/main.c . You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3519 -- Commit Summary -- * tls: remove openssl_mutex_shared -- File Changes --

[sr-dev] Re: [kamailio/kamailio] tlsa + OpenSSL 3.0.x: multiple unpredictable shm corruption (Issue #3222)

2023-07-08 Thread Дилян Палаузов
> @dilyanpalauzov : in recent versions of kamailio this is already included Yes, but it is unclear, if this is still necessary. That is what I asked: is openssl_mutex_shared needed by your setup. Do you mean by included, that you have

[sr-dev] Re: [kamailio/kamailio] tlsa + OpenSSL 3.0.x: multiple unpredictable shm corruption (Issue #3222)

2023-07-06 Thread Дилян Палаузов
Do you use this thing: https://github.com/kamailio/kamailio/tree/master/src/modules/tls/utils/openssl_mutex_shared ? The documentation says on some places, it is needed, on other — it is not needed anymore. -- Reply to this email directly or view it on GitHub:

[sr-dev] [kamailio/kamailio] Typos (PR #3491)

2023-06-21 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3491 -- Commit Summary -- * mohqueue: typo * osp: typos * p_usrloc: typo * permissions: typos * pike: typo * presence: typo * qos: typos * ratelimit: typo * regex:

[sr-dev] Re: [kamailio/kamailio] Silence “make: --libs: No such file or directory” warnings (PR #3475)

2023-06-20 Thread Дилян Палаузов
When the file `a` contains `BUILDER:=a`, the file `b` contains `BUILDER:=b`, and `Makefile` contains ```make include a include b all: @echo $(BUILDER) ``` (there is a TAB before @), then typing `make` prints `b`. I do not understand the question and I do not understand the build system

[sr-dev] Re: [kamailio/kamailio] Typos (PR #3487)

2023-06-18 Thread Дилян Палаузов
@dilyanpalauzov pushed 1 commit. 437b435c86c109bdda213998c6448ed255790018 utils: typos -- View it on GitHub: https://github.com/kamailio/kamailio/pull/3487/files/0bb781532bba6dbb1e3608f28860f81d60c29e88..437b435c86c109bdda213998c6448ed255790018 You are receiving this because you are subscribed

[sr-dev] Re: [kamailio/kamailio] Typos (PR #3487)

2023-06-18 Thread Дилян Палаузов
@dilyanpalauzov pushed 18 commits. 9e4b47f75a572bb7732c90aac4a6b4346737ad37 core: typos d61cd534f8ec10a232b8f446f277386b1abc2d66 src/Makefile*: typos 854e69f9db966f11d92c630a4000fc214726ef9e lwsc: typos b682bedc8f33ac58297cfead05489e2c4c5204d3 mangler: typos

[sr-dev] Re: [kamailio/kamailio] Typos (PR #3487)

2023-06-18 Thread Дилян Палаузов
@dilyanpalauzov pushed 18 commits. 734287cf22b720bc5bd81e2facdc7ed600265d05 core: typos c73fda6ab71c1bc5b61ecff00743487211d100e9 src/Makefile*: typos 70e898ea48735ef8583b8fbfbcc32d53ba6ca828 lwsc: typos 6a30b0efb6a892ae3282f200cf4a441f95b1723b mangler: typos

[sr-dev] Re: [kamailio/kamailio] Typos (PR #3487)

2023-06-18 Thread Дилян Палаузов
@dilyanpalauzov pushed 15 commits. dcc93fc0943ecce34af3b91c4e8452f9e6875b9e mangler: typos 5153593bd0c2fa11ee75dd2b40aaed13a16ff85d matrix: typo 8c84f33fbad82acb1ceabf01106106d795e36c47 maxfwd: typo 8166ea5fa8f2cbec48756c39b0444e4f270c19ed mediaproxy: typos

[sr-dev] [kamailio/kamailio] src/Makefile.defs: complete removal of VQ_MALLOC memory manager (PR #3488)

2023-06-18 Thread Дилян Палаузов
… started in commit f92113d8e16d6e59c170dfa703d9db30980b6877. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3488 -- Commit Summary -- * src/Makefile.defs: complete removal of VQ_MALLOC memory manager -- File Changes -- M

[sr-dev] [kamailio/kamailio] Typos (PR #3487)

2023-06-18 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3487 -- Commit Summary -- * core: typos * src/Makefile*: typos * lwsc: typos * mangler: typos * matrix: typo * maxfwd: typo * mediaproxy: typos * memcached: typos *

[sr-dev] [kamailio/kamailio] Silence “make: --libs: No such file or directory” warnings (PR #3475)

2023-06-01 Thread Дилян Палаузов
With GNU make 4.4 building Kamailio prints many “make: --libs: No such file or directory” and similar warnings. This changeset utilizes immediate variables and silences the warnings on my systems. You can view, comment on, or merge this pull request online at:

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

2023-05-31 Thread Дилян Палаузов
@dilyanpalauzov commented on this pull request. > @@ -92,11 +92,13 @@ modparam("topoh", "mask_key", "some secret here") SIP URIs. Can be any IP address, even a private-space or non-existing IP address (e.g., 192.168.1.1, 127.0.0.2), including the SIP server

[sr-dev] Re: [kamailio/kamailio] Typos (PR #3441)

2023-05-25 Thread Дилян Палаузов
@dilyanpalauzov pushed 17 commits. 6c7e28329282f5e0399524adebf52c1bced18b73 jansson: typo xavps's → xavp's 5ada4287313d9fde5ccd33cd005530d1d423f156 lcr: typos 17440cb64f2f2f0ff80b6bf3b5d6c83b66ed23c1 lost: typos 9883c74bdcc070c67342a646247cc80907470f30 lrkproxy: typos

[sr-dev] Re: [kamailio/kamailio] Typos (PR #3441)

2023-05-24 Thread Дилян Палаузов
@dilyanpalauzov pushed 17 commits. a68fc0ca6461e871a7ee848189b1d77880ed9534 jansson: typo xavps's → xavp's 9094402dda43f1a8c71cb1fe12148668e49e110a lcr: typos f7988c5e68e496cf9dba153f02ba1781052d582b lost: typos 356121ac512a6d12c345c112734a12b168d4dd2b lrkproxy: typos

[sr-dev] Re: [kamailio/kamailio] src/Makefile.defs: recognize GCC 13 as recent ⇔ not too old compiler (PR #3469)

2023-05-24 Thread Дилян Палаузов
I have the trouble with v5.7 and I assume the same applies for the `master` branch. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3469#issuecomment-1561803720 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] [kamailio/kamailio] src/Makefile.defs: recognize GCC 13 as recent ⇔ not too old compiler (PR #3469)

2023-05-24 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3469 -- Commit Summary -- * src/Makefile.defs: recognize GCC 13 as recent ⇔ not too old compiler -- File Changes -- M src/Makefile.defs (8) -- Patch Links --

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

2023-05-21 Thread Дилян Палаузов
@dilyanpalauzov commented on this pull request. > @@ -92,11 +92,13 @@ modparam("topoh", "mask_key", "some secret here") SIP URIs. Can be any IP address, even a private-space or non-existing IP address (e.g., 192.168.1.1, 127.0.0.2), including the SIP server

[sr-dev] [kamailio/kamailio] Typos (PR #3441)

2023-05-08 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3441 -- Commit Summary -- * jansson: typo xavpss → xavps * lcr: typos * lost: typos * lrkproxy: typos * rtpproxy: typos * tmrec: typos recurrence * tmx: update hyperlink to

[sr-dev] [kamailio/kamailio] Typos (PR #3415)

2023-04-10 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3415 -- Commit Summary -- * call_control: typos * cfgutils: typos * ims_registrar_pcscf: typo * janssonrpcc: typo * jsonrpcs: typos * jwt: typo algoritm → algorithm *

[sr-dev] Re: [kamailio/kamailio] rtpengine clarifications (PR #3414)

2023-04-09 Thread Дилян Палаузов
@dilyanpalauzov pushed 1 commit. 432cd5a88767a77377b164874503f1f974afc138 rtpengine clarifications -- View it on GitHub: https://github.com/kamailio/kamailio/pull/3414/files/31181e460ced78ca6e999bb5cd69323173b2feae..432cd5a88767a77377b164874503f1f974afc138 You are receiving this because you

[sr-dev] [kamailio/kamailio] rtpengine clarifications (PR #3414)

2023-04-08 Thread Дилян Палаузов
See http://lists.sipwise.com/pipermail/spce-user_lists.sipwise.com/2023-April/030012.html and https://github.com/sipwise/rtpengine/issues/1595#issuecomment-1382378886 . You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3414 -- Commit

[sr-dev] [kamailio/kamailio] Typos (PR #3411)

2023-04-02 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3411 -- Commit Summary -- * benchmark: typo * carrierroute: typos * cdp: typos * cnxcc: typos * cplp: typos * ctl: typos * corex: typos * crypto: typo * enum: typos *

[sr-dev] Re: [kamailio/kamailio] utils/kamctl: allow the definition of a specific startup file at star… (PR #3387)

2023-03-04 Thread Дилян Палаузов
If this can be managed over an environment variable, then the latter shall include all the directories. As far as I see with this change, the file must still be in /etc (or $ETCDIR, but I assume both are almost thr same). -- Reply to this email directly or view it on GitHub:

[sr-dev] Re: [kamailio/kamailio] topoh: th_mask_decode (th_mask_encode (x)) ≠ x (Issue #3358)

2023-02-15 Thread Дилян Палаузов
The above input/output is generated by: ``` modparam("topoh", "mask_key", "TEAI32l)- eauiDEUIA!?()") modparam("topoh", "mask_ip", "127.3.4.84") modparam("topoh", "sanity_checks", 0) modparam("topoh", "event_mode", 0) modparam("topoh", "event_callback", "ksr_topoh_event") ``` and there is no

[sr-dev] Re: [kamailio/kamailio] topoh: th_mask_decode (th_mask_encode (x)) ≠ x (Issue #3358)

2023-02-15 Thread Дилян Палаузов
I try here to present the problem as clear as possible. I make these changes to 5.6, which let me see the input and output of th_mask_encode and th_mask_decode. ```diff diff --git a/src/modules/topoh/th_mask.c b/src/modules/topoh/th_mask.c index 122d0935a5..895864e3c4 100644 ---

[sr-dev] Re: [kamailio/kamailio] Typos (PR #3375)

2023-02-15 Thread Дилян Палаузов
> I have also changes which change typos and documentation, which change/add/remove sentences in the documentation -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3375#issuecomment-1431039692 You are receiving this because you are subscribed to

[sr-dev] Re: [kamailio/kamailio] Typos (PR #3375)

2023-02-15 Thread Дилян Палаузов
I have changes on my computer concerning typos, and I submit them in batches for some modules. I have also changes which change typos and documentation, these I will submit in separate PR per modules, once I am confident with the changes. The changes were **caught** by using regex and

[sr-dev] Re: [kamailio/kamailio] Typos (PR #3375)

2023-02-14 Thread Дилян Палаузов
The changes here are for modules, whose name starts with “a”. I do not expect more typos in modules, whose name starts with “a”. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3375#issuecomment-1430063609 You are receiving this because you are

[sr-dev] [kamailio/kamailio] Typos (PR #3375)

2023-02-14 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3375 -- Commit Summary -- * acc: typo * acc_json: typos * acc_radius: typo * alias_db: typo * app_java: typos * app_jsdt: typos * app_lua: typos * app_lua_sr: typos *

[sr-dev] [kamailio/kamailio] Typos (PR #3374)

2023-02-13 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3374 -- Commit Summary -- * sanity: typos * sca: typos * sctp: typos * sdpops: typos * secfilter: typo * secsipid: typos * secsipid_proc: typos * sipcapture: typos *

[sr-dev] Re: [kamailio/kamailio] presence: docs for rpc presence.presentity_show command (c167c47)

2023-02-13 Thread Дилян Палаузов
This line likely means > kamctl rpc presence.presentity_show full * sipserver.com -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/commit/c167c4738fce31cec292edc803160d607a3faaf6#r100474300 You are receiving this because you are subscribed to this

[sr-dev] [kamailio/kamailio] Typos (PR #3373)

2023-02-13 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3373 -- Commit Summary -- * drouting: typos * domainpolicy: typos * domain: typo * dmq: typos * dlgs: typos * dmq_usrloc: typo * dispatcher: typos * dialplan: typos *

[sr-dev] [kamailio/kamailio] Typos (PR #3371)

2023-02-13 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3371 -- Commit Summary -- * p_usrloc: typos * pdb: typos * peering: typo * permissions: typos * pike: typos * pipelimit: typo * presence: typos * presence_conference:

[sr-dev] [kamailio/kamailio] rtpengine: document options SDES-pad and SDES-nonew (PR #3369)

2023-02-12 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3369 -- Commit Summary -- * rtpengine: document options SDES-pad and SDES-nonew -- File Changes -- M src/modules/rtpengine/bencode.h (2) M

[sr-dev] [kamailio/kamailio] Typos (PR #3363)

2023-02-08 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3363 -- Commit Summary -- * imc: typo * ims_auth: typos * ims_charging: typos * ims_dialog: typos * ims_diameter_server: typos * ims_icscf: typos * ims_ipsec_pcscf: typos

[sr-dev] [kamailio/kamailio] Typos (PR #3362)

2023-02-08 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3362 -- Commit Summary -- * doc/scripts: typos * seas: typos * siputis: typos * tls: typos * tls_wolfssl: typos * xcap_client: typos * xcap_server: typos * xhttp: typo

[sr-dev] Re: [kamailio/kamailio] typos https://kamailio.org/docs/db-tables/kamailio-db-devel.html (Issue #3361)

2023-02-07 Thread Дилян Палаузов
Closed #3361 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3361#event-8455282508 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] Re: [kamailio/kamailio] typos https://kamailio.org/docs/db-tables/kamailio-db-devel.html (Issue #3361)

2023-02-07 Thread Дилян Палаузов
Thanks for your answer. Since December there is no `,if` without space in between in the source code, but on the above webpage `,if` does exist. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3361#issuecomment-1420371602 You are receiving this

[sr-dev] [kamailio/kamailio] typos https://kamailio.org/docs/db-tables/kamailio-db-devel.html (Issue #3361)

2023-02-06 Thread Дилян Палаузов
https://kamailio.org/docs/db-tables/kamailio-db-devel.html#idm9040 contains: * thay instead of they, (UID_AUTH_DB Module) * `,if` without comma in between (in UAC Module) * for table "uacreg" column realm is described as “Remote username”, it should be Remote realm For table dbaliases indexes

[sr-dev] Re: [kamailio/kamailio] No warning generated when UDP and no Content-Length header (PR #3359)

2023-02-05 Thread Дилян Палаузов
This should apply not only to UDP and SCTP, but also to WebSockets: https://www.rfc-editor.org/rfc/rfc7118.html#section-5 > Because the WebSocket transport preserves message boundaries, the use of the > Content-Length header in SIP messages is not necessary when they are > transported using

[sr-dev] [kamailio/kamailio] topoh: th_mask_decode (th_mask_encode (x)) ≠ x (Issue #3358)

2023-02-04 Thread Дилян Палаузов
The file [th-test.c.txt](https://github.com/kamailio/kamailio/files/10608862/th-test.c.txt) contains the exact copy of the th_mask_encode() and th_mask_decode() functions from src/modules/topoh/th_mask.c . (except LM_ERR is removed and pkg_malloc/pkg_free are replaced with malloc/free). When

[sr-dev] [kamailio/kamailio] Typos (PR #3357)

2023-02-04 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3357 -- Commit Summary -- * usrloc: typos * src/lib: typos * acc: typos * acc_diameter: typos -- File Changes -- M src/lib/Makefile.defs (6) M src/lib/README (2) M

[sr-dev] Re: [kamailio/kamailio] core: include DBG_SYS_MEMORY and MEM_JOIN_FREE on flags: output (PR #3356)

2023-02-03 Thread Дилян Палаузов
> Include `, DBG_SYS_MEMORY` in flags: output, too. and MEM_JOIN_FREE -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3356#issuecomment-1416136172 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] Re: [kamailio/kamailio] core: include DBG_SYS_MEMORY on flags: output (PR #3356)

2023-02-03 Thread Дилян Палаузов
Include `, DBG_SYS_MEMORY` in flags: output, too. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3356#issuecomment-1416042064 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] Re: [kamailio/kamailio] core: include DBG_SYS_MEMORY on flags: output (PR #3356)

2023-02-03 Thread Дилян Палаузов
@dilyanpalauzov pushed 1 commit. 2ac8bf389582be7e7a73789314e35e7d1d7b374e core: include DBG_SYS_MEMORY and MEM_JOIN_FRE on flags: output -- View it on GitHub:

[sr-dev] Re: [kamailio/kamailio] Unclear default value for mem_join (Issue #3355)

2023-02-03 Thread Дилян Палаузов
Closing the ticket does not remove the contradictions imposed by the documentation. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3355#issuecomment-1416025554 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] [kamailio/kamailio] core: include DBG_SYS_MEMORY on flags: output (PR #3356)

2023-02-03 Thread Дилян Палаузов
E.g. when calling `core.info` RPC. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3356 -- Commit Summary -- * core: include DBG_SYS_MEMORY on flags: output -- File Changes -- M src/core/ver_defs.h (8) -- Patch Links --

[sr-dev] [kamailio/kamailio] Unclear default value for mem_join (Issue #3355)

2023-02-03 Thread Дилян Палаузов
According to https://www.kamailio.org/wikidocs/cookbooks/devel/core/#mem_join the default value for mem_join is `1 (enabled)`.

[sr-dev] [kamailio/kamailio] core: typos (PR #3354)

2023-02-03 Thread Дилян Палаузов
In src/core/ccopts.sh:98 there is “acroos”. This could be macros or across. Likewise in 2× src/Makefile.defs In src/core/dst_blocklist.c:625 `taget` could be `target` or `tagged`. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3354

[sr-dev] Re: [kamailio/kamailio] websocket: move some logging from ERROR to INFO (PR #3349)

2023-02-03 Thread Дилян Палаузов
> I also see an INFO changed to DEBUG when the connection was unexpectedly lost. This change was not intended, I have reverted it. I agree the remaining changes are related to error conditions, not just to basic logging. I have removed them, leaving in the changeset only typos. -- Reply to

[sr-dev] Re: [kamailio/kamailio] websocket: move some logging from ERROR to INFO (PR #3349)

2023-02-03 Thread Дилян Палаузов
@dilyanpalauzov pushed 1 commit. cbd06b1c67fa9dc9988496f1d540581a41d5e583 websocket: typos -- View it on GitHub: https://github.com/kamailio/kamailio/pull/3349/files/e250e9264a17bb1ee028ec0d3b9ffb864aef5325..cbd06b1c67fa9dc9988496f1d540581a41d5e583 You are receiving this because you are

[sr-dev] Re: [kamailio/kamailio] websocket: move some logging from ERROR to INFO (PR #3349)

2023-02-03 Thread Дилян Палаузов
@dilyanpalauzov pushed 1 commit. e250e9264a17bb1ee028ec0d3b9ffb864aef5325 websocket: move some logging from ERROR to INFO -- View it on GitHub: https://github.com/kamailio/kamailio/pull/3349/files/8fba637cbcd583a3d9f5dd0e1aa02cfa7db43814..e250e9264a17bb1ee028ec0d3b9ffb864aef5325 You are

[sr-dev] Re: [kamailio/kamailio] Crash in dns_cache.c with use_dns_cache=off (Issue #3350)

2023-02-01 Thread Дилян Палаузов
Without topoh, calling +437200101011, resolved by the enum module to sip:enum-t...@sip.nemox.net, running under valgrind with ``` dns_try_ipv6=off dns_retr_time=2 use_dns_failover=on dns_srv_lb=on dns_try_naptr=on dns_cache_flags=1 ``` I get: * 1st test case ``` dns_cache_init=off

[sr-dev] [kamailio/kamailio] doc/tutorials/dns: USE_NAPTR is on by default (PR #3353)

2023-02-01 Thread Дилян Палаузов
My reading of the original text: USE_NAPTR - if defined the naptr lookup support will be compiled in. NAPTR support still has to be enabled from Kamailios config file (its off by default). was that USE_NAPTR was by default off (undefined). This change clarifies that by

[sr-dev] Re: [kamailio/kamailio] Improper logging of unparsable next hop URI (Issue #3351)

2023-01-29 Thread Дилян Палаузов
As far as I can see this problem happens with `modparam("topoh", "mask_key", "TEAI32l)- eauiDEUIA!?()")` and does not happen with `modparam("topoh", "mask_key", "TEAI32l")`. Does this qualify now as bug report? -- Reply to this email directly or view it on GitHub:

[sr-dev] Re: [kamailio/kamailio] Improper logging of unparsable next hop URI (Issue #3351)

2023-01-28 Thread Дилян Палаузов
With this change: ```diff diff --git a/src/core/kemi.c b/src/core/kemi.c --- a/src/core/kemi.c +++ b/src/core/kemi.c @@ -1570,6 +1570,7 @@ static int sr_kemi_core_to_proto_helper(sip_msg_t *msg) } } if(parse_uri(uri.s, uri.len, _uri)!=0) { +

[sr-dev] Re: [kamailio/kamailio] rr: doc — replace enable_username with add_username (PR #3352)

2023-01-28 Thread Дилян Палаузов
@dilyanpalauzov pushed 1 commit. c892deb3fb3be105254b0e6d6fe5b2924a11c501 rr: doc — replace enable_username with add_username -- View it on GitHub: https://github.com/kamailio/kamailio/pull/3352/files/81516bf702e006d923e0b5d6b9c48fc21bbc0456..c892deb3fb3be105254b0e6d6fe5b2924a11c501 You are

[sr-dev] [kamailio/kamailio] rr: doc — replace enable_username with add_username (PR #3352)

2023-01-28 Thread Дилян Палаузов
The documentation of rr.custom_user_avp speaks about option `enable_username`, but there is no such option. I assume the documentation means `add_username`. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3352 -- Commit Summary --

[sr-dev] [kamailio/kamailio] Improper logging of unparsable next hop URI (Issue #3351)

2023-01-28 Thread Дилян Палаузов
On receiving: ``` BYE sip:127.3.4.84;line=sr-bkhov30ei16khxiahn60i3jelfjzcwdavb.a8bhyuzda8bbavz8ymwb4lsi0iqmoigitdx9vbqgv5fjhdb.j8wcg5x8evkyv5fyqdbnguztav1yamwha8bwqfz8ymwl28o** SIP/2.0 Route: Route: Via: SIP/2.0/WSS 964rp9hud53v.invalid;branch=z9hG4bK47814 Max-Forwards: 70 To:

[sr-dev] [kamailio/kamailio] Crash in dns_cache.c with use_dns_cache=off (Issue #3350)

2023-01-27 Thread Дилян Палаузов
With Kamailio 5.6.3 I add this code ```diff -- Routing to foreign domains function ksr_route_sipout() +local y = KSR.enum.enum_query() +KSR.err('ENUM RESULT ' .. tostring(y) .. ' :RURI ' .. KSR.kx.get_ruri() .. '\n') if KSR.is_myself_ruri() then return 1; end

[sr-dev] Re: [kamailio/kamailio] websocket: move some logging from ERROR to INFO (PR #3349)

2023-01-27 Thread Дилян Палаузов
I have changed the logging level to DEBUG (`LM_DBG()`). -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3349#issuecomment-1406803045 You are receiving this because you are subscribed to this thread. Message ID:

[sr-dev] Re: [kamailio/kamailio] websocket: move some logging from ERROR to INFO (PR #3349)

2023-01-27 Thread Дилян Палаузов
@dilyanpalauzov pushed 1 commit. 8fba637cbcd583a3d9f5dd0e1aa02cfa7db43814 websocket: move some logging from ERROR to INFO -- View it on GitHub: https://github.com/kamailio/kamailio/pull/3349/files/be844b1b2f2bdbb34b8c2781b9f99624119be31b..8fba637cbcd583a3d9f5dd0e1aa02cfa7db43814 You are

[sr-dev] Re: [kamailio/kamailio] nathelper: ping_nated_only=1 sends SIP-request-based pings (PR #3346)

2023-01-27 Thread Дилян Палаузов
> for me implies, that sipping_bflag is relevant when ping_nated_only=1, and > irrelevant when ping_nated_only=0. READ: for me implies, that sipping_bflag is relevant when ping_nated_only=0, and irrelevant when ping_nated_only=1. -- Reply to this email directly or view it on GitHub:

[sr-dev] [kamailio/kamailio] websocket: move some logging from ERROR to INFO (PR #3349)

2023-01-27 Thread Дилян Палаузов
I run Kamailio with ``` log_stderror=yes /* LOG Levels: 3=DBG, 2=INFO, 1=NOTICE, 0=WARN, -1=ERR, ... */ debug=-1 ``` and inspect the output. Some messages from the websocket module are logged repeatedly at ERROR-level, but they are in no way errors. This changes moves such periodially logged

[sr-dev] [kamailio/kamailio] core: msg_translator.c put new line after «parsing failed» error message (PR #3348)

2023-01-27 Thread Дилян Палаузов
I use Kamailio with `log_stderror=yes`. When Kamailio receives confirmation from the Websocket reverse proxy, it logs on stderr ``` 23(24) ERROR: core [core/parser/parse_fline.c:271]: parse_first_line(): parse_first_line: bad message (offset: 22) 23(24) ERROR: core

[sr-dev] Re: [kamailio/kamailio] nathelper: ping_nated_only=1 sends 4-bytes UDP-packet pings (PR #3346)

2023-01-27 Thread Дилян Палаузов
To be precise, the current wording: > If this parameter is set to 1 then only contacts that have the behind NAT > “nat_bflag” flag in user location database set will get ping. > > If it is 0 and sipping_bflag is not set, then the 4-bytes UDP ping is sent to > all contacts. If it is 0 and

[sr-dev] Re: [kamailio/kamailio] nathelper: ping_nated_only=1 sends 4-bytes UDP-packet pings (PR #3346)

2023-01-27 Thread Дилян Палаузов
The documentation says for nathelper.ping_nated_only=1 that only contacts that have the behind NAT “nat_bflag” flag in user location database set will get ping. The current wording does not specify, if the latter ping is 4-byted UPD-packet ping, or SIP-request-based ping. Yesterday when I

[sr-dev] Re: [kamailio/kamailio] nathelper: ping_nated_only=1 sends 4-bytes UDP-packet pings (PR #3346)

2023-01-27 Thread Дилян Палаузов
@dilyanpalauzov pushed 1 commit. d1ac6d99f34775e2f95e725a4de015824f3e9c20 nathelper: ping_nated_only sends 4-bytes UDP-packet pings -- View it on GitHub: https://github.com/kamailio/kamailio/pull/3346/files/5b7ea662ccd28dc998700610ab2e682b40f839cb..d1ac6d99f34775e2f95e725a4de015824f3e9c20 You

[sr-dev] [kamailio/kamailio] nathelper: ping_nated_only=1 sends 4-bytes UDP-packet pings (PR #3346)

2023-01-26 Thread Дилян Палаузов
My experiments show that with ``` modparam(nathelper, ping_nated_only, 1) modparam(nathelper, sipping_bflag, FLB_NATSIPPING) modparam(usrloc, nat_bflag, FLB_NATB) ``` when both FLB_NATB and FLB_NATSIPPING branch flags are set, no OPTIONS from Kamailio to UAC is generated, so I assume that

[sr-dev] [kamailio/kamailio] Typos (PR #3344)

2023-01-26 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3344 -- Commit Summary -- * test/ : typos * snmpstats: typos -- File Changes -- M src/modules/snmpstats/doc/snmpstats_admin.xml (8) M src/modules/snmpstats/hashTable.c (2)

[sr-dev] Re: [kamailio/kamailio] Typos (PR #3339)

2023-01-23 Thread Дилян Палаузов
@dilyanpalauzov pushed 4 commits. c54113b4167bb0f71b759d8bde2cbcc909918d16 fixup pkg 43a6b59f2e49ed69f3f8f534eadc2e0c8c8e6ddb lib/srdb1: typos 747254a702d332a14cccaae50e77cc9d0925b5a9 typos 73052f6f02ebf2bea4d3958778a002e63737293d erlang: typos -- View it on GitHub:

[sr-dev] [kamailio/kamailio] Typos (PR #3339)

2023-01-23 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3339 -- Commit Summary -- * pkg: update www.kamailio.org links to HTTPS * lib/srdb1: typos * typos * erlang: typos -- File Changes -- M .github/CONTRIBUTING.md (6) M

[sr-dev] [kamailio/kamailio] modparam("dialog", … for ims_dialog (Issue #3337)

2023-01-22 Thread Дилян Палаузов
https://www.kamailio.org/docs/modules/devel/modules/ims_dialog.html contains examples how to configure the module: ``` modparam("ims_dialog", "dlg_extra_hdrs", "Hint: credit expired\r\n") modparam("ims_dialog", "detect_spirals", 1) modparam("dialog", "profiles_with_value", "caller ; my_profile")

[sr-dev] [kamailio/kamailio] dialog.dlg_filter_mode has ambiguous definition with double negation (Issue #3336)

2023-01-22 Thread Дилян Палаузов
https://www.kamailio.org/docs/modules/devel/modules/dialog#dialog.p.dlg_filter_mode says for dialog.dlg_filter_mode (int) > Set dialog fitering mode, which can specify what dialogs are processed. Its > value can be a combination (the sum) of following flags: > > 1 - do not send keepalives and

[sr-dev] [kamailio/kamailio] Typos (PR #3335)

2023-01-22 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3335 -- Commit Summary -- * replace SIGINFO64_WORKARROUND with SIGINFO64_WORKAROUND * utils/kamctl: typos * doc/tutorials: typos -- File Changes -- M

[sr-dev] Re: [kamailio/kamailio] tls.reload during tls relaying cause tls memory double free if done every second (Issue #3319)

2023-01-21 Thread Дилян Палаузов
Running this under `valgrind` will provide precise information on where memory is allocated, freed, and double-freed. As alternative compiling the software under "address sanitizer" should reveal the same ingotmation. -- Reply to this email directly or view it on GitHub:

[sr-dev] [kamailio/kamailio] websocket: example must listen on ports 80 and 443 (PR #3332)

2023-01-15 Thread Дилян Палаузов
otherwise the code below always matches: ``` if ($Rp != 80 $Rp != 443) { ``` You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3332 -- Commit Summary -- * websocket: example must listen on ports 80 and 443 -- File Changes --

[sr-dev] Re: [kamailio/kamailio] sipdump: clarify PV must be loaded in advance for mode=2 (PR #3329)

2023-01-13 Thread Дилян Палаузов
This is not correct. Why do I get with ``` modparam("sipdump", "enable", 1) modparam("sipdump", "mode", 2) modparam("sipdump", "event_callback", "ksr_sipdump_event") … function ksr_sipdump_event(evname) KSR.err("ksr_sipdump_event 1-" .. evname .. "\n" ..

[sr-dev] [kamailio/kamailio] sipdump: clarify PV must be loaded in advance for mode=2 (PR #3329)

2023-01-13 Thread Дилян Палаузов
I hope the example will be properly formatted. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3329 -- Commit Summary -- * sipdump: clarify PV must be loaded in advance for mode=2 -- File Changes -- M

[sr-dev] [kamailio/kamailio] Remove double #include<>d headers (PR #3328)

2023-01-11 Thread Дилян Палаузов
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3328 -- Commit Summary -- * core: remove double #included headers * app_java: remove double #included headers * auth_diameter: remove double #included headers * auth_ephemeral:

  1   2   >