Module: kamailio
Branch: master
Commit: 33b2e513aefe00bf166cdcae360e305615b223da
URL: https://github.com/kamailio/kamailio/commit/33b2e513aefe00bf166cdcae360e305…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2024-01-23T21:05:41+01:00
tm: release transaction on kill if a final response was sent
- do not send another final response for error cases
---
Modified: src/modules/tm/t_funcs.c
---
Diff: https://github.com/kamailio/kamailio/commit/33b2e513aefe00bf166cdcae360e305…
Patch: https://github.com/kamailio/kamailio/commit/33b2e513aefe00bf166cdcae360e305…
---
diff --git a/src/modules/tm/t_funcs.c b/src/modules/tm/t_funcs.c
index b9f9949d0fb..5515a4aba6f 100644
--- a/src/modules/tm/t_funcs.c
+++ b/src/modules/tm/t_funcs.c
@@ -177,6 +177,11 @@ int kill_transaction(struct cell *trans, int error)
int reply_ret;
int ret;
+ if((trans->uas.request != NULL)
+ && (trans->uas.request->msg_flags & FL_FINAL_REPLY)) {
+ return t_release_transaction(trans);
+ }
+
/* we reply statefully and enter WAIT state since error might
have occurred in middle of forking and we do not
want to put the forking burden on upstream client;
@@ -205,6 +210,11 @@ int kill_transaction_unsafe(struct cell *trans, int error)
int reply_ret;
int ret;
+ if((trans->uas.request != NULL)
+ && (trans->uas.request->msg_flags & FL_FINAL_REPLY)) {
+ return t_release_transaction(trans);
+ }
+
/* we reply statefully and enter WAIT state since error might
have occurred in middle of forking and we do not
want to put the forking burden on upstream client;
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:
https://github.com/kamailio/kamailio/pull/3475
-- Commit Summary --
* lib/ims: Aviod “make: --libs: No such file or directory” warnings
* auth_identity: Aviod “make: --libs: No such file or directory” warnings
* auth_ephemeral: Aviod “make: --libs: No such file or directory” warnings
* app_python3: Avoid “/bin/sh: line 1: --ldflags: command not found” warnings
* app_python3s: Avoid “/bin/sh: line 1: --ldflags: command not found” warnings
* cdp: Aviod “make: --libs: No such file or directory” warnings
* cnxcc: Avoid “make: --libs-only-L: No such file or directory” warnings
* cplc: Avoid “make: --libs-only-L: No such file or directory” warnings
* crypto: Aviod “make: --libs: No such file or directory” warnings
* db_postgres: Avoid “make: --libs-only-L: No such file or directory” warnings
* db_redis: Avoid “make: --libs-only-L: No such file or directory” warning
* dialplan: Avoid “make: --libs-only-L: No such file or directory” warning
* gzcompress: Aviod “make: --libs: No such file or directory” warnings
* http_async_client: Aviod “make: --libs: No such file or directory” warnings
* http_client: Aviod “make: --libs: No such file or directory” warning
* ims_auth: Avoid “make: --libs-only-L: No such file or directory” warning
* ims_icscf: Aviod “make: --libs: No such file or directory” warning
* ims_registrar_scscf: Avoid “make: --libs-only-L: No such file or directory” warnings
* ims_registrar_pcscf: Aviod “make: --libs: No such file or directory” warning
* ims_isc: Avoid “make: --libs-only-L: No such file or directory” warnings
* jansson: Aviod “make: --libs: No such file or directory” warning
* kazoo: Avoid “make: --libs: No such file or directory’ warning
* lcr: Aviod “make: --libs: No such file or directory” warning
* log_systemd: Aviod “make: --libs: No such file or directory” warning
* lost: Aviod “make: --libs: No such file or directory” warnings
* lwsc: Aviod “make: --libs: No such file or directory” warning
* ndb_redis: Aviod “make: --libs: No such file or directory” warning
* osp: Avoid “make: --libs: No such file or directory’ warning
* outbound: Aviod “make: --libs: No such file or directory” warnings
* presence: Avoid “make: --libs-only-L: No such file or directory” warnings
* presence_conference: Aviod “make: --libs: No such file or directory” warning
* presence_dialog: Aviod “make: --libs: No such file or directory” warning
* presence_reginfo: Aviod “make: --libs: No such file or directory” warning
* presence_xml: Avoid “make: --libs-only-L: No such file or directory” warnings
* pua: Avoid “make: --libs-only-L: No such file or directory” warnings
* pua_bla: Aviod “make: --libs: No such file or directory” warning
* pua_dialoginfo: Aviod “make: --libs: No such file or directory” warning
* pua_reginfo: Aviod “make: --libs: No such file or directory” warning
* pua_usrloc: Aviod “make: --libs: No such file or directory” warning
* pua_xmpp: Aviod “make: --libs: No such file or directory” warning
* regex: Aviod “make: --libs: No such file or directory” warning
* rls: Avoid “make: --libs-only-L: No such file or directory” warnings
* slack: Aviod “make: --libs: No such file or directory” warning
* systemdops: Avoid “make: --libs-only-L: No such file or directory” warning
* tls: Aviod “make: --libs: No such file or directory” warnings
* tlsa: Avoid “make: --libs-only-L: No such file or directory” warning
* topos_redis: Aviod “make: --libs: No such file or directory” warning
* utils: Avoid “make: --libs-only-L: No such file or directory” warnings
* uuid: Aviod “make: --libs: No such file or directory” warning
* websocket: Aviod “make: --libs: No such file or directory” warnings
* xcap_server: Avoid “make: --libs-only-L: No such file or directory” warnings
* xhttp_pi: Avoid “make: --libs-only-L: No such file or directory” warnings
* xmlops: Aviod “make: --libs: No such file or directory” warning
* xmlrpc: Aviod “make: --libs: No such file or directory” warning
-- File Changes --
M src/lib/ims/Makefile (4)
M src/modules/app_python3/Makefile (6)
M src/modules/app_python3s/Makefile (6)
M src/modules/auth_ephemeral/Makefile (2)
M src/modules/auth_identity/Makefile (4)
M src/modules/cdp/Makefile (4)
M src/modules/cnxcc/Makefile (4)
M src/modules/cplc/Makefile (4)
M src/modules/crypto/Makefile (2)
M src/modules/db_postgres/Makefile (2)
M src/modules/db_redis/Makefile (2)
M src/modules/dialplan/Makefile (2)
M src/modules/gzcompress/Makefile (2)
M src/modules/http_async_client/Makefile (6)
M src/modules/http_client/Makefile (2)
M src/modules/ims_auth/Makefile (4)
M src/modules/ims_icscf/Makefile (4)
M src/modules/ims_isc/Makefile (4)
M src/modules/ims_registrar_pcscf/Makefile (4)
M src/modules/ims_registrar_scscf/Makefile (4)
M src/modules/jansson/Makefile (2)
M src/modules/kazoo/Makefile (2)
M src/modules/lcr/Makefile (2)
M src/modules/log_systemd/Makefile (2)
M src/modules/lost/Makefile (6)
M src/modules/lwsc/Makefile (2)
M src/modules/ndb_redis/Makefile (2)
M src/modules/osp/Makefile (2)
M src/modules/outbound/Makefile (7)
M src/modules/presence/Makefile (4)
M src/modules/presence_conference/Makefile (4)
M src/modules/presence_dialoginfo/Makefile (4)
M src/modules/presence_reginfo/Makefile (4)
M src/modules/presence_xml/Makefile (4)
M src/modules/pua/Makefile (4)
M src/modules/pua_bla/Makefile (4)
M src/modules/pua_dialoginfo/Makefile (4)
M src/modules/pua_reginfo/Makefile (4)
M src/modules/pua_usrloc/Makefile (4)
M src/modules/pua_xmpp/Makefile (4)
M src/modules/regex/Makefile (2)
M src/modules/rls/Makefile (4)
M src/modules/slack/Makefile (2)
M src/modules/systemdops/Makefile (2)
M src/modules/tls/Makefile (4)
M src/modules/tlsa/Makefile (4)
M src/modules/topos_redis/Makefile (2)
M src/modules/utils/Makefile (4)
M src/modules/uuid/Makefile (2)
M src/modules/websocket/Makefile (2)
M src/modules/xcap_server/Makefile (4)
M src/modules/xhttp_pi/Makefile (4)
M src/modules/xmlops/Makefile (4)
M src/modules/xmlrpc/Makefile (4)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3475.patchhttps://github.com/kamailio/kamailio/pull/3475.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3475
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3475(a)github.com>
Actual implementation doesn't work with async. Need to investigate how to support it
---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/684
### Description
We have updated one system from a Ubuntu Focal 5.6.x container to a Ubuntu Jammy 5.7.2 container. In the last 24 hours, we saw three crashes. The logs of the last crash say:
```
Nov 09 20:39:34 sipproxy /usr/sbin/kamailio[4332]: CRITICAL: <core> [core/mem/q_malloc.c:519]: qm_free(): BUG: freeing already freed pointer (0x7f[4434056180](tel:4434056180)), called from tls: tls_init.c: ser_free>
Nov 09 20:39:34 sipproxy /usr/sbin/kamailio[4331]: CRITICAL: <core> [core/mem/q_malloc.c:123]: qm_debug_check_frag(): BUG: qm: fragm. 0x7f[44340561](tel:44340561)d0 (address 0x7f[4434056208](tel:4434056208)) beginning overwritten >
Nov 09 20:39:43 sipproxy /usr/sbin/kamailio[4273]: ALERT: <core> [main.c:776]: handle_sigs(): child process 4331 exited by a signal 6
Nov 09 20:39:43 sipproxy /usr/sbin/kamailio[4273]: ALERT: <core> [main.c:779]: handle_sigs(): core was generated
```
#### Debugging Data
Backtraces of the three crashes are attached.
[kamailio-crash1.log](https://github.com/kamailio/kamailio/files/13313115/ka…
[kamailio-crash2.log](https://github.com/kamailio/kamailio/files/13313116/ka…
[kamailio-crash3.log](https://github.com/kamailio/kamailio/files/13313117/ka…
### Additional Information
We are using the official Kamailio debian packages.
* **Kamailio Version** - output of `kamailio -v`
```
version: kamailio 5.7.2 (x86_64/linux)
flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, MEM_JOIN_FREE, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLOCKLIST, HAVE_RESOLV_RES, TLS_PTHREAD_MUTEX_SHARED
ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB
poll method support: poll, epoll_lt, epoll_et, sigio_rt, select.
id: unknown
compiled with gcc 11.4.0
```
* **Operating System**:
<!--
Details about the operating system, the type: Linux (e.g.,: Debian 8.4, Ubuntu 16.04, CentOS 7.1, ...), MacOS, xBSD, Solaris, ...;
Kernel details (output of `lsb_release -a` and `uname -a`)
-->
```
root@sipproxy:/WORKSPACE/sipproxy-container# lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description: Ubuntu 22.04.3 LTS
Release: 22.04
Codename: jammy
root@sipproxy:/WORKSPACE/sipproxy-container# uname -a
Linux sipproxy 5.15.0-87-generic #97-Ubuntu SMP Mon Oct 2 21:09:21 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
```
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3635
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3635(a)github.com>
Module: kamailio
Branch: master
Commit: 569b536404afd855742a42320d7c858a8bb4952b
URL: https://github.com/kamailio/kamailio/commit/569b536404afd855742a42320d7c858…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2024-01-22T14:11:29+01:00
tm: mark request with final reply flag
- send error function checks flag and returns if set to avoid sending
more than one final reply in case of failure
---
Modified: src/modules/tm/t_reply.c
Modified: src/modules/tm/tm.c
---
Diff: https://github.com/kamailio/kamailio/commit/569b536404afd855742a42320d7c858…
Patch: https://github.com/kamailio/kamailio/commit/569b536404afd855742a42320d7c858…
---
diff --git a/src/modules/tm/t_reply.c b/src/modules/tm/t_reply.c
index f52ecc7d3ad..e012dc29c28 100644
--- a/src/modules/tm/t_reply.c
+++ b/src/modules/tm/t_reply.c
@@ -640,6 +640,11 @@ static int _reply_light(struct cell *trans, char *buf, unsigned int len,
* (timer_allow_del()) (there's no chance of having the wait handler
* executed while we still need t) --andrei */
put_on_wait(trans);
+
+ /* mark the request with final reply flag */
+ if(trans->uas.request != NULL) {
+ trans->uas.request->msg_flags |= FL_FINAL_REPLY;
+ }
}
pkg_free(buf);
LM_DBG("finished\n");
diff --git a/src/modules/tm/tm.c b/src/modules/tm/tm.c
index 0be958a0848..0147bf44579 100644
--- a/src/modules/tm/tm.c
+++ b/src/modules/tm/tm.c
@@ -1608,6 +1608,11 @@ int ki_t_reply_error(sip_msg_t *msg)
int sip_err;
int ret;
+ if(msg->msg_flags & FL_FINAL_REPLY) {
+ LM_INFO("message marked with final-reply flag\n");
+ return -2;
+ }
+
ret = err2reason_phrase(
prev_ser_error, &sip_err, err_buffer, sizeof(err_buffer), "TM");
if(ret > 0) {
Module: kamailio
Branch: master
Commit: 0a11489d4b699a62596b2c4626171e8d92e966fd
URL: https://github.com/kamailio/kamailio/commit/0a11489d4b699a62596b2c4626171e8…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2024-01-21T10:20:39+01:00
lib/srdb2: review and adapt old comments
---
Modified: src/lib/srdb2/db_cmd.c
Modified: src/lib/srdb2/db_gen.h
---
Diff: https://github.com/kamailio/kamailio/commit/0a11489d4b699a62596b2c4626171e8…
Patch: https://github.com/kamailio/kamailio/commit/0a11489d4b699a62596b2c4626171e8…
---
diff --git a/src/lib/srdb2/db_cmd.c b/src/lib/srdb2/db_cmd.c
index 2782f4dc7c5..9ba1d22ee66 100644
--- a/src/lib/srdb2/db_cmd.c
+++ b/src/lib/srdb2/db_cmd.c
@@ -58,7 +58,7 @@ db_cmd_t *db_cmd(enum db_cmd_type type, db_ctx_t *ctx, char *table,
newp->type = type;
- /** FIXME: it is not clear now that this is necessary
+ /** NOTE: it is not clear now that this is necessary
* when we have match and value separate arrays */
if(result) {
newp->result = db_fld_copy(result);
@@ -78,7 +78,7 @@ db_cmd_t *db_cmd(enum db_cmd_type type, db_ctx_t *ctx, char *table,
goto err;
}
- /* FIXME: This should be redesigned so that we do not need to connect
+ /* NOTE: This should be redesigned so that we do not need to connect
* connections in context before commands are created, this takes splitting
* the command initialization sequence in two steps, one would be creating
* all the data structures and the second would be checking corresponding
@@ -242,7 +242,6 @@ int db_exec(db_res_t **res, db_cmd_t *cmd)
return -1;
}
- /* FIXME */
db_payload_idx = 0;
ret = cmd->exec[0](r, cmd);
if(ret < 0) {
diff --git a/src/lib/srdb2/db_gen.h b/src/lib/srdb2/db_gen.h
index eca24c76046..f8dd483b87a 100644
--- a/src/lib/srdb2/db_gen.h
+++ b/src/lib/srdb2/db_gen.h
@@ -95,7 +95,7 @@ extern "C"
*/
/*
- * FIXME: We should find some other way of doing this than just copying
+ * NOTE: We should find some other way of doing this than just copying
* and pasting the code from STAILQ_FOREACH
*/
#define DBLIST_FOREACH(var, head) \
@@ -111,7 +111,7 @@ extern "C"
*/
/*
- * FIXME: We should find some other way of doing this than just copying
+ * NOTE: We should find some other way of doing this than just copying
* and pasting the code from STAILQ_FOREACH_SAFE
*/
#define DBLIST_FOREACH_SAFE(var, head, tvar) \
Hello.
I have installed Kamailio version devel on Ubuntu 22.04 and use the mysql database on the same machine.
when i run
systemctl status kamailio.service
● kamailio.service - LSB: Start the Kamailio SIP proxy server
Loaded: loaded (/etc/init.d/kamailio; generated)
Active: active (exited) since Sat 2024-01-20 11:35:09 +0330; 21min ago
Docs: man:systemd-sysv-generator(8)
CPU: 48ms
faraz /usr/local/sbin/kamailio[173481]: ERROR: usrloc [dlist.c:846]: register_udomain(): failed to open database connection
faraz /usr/local/sbin/kamailio[173481]: ERROR: registrar [registrar.c:767]: domain_fixup(): failed to register domain
faraz /usr/local/sbin/kamailio[173481]: ERROR: <core> [core/route.c:1193]: fix_actions(): fixing failed (code=-1) at cfg:/usr/local/etc/kamailio/kamailio.cfg:717
faraz /usr/local/sbin/kamailio[173481]: ERROR: <core> [core/rvalue.c:3818]: fix_rval_expr(): failure in cfg at line: 717 col: 22
faraz /usr/local/sbin/kamailio[173481]: ERROR: <core> [core/rvalue.c:3818]: fix_rval_expr(): failure in cfg at line: 717 col: 22
faraz /usr/local/sbin/kamailio[173481]: ERROR: <core> [core/route.c:1193]: fix_actions(): fixing failed (code=-1) at cfg:/usr/local/etc/kamailio/kamailio.cfg:720
faraz /usr/local/sbin/kamailio[173481]: INFO: <core> [core/sctp_core.c:53]: sctp_core_destroy(): SCTP API not initialized
faraz kamailio[173463]: * already running
faraz kamailio[173463]: ...done.
faraz systemd[1]: Started LSB: Start the Kamailio SIP proxy server.
In /usr/local/etc/kamailio/kamctlrc, I uncomment these lines:
SIP_DOMAIN=ims.mnc55.mcc999.farazNET.org
DBENGINE=MYSQL
DBHOST=localhost
in /usr/local/etc/kamailio/kamailio.cfg i apply this changes:
auto_aliases=no
alias="ims.mnc55.mcc999.farazNET.org"
listen=udp:192.168.75.185:5060
listen=tcp:192.168.75.185:5060
how can i solved it?
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3721
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3721(a)github.com>
Hello. I am trying to install Kamailio branch devel for using Volte on open5gs.
when compiled have this error:
/bin/sh: 3: ./autogen.sh: not found
env: ‘./configure’: No such file or directory
make[2]: *** [Makefile:28: ../../../misc/external/wolfssl/build/include/wolfssl/options.h] Error 127
make[2]: Leaving directory '/usr/local/src/kamailio-devel/kamailio/src/modules/tls_wolfssl'
make[1]: *** [Makefile:508: modules] Error 1
make[1]: Leaving directory '/usr/local/src/kamailio-devel/kamailio/src'
make: *** [Makefile:34: all] Error 2
I answer my question also on the user mailing list but no one reply me.
It is noticeable that I installed libwolfssl-dev.
Thanks
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3712
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3712(a)github.com>
The dialog module will expire dialogs that don't actually belong to the particular proxy.
Scenario: 2 proxies on 5.2.4 with a loadbalancer in front. Call was routed over proxy1, dialog timeout on proxy2 was shorter (to make it easier to reproduce it). The results:
* Dialog synchronized with DMQ: proxy 2 would timeout the dialog
* Dialog entry synchronized to proxy2 DB and proxy2 restarted: Proxy2 would timeout the dialog
* Dialog entry synchronized to proxy2 DB and proxy2 not restarted: Proxy2 would _not_ timeout the dialog (I’ve tried both db_mode 1 and 2)
So the proxy2 would load the dialog into memory and then later expire it. So timeout routes, send_bye on timeout would not work correctly.
One idea to solve this would be to check e.g. over the existing socket data if the dialog belongs to this proxy and skip loading it in this case. This could be also made configurable e.g. with a module parameter.
Any comments? Better ideas how to solve it?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/2080
<!-- 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 -->
Add kemi xlog function to be able to log in other facilities other than the default one. Can this be backported to 5.6 and 5.7?
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3708
-- Commit Summary --
* xlog: add kemi xlog_facility function
-- File Changes --
M src/modules/xlog/xlog.c (51)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3708.patchhttps://github.com/kamailio/kamailio/pull/3708.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3708
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3708(a)github.com>
### Description & troubleshooting
I have defined a redis cluster as followed. The cluster consists of 3 masters (port 6379) and 3 slaves (port 6380).
The password is identical on all nodes through redis option 'requirepass'.
```
modparam("ndb_redis", "server", "name=sbcredis;addr=SBCMASTERREDISIP;port=6379;db=0;pass=REDACTED")
modparam("ndb_redis", "server", "name=sbcredis;addr=PRIVATEIP;port=6380;db=0;pass=REDACTED")
modparam("ndb_redis", "server", "name=sbcredis;addr=ASTP5IP;port=6380;db=0;pass=REDACTED")
modparam("ndb_redis", "server", "name=sbcredis;addr=ASTP6IP;port=6379;db=0;pass=REDACTED")
modparam("ndb_redis", "server", "name=sbcredis;addr=ASTP3IP;port=6380;db=0;pass=REDACTED")
modparam("ndb_redis", "server", "name=sbcredis;addr=ASTP4IP;port=6379;db=0;pass=REDACTED")
modparam("ndb_redis", "cluster", 1)
modparam("ndb_redis", "allow_dynamic_nodes", 1)
modparam("ndb_redis", "debug", 3)
```
When executing the redis_cmd command, most of the times an expected warning is shown due to another server providing the requested slot:
`"(error) MOVED ... 192.168.136.241:6380"`
Notice that I've enabled "allow_dynamic_nodes" to make sure there is an automated connection attempt to the indicated server.
However, it appears that sometimes a "NOAUTH Authentication required." error is thrown when there is a redirection to a slave server serving that slot. I can authenticate succesfull on the redis-cli when using the identical password though.
Also note that when a redis master is queried, the redis_cmd is successful.
#### Reproduction
Each simple redis command that needs authentication (GET, SET, ...)
`redis_cmd("sbcredis", "SET TESTKEY TESTVALUE", "r");`
#### Log Messages
```
ndb_redis [redis_client.c:1089]: redisc_exec(): Redis error:NOAUTH Authentication required.
```
### Possible Solutions
None for now except disabling the password on all nodes.
### Additional Information
```
kamailio 5.7.2
```
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3654
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3654(a)github.com>
Module: kamailio
Branch: master
Commit: 304b8a41464f1bed83d733a3002088b759c7e679
URL: https://github.com/kamailio/kamailio/commit/304b8a41464f1bed83d733a3002088b…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2024-01-18T16:01:19+01:00
modules: readme files regenerated - kex ... [skip ci]
---
Modified: src/modules/kex/README
---
Diff: https://github.com/kamailio/kamailio/commit/304b8a41464f1bed83d733a3002088b…
Patch: https://github.com/kamailio/kamailio/commit/304b8a41464f1bed83d733a3002088b…
---
diff --git a/src/modules/kex/README b/src/modules/kex/README
index 55d341211d0..d0661e611ed 100644
--- a/src/modules/kex/README
+++ b/src/modules/kex/README
@@ -47,9 +47,10 @@ Ovidiu Sas
3.9. isdsturiset()
3.10. pv_printf(var, str)
3.11. is_myself(uri)
- 3.12. setdebug(level)
- 3.13. resetdebug()
- 3.14. km_append_branch([uri])
+ 3.12. is_myhost(uri)
+ 3.13. setdebug(level)
+ 3.14. resetdebug()
+ 3.15. km_append_branch([uri])
4. RPC Commands
@@ -102,8 +103,9 @@ Ovidiu Sas
1.9. isdsturiset usage
1.10. pv_printf usage
1.11. is_myself usage
- 1.12. setdebug usage
- 1.13. resetdebug usage
+ 1.12. is_myhost usage
+ 1.13. setdebug usage
+ 1.14. resetdebug usage
Chapter 1. Admin Guide
@@ -128,9 +130,10 @@ Chapter 1. Admin Guide
3.9. isdsturiset()
3.10. pv_printf(var, str)
3.11. is_myself(uri)
- 3.12. setdebug(level)
- 3.13. resetdebug()
- 3.14. km_append_branch([uri])
+ 3.12. is_myhost(uri)
+ 3.13. setdebug(level)
+ 3.14. resetdebug()
+ 3.15. km_append_branch([uri])
4. RPC Commands
@@ -206,9 +209,10 @@ Chapter 1. Admin Guide
3.9. isdsturiset()
3.10. pv_printf(var, str)
3.11. is_myself(uri)
- 3.12. setdebug(level)
- 3.13. resetdebug()
- 3.14. km_append_branch([uri])
+ 3.12. is_myhost(uri)
+ 3.13. setdebug(level)
+ 3.14. resetdebug()
+ 3.15. km_append_branch([uri])
3.1. setsflag(flag)
@@ -391,7 +395,8 @@ pv_printf("$avp(x)", "From: $fU - To: $tU");
Meaning of the parameters is as follows:
* uri - Valid SIP URI or IP address to check against the list of
- local IP addresses or domains. The parameter value can contain
+ local IP addresses or domains, matching as well the port and
+ protocol if they are provided. The parameter value can contain
pseudo-variables.
This function can be used from ANY_ROUTE.
@@ -403,7 +408,27 @@ if(is_myself("$fu")) {
}
...
-3.12. setdebug(level)
+3.12. is_myhost(uri)
+
+ Check if the host part of the parameter matches a local domain or IP
+ address.
+
+ Meaning of the parameters is as follows:
+ * uri - Valid SIP URI, hostname of IP address to check against the
+ list of local IP addresses or domains. If it is a SIP URI, the port
+ and protocol are ignored. The parameter value can contain
+ pseudo-variables.
+
+ This function can be used from ANY_ROUTE.
+
+ Example 1.12. is_myhost usage
+...
+if(is_myhost("$tu")) {
+ ...
+}
+...
+
+3.13. setdebug(level)
Set the debug log level per process.
@@ -413,7 +438,7 @@ if(is_myself("$fu")) {
This function can be used from ANY_ROUTE.
- Example 1.12. setdebug usage
+ Example 1.13. setdebug usage
...
setdebug("1");
...
@@ -421,19 +446,19 @@ $var(level) = 2;
setdebug("$var(level)");
...
-3.13. resetdebug()
+3.14. resetdebug()
Reset the local debug log level back to the value of core parameter
'debug'.
This function can be used from ANY_ROUTE.
- Example 1.13. resetdebug usage
+ Example 1.14. resetdebug usage
...
resetdebug();
...
-3.14. km_append_branch([uri])
+3.15. km_append_branch([uri])
This function was replaced by append_branch() from corex module,
starting with version 4.0.0.
Module: kamailio
Branch: master
Commit: ee191ccd01ec6391c715e22efed2dc6581fe9d72
URL: https://github.com/kamailio/kamailio/commit/ee191ccd01ec6391c715e22efed2dc6…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2024-01-18T09:29:21+01:00
kex: docs for is_myhost()
---
Modified: src/modules/kex/doc/kex_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/ee191ccd01ec6391c715e22efed2dc6…
Patch: https://github.com/kamailio/kamailio/commit/ee191ccd01ec6391c715e22efed2dc6…
---
diff --git a/src/modules/kex/doc/kex_admin.xml b/src/modules/kex/doc/kex_admin.xml
index 91080d8c1db..9f99e6a6bc4 100644
--- a/src/modules/kex/doc/kex_admin.xml
+++ b/src/modules/kex/doc/kex_admin.xml
@@ -360,7 +360,8 @@ pv_printf("$avp(x)", "From: $fU - To: $tU");
<listitem>
<para>
<emphasis>uri</emphasis> - Valid SIP URI or IP address to
- check against the list of local IP addresses or domains.
+ check against the list of local IP addresses or domains, matching
+ as well the port and protocol if they are provided.
The parameter value can contain pseudo-variables.
</para>
</listitem>
@@ -376,6 +377,37 @@ if(is_myself("$fu")) {
...
}
...
+</programlisting>
+ </example>
+ </section>
+ <section id="kex.f.is_myhost">
+ <title><function moreinfo="none">is_myhost(uri)</function></title>
+ <para>
+ Check if the host part of the parameter matches a local domain
+ or IP address.
+ </para>
+ <para>Meaning of the parameters is as follows:</para>
+ <itemizedlist>
+ <listitem>
+ <para>
+ <emphasis>uri</emphasis> - Valid SIP URI, hostname of IP address to
+ check against the list of local IP addresses or domains. If it
+ is a SIP URI, the port and protocol are ignored.
+ The parameter value can contain pseudo-variables.
+ </para>
+ </listitem>
+ </itemizedlist>
+ <para>
+ This function can be used from ANY_ROUTE.
+ </para>
+ <example>
+ <title><function>is_myhost</function> usage</title>
+ <programlisting format="linespecific">
+...
+if(is_myhost("$tu")) {
+ ...
+}
+...
</programlisting>
</example>
</section>
@@ -992,4 +1024,3 @@ Module: kex
</section>
</chapter>
-
<!-- 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
- [ ] Commits are split per component (core, individual modules, libs, utils, ...)
- [ ] 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
- [ ] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
Add support for LoongArch.The LoongArch architecture (LoongArch) is an Instruction Set Architecture (ISA) that has a RISC style.
Documentations:
ISA:
https://loongson.github.io/LoongArch-Documentation/LoongArch-Vol1-EN.html
ABI:
https://loongson.github.io/LoongArch-Documentation/LoongArch-ELF-ABI-EN.html
More docs can be found at:
https://loongson.github.io/LoongArch-Documentation/README-EN.html
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3715
-- Commit Summary --
* Add support for LoongArch.
-- File Changes --
M src/modules/app_jsdt/duk_config.h (11)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3715.patchhttps://github.com/kamailio/kamailio/pull/3715.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3715
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3715(a)github.com>
Hello,
Kamailio SIP Server v5.7.4 stable release is out.
This is a maintenance release of the latest stable branch, 5.7, that
includes fixes since the release of v5.7.3. There is no change to
database schema or configuration language structure that you have to do
on previous installations of v5.7.x. Deployments running previous v5.7.x
versions are strongly recommended to be upgraded to v5.7.4.
For more details about version 5.7.4 (including links and guidelines to
download the tarball or from GIT repository), visit:
* https://www.kamailio.org/w/2024/01/kamailio-v5-7-4-released/
RPM, Debian/Ubuntu packages will be available soon as well.
Many thanks to all contributing and using Kamailio!
Cheers,
Daniel
--
Daniel-Constantin Mierla (@ asipto.com)
twitter.com/miconda -- linkedin.com/in/miconda
Kamailio Consultancy, Training and Development Services -- asipto.com
Kamailio Advanced Training, February 20-22, 2024 -- asipto.com
Kamailio World Conference, April 18-19, 2024, Berlin -- kamailioworld.com
Hi devs,
The CI/CD seems to be broken after I pushed a few commits to master.
There seem to be docker pull errors.
Appreciate if someone could take a look.
Regards
Shih-Ping