>From https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=906233
> In the current version missing command line options for the
read-only psql call (in pgsql_ro_query) prevents usage of
kamctl for PostgreSQL backend, e.g. when adding a _new_ user:
> root@debian9 ~ # kamctl add phone_u phone_p
database engine 'PGSQL' loaded
Control engine 'FIFO' loaded
is_user: user counter=1
INFO: user 'phone_u' already exists
> This is because the psql query returns the correct result
(0 in this case) but the pgsql_ro_query function lacks the
psql options -A -q -t as defined.
>As a result the query contains the result and the coloumn name and
footer,e.g. in this case:
```
root@debian9 ~ # /usr/bin/psql -h localhost -U kamailioro kamailio \
-c "select count(*) from subscriber where username='phone_u';"
count
-------
0
(1 row)
```
>In this example the "kamctl add" command looks at the last line of the
output to check if the user is already defined and misinterprets the
result as it expectes "0" as the result.
>This affect not only the "kamctl add" command but most calls that calls
$DBROCMD in kamctl.
--
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/1750
Thanks Andreas Maus <maus+debianbug(a)ypbind.de>
<!-- 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
- [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
- [X] Related to issue #1750
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1751
-- Commit Summary --
* utils/kamctl: [pgsql] add missing options for pgsql_ro_query()
-- File Changes --
M utils/kamctl/kamctl.pgsql (4)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1751.patchhttps://github.com/kamailio/kamailio/pull/1751.diff
--
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/pull/1751
Module: kamailio
Branch: master
Commit: 39b89a18a8c357151a173ab02dc95dff1f02715d
URL: https://github.com/kamailio/kamailio/commit/39b89a18a8c357151a173ab02dc95df…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2018-11-30T16:05:30+01:00
tm: reply_received() - simplify locking for processing sip response
- leverage the recursive mutex and skip several zones of unlock/lock,
which can lead to races on delayed processing or fast reply
retransmissions
- related to GH #1613 #1744
---
Modified: src/modules/tm/t_reply.c
---
Diff: https://github.com/kamailio/kamailio/commit/39b89a18a8c357151a173ab02dc95df…
Patch: https://github.com/kamailio/kamailio/commit/39b89a18a8c357151a173ab02dc95df…
---
diff --git a/src/modules/tm/t_reply.c b/src/modules/tm/t_reply.c
index 8a95fe758d..859c73cd37 100644
--- a/src/modules/tm/t_reply.c
+++ b/src/modules/tm/t_reply.c
@@ -2189,7 +2189,7 @@ int reply_received( struct sip_msg *p_msg )
#ifdef WITH_XAVP
sr_xavp_t **backup_xavps;
#endif
- int replies_locked;
+ int replies_locked = 0;
#ifdef USE_DNS_FAILOVER
int branch_ret;
int prev_branch;
@@ -2223,10 +2223,15 @@ int reply_received( struct sip_msg *p_msg )
/* if transaction found, increment the rpl_received counter */
t_stats_rpl_received();
+ /* lock -- onreply_route, safe avp usage, ... */
+ /* - it is a recurrent mutex, so it is safe if a function executed
+ * down here does another lock/unlock */
+ LOCK_REPLIES( t );
+ replies_locked=1;
+
tm_ctx_set_branch_index(branch);
init_cancel_info(&cancel_data);
msg_status=p_msg->REPLY_STATUS;
- replies_locked=0;
uac=&t->uac[branch];
LM_DBG("org. status uas=%d, uac[%d]=%d local=%d is_invite=%d)\n",
@@ -2363,11 +2368,6 @@ int reply_received( struct sip_msg *p_msg )
/* processing of on_reply block */
if (onreply_route) {
set_route_type(TM_ONREPLY_ROUTE);
-
- /* lock onreply_route, for safe avp usage */
- LOCK_REPLIES( t );
- replies_locked=1;
-
/* transfer transaction flag to message context */
if (t->uas.request) {
p_msg->flags=t->uas.request->flags;
@@ -2444,27 +2444,12 @@ int reply_received( struct sip_msg *p_msg )
if (unlikely((ctx.run_flags&DROP_R_F) && (msg_status<200)))
#endif /* TM_ONREPLY_FINAL_DROP_OK */
{
- if (likely(replies_locked)) {
- replies_locked = 0;
- UNLOCK_REPLIES( t );
- }
goto done;
}
#ifdef TM_ONREPLY_FINAL_DROP_OK
if (msg_status >= 200) {
/* stop final reply timers, now that we executed the onreply route
* and the reply was not DROPed */
- if (likely(replies_locked)){
- /* if final reply => we have to execute stop_rb_timers,
- * but with replies unlocked to avoid a possible deadlock
- * (if the timer is currently running, stop_rb_timers()
- * will wait until the timer handler ends, but the
- * final_response_handler() will try to lock replies
- * => deadlock).
- */
- UNLOCK_REPLIES( t );
- replies_locked=0;
- }
stop_rb_timers(&uac->request);
}
#endif /* TM_ONREPLY_FINAL_DROP_OK */
@@ -2520,11 +2505,6 @@ int reply_received( struct sip_msg *p_msg )
branch_ret=add_uac_dns_fallback(t, t->uas.request,
uac, !replies_locked);
prev_branch=-1;
- /* unlock replies to avoid sending() while holding a lock */
- if (unlikely(replies_locked)) {
- UNLOCK_REPLIES( t );
- replies_locked = 0;
- }
while((branch_ret>=0) &&(branch_ret!=prev_branch)){
prev_branch=branch_ret;
branch_ret=t_send_branch(t, branch_ret, t->uas.request , 0, 1);
@@ -2533,12 +2513,8 @@ int reply_received( struct sip_msg *p_msg )
#endif
if (unlikely(p_msg->msg_flags&FL_RPL_SUSPENDED)) {
- goto skip_send_reply;
- /* suspend the reply (async), no error */
- }
- if (unlikely(!replies_locked)){
- LOCK_REPLIES( t );
- replies_locked=1;
+ /* suspended the reply (async) - no error */
+ goto done;
}
if ( is_local(t) ) {
/* local_reply() does UNLOCK_REPLIES( t ) */
@@ -2596,18 +2572,16 @@ int reply_received( struct sip_msg *p_msg )
uac->request.flags|=F_RB_FR_INV; /* mark fr_inv */
} /* provisional replies */
-skip_send_reply:
-
- if (likely(replies_locked)){
- /* unlock replies if still locked coming via goto skip_send_reply */
+done:
+ if (unlikely(replies_locked)){
+ /* unlock replies if still locked coming via goto */
UNLOCK_REPLIES(t);
replies_locked=0;
}
-done:
tm_ctx_set_branch_index(T_BR_UNDEFINED);
- /* we are done with the transaction, so unref it - the reference
- * was incremented by t_check() function -bogdan*/
+ /* done processing the transaction, so unref it
+ * - the reference counter was incremented by t_check() function */
t_unref(p_msg);
/* don't try to relay statelessly neither on success
* (we forwarded statefully) nor on error; on troubles,
<!--
Kamailio Project uses GitHub Issues only for bugs in the code or feature requests. Please use this template only for bug reports.
If you have questions about using Kamailio or related to its configuration file, ask on sr-users mailing list:
* http://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
If you have questions about developing extensions to Kamailio or its existing C code, ask on sr-dev mailing list:
* http://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
Please try to fill this template as much as possible for any issue. It helps the developers to troubleshoot the issue.
If there is no content to be filled in a section, the entire section can be removed.
You can delete the comments from the template sections when filling.
You can delete next line and everything above before submitting (it is a comment).
-->
### Description
<!--
Explain what you did, what you expected to happen, and what actually happened.
-->
I tested kamailio's topoh+topos modules. udp worker's private memory is leaking when i enable both.
after a couple of hour kamailio stops and spams error messages.
### Troubleshooting
kamcmd pkg.stats shows lots of real_used memory(only udp workers are leaking).
#### Reproduction
Enable both modules.
<!--
If the issue can be reproduced, describe how it can be done.
-->
#### Debugging Data
<!--
If you got a core dump, use gdb to extract troubleshooting data - full backtrace,
local variables and the list of the code at the issue location.
gdb /path/to/kamailio /path/to/corefile
bt full
info locals
list
If you are familiar with gdb, feel free to attach more of what you consider to
be relevant.
-->
#### Log Messages
<!--
Check the syslog file and if there are relevant log messages printed by Kamailio, add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site).
-->
```
Oct 18 17:00:55 /usr/sbin/kamailio[20801]: ERROR: <core> [core/receive.c:162]: receive_msg(): no mem for sip_msg
Oct 18 17:00:55 /usr/sbin/kamailio[20802]: ERROR: <core> [core/mem/q_malloc.c:291]: qm_find_free(): qm_find_free(0x7fe6f5bee010, 1776); Free fragment not found!
Oct 18 17:00:55 /usr/sbin/kamailio[20802]: ERROR: <core> [core/mem/q_malloc.c:425]: qm_malloc(): qm_malloc(0x7fe6f5bee010, 1776) called from core: core/receive.c: receive_msg(160), module: core; Free fragment not found!
```
<!--
If the issue is exposed by processing specific SIP messages, grab them with ngrep or save in a pcap file, then add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site).
-->
### Possible Solutions
<!--
If you found a solution or workaround for the issue, describe it. Ideally, provide a pull request with a fix.
-->
disable one of modules
### Additional Information
* **/etc/default/kamailio**
```
SHM_MEMORY=512
PKG_MEMORY=64
```
Number of children = 8
* **Parameters**:
```
loadmodule "topoh.so"
modparam("topoh", "mask_key", "N2hkVvBTxJ6Y")
modparam("topoh", "mask_callid", 0)
modparam("topoh", "sanity_checks", 1)
modparam("topoh", "uri_prefix_checks", 1)
modparam("topoh", "mask_ip", MY_IP_ADDR)
loadmodule "topos.so"
loadmodule "topos_redis.so"
modparam("topos", "storage", "redis")
modparam("topos", "branch_expire", 300)
modparam("topos", "dialog_expire", 10800)
modparam("topos", "clean_interval", 30)
modparam("topos_redis", "serverid", "topo")
```
* **Kamailio Version** - output of `kamailio -v`
```
# kamailio -v
version: kamailio 5.1.6 (x86_64/linux)
flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MEM, SHM_MMAP, PKG_MALLOC, 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_BLACKLIST, HAVE_RESOLV_RES
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 4.9.2
```
* **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 `uname -a`)
-->
```
Debian 8.10
```
--
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/1682
### Description
1. Added 3 gateways to the dispatcher list file:
```
1 sip:naptr.mydomain.net 0 0
2 sip:srv.mydomain.net 0 0
```
started kamailio and ran: ```kamcmd dispatcher.list```
Output: ```error: 500 - No Destination Sets```
If I append to the list, one gateway which resolves to an A record (or an IP:PORT format) like e.g.:
```3 sip:a.mydomain.net:5060 0 0```, it shows up in the dispatcher list just fine:
```
{
NRSETS: 1
RECORDS: {
SET: {
ID: 3
TARGETS: {
DEST: {
URI: sip:a.mydomain.net:5060
FLAGS: AP
PRIORITY: 0
LATENCY: {
AVG: 14.642
STD: 0.05
EST: 15.0345
MAX: 25.452
TIMEOUT: 0
}
}
}
}
}
}
```
I also verified the DNS resolution with: ```kamcmd dns.lookup naptr naptr.mydomain.net``` and ```kamcmd dns.lookup srv srv.mydomain.net``` which was perfectly as expected.
Since I wasn't even able to see a dispatcher loaded, there was no operation that I could do to it to set it to active or anything else.
A network trace just shows A record queries to all gateways with no success on those which have NAPTR/SRV records.
#### Reproduction
100% reproducible with the description above.
### Additional Information
Tested on Kamailio version 4.4.5 and 5.1.1 with the following detailed version info:
```
version: kamailio 5.1.1 (x86_64/linux)
flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MEM, SHM_MMAP, PKG_MALLOC, 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_BLACKLIST, HAVE_RESOLV_RES
ADAPTIVE_WAIT_LOOPS=1024, MAX_RECV_BUFFER_SIZE 262144, MAX_LISTEN 16, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB
poll method support: poll, epoll_lt, epoll_et, sigio_rt, select.
compiled with /usr/bin/gcc 4.1.2
```
--
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/1743
<!-- Kamailio Pull Request Template -->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [x] Commit message has the format required by CONTRIBUTING guide
- [x] Commits are split per component (core, individual modules, libs, utils, ...)
- [x] Each component has a single commit (if not, squash them into one commit)
- [x] No commits to README files for modules (changes must be done to docbook files
in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change
- [ ] 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 -->
- [ ] 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 -->
added a new function as get_regsub_string for getting substring with regular expression
-regexec() function with REG_EXTENDED flags is used
-setting variable as result
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1733
-- Commit Summary --
* textops : added a new function for getting substring with regular expression
* textops: doc is added for get_regsub_string
-- File Changes --
M src/modules/textops/doc/textops_admin.xml (147)
M src/modules/textops/textops.c (122)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1733.patchhttps://github.com/kamailio/kamailio/pull/1733.diff
--
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/pull/1733
<!-- 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 -->
- [ ] Commit message has the format required by CONTRIBUTING guide
- [ ] Commits are split per component (core, individual modules, libs, utils, ...)
- [ ] Each component has a single commit (if not, squash them into one commit)
- [ ] No commits to README files for modules (changes must be done to docbook files
in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1739
-- Commit Summary --
* Add files via upload
* Merge pull request #1 from thulasizwe-n/thulasizwe-n-patch-1
* fixed possible list corruption
* cdp -- lockup fix
-- File Changes --
A modules/routing.c (299)
A msrp_env.c (226)
A msrp_mod.c (505)
A msrp_netio.c (453)
A msrp_netio.h (42)
A msrp_options.c (75)
A msrp_options.h (21)
A ro_session_hash.h (244)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1739.patchhttps://github.com/kamailio/kamailio/pull/1739.diff
--
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/pull/1739
…ailio/kamailio/issues/1681
<!-- 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 -->
- [ ] Commit message has the format required by CONTRIBUTING guide
- [ ] Commits are split per component (core, individual modules, libs, utils, ...)
- [ ] Each component has a single commit (if not, squash them into one commit)
- [ ] No commits to README files for modules (changes must be done to docbook files
in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1706
-- Commit Summary --
* Allow startup with offline database. See issue https://github.com/kamailio/kamailio/issues/1681
-- File Changes --
M src/modules/sqlops/README (25)
M src/modules/sqlops/sqlops.c (34)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1706.patchhttps://github.com/kamailio/kamailio/pull/1706.diff
--
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/pull/1706
Module: kamailio
Branch: master
Commit: 1b014d65e26a99f9f5d4aea1b27949776f083c2d
URL: https://github.com/kamailio/kamailio/commit/1b014d65e26a99f9f5d4aea1b279497…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2018-11-30T09:19:16+01:00
lib/srdb1: memset db connection structure earlier in db_bind_mod()
---
Modified: src/lib/srdb1/db.c
---
Diff: https://github.com/kamailio/kamailio/commit/1b014d65e26a99f9f5d4aea1b279497…
Patch: https://github.com/kamailio/kamailio/commit/1b014d65e26a99f9f5d4aea1b279497…
---
diff --git a/src/lib/srdb1/db.c b/src/lib/srdb1/db.c
index 4a250bc819..7d53f70361 100644
--- a/src/lib/srdb1/db.c
+++ b/src/lib/srdb1/db.c
@@ -163,6 +163,12 @@ int db_bind_mod(const str* mod, db_func_t* mydbf)
LM_CRIT("null dbf parameter\n");
return -1;
}
+
+ /* for safety we initialize mydbf with 0 (this will cause
+ * a segfault immediately if someone tries to call a function
+ * from it without checking the return code from bind_dbmod */
+ memset((void*)mydbf, 0, sizeof(db_func_t));
+
if (mod->len > MAX_URL_LENGTH)
{
LM_ERR("SQL URL too long\n");
@@ -178,11 +184,6 @@ int db_bind_mod(const str* mod, db_func_t* mydbf)
memcpy(name+3, mod->s, mod->len);
name[mod->len+3] = 0;
- /* for safety we initialize mydbf with 0 (this will cause
- * a segfault immediately if someone tries to call a function
- * from it without checking the return code from bind_dbmod */
- memset((void*)mydbf, 0, sizeof(db_func_t));
-
p = strchr(name, ':');
if (p) {
len = p - name;
ims_charging : doc is fixed
<!-- 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
- [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 -->
Ro_CCR function doesnt get parameter like domain. it removed from doc.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1741
-- Commit Summary --
* ims_charging : doc is fixed
-- File Changes --
M src/modules/ims_charging/doc/ims_charging_admin.xml (7)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1741.patchhttps://github.com/kamailio/kamailio/pull/1741.diff
--
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/pull/1741
<!--
Kamailio Project uses GitHub Issues only for bugs in the code or feature requests. Please use this template only for feature requests.
If you have questions about using Kamailio or related to its configuration file, ask on sr-users mailing list:
* http://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
If you have questions about developing extensions to Kamailio or its existing C code, ask on sr-dev mailing list:
* http://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
Please try to fill this template as much as possible for any issue. It helps the developers to troubleshoot the issue.
If you submit a feature request (or enhancement) add the description of what you would like to be added.
If there is no content to be filled in a section, the entire section can be removed.
You can delete the comments from the template sections when filling.
You can delete next line and everything above before submitting (it is a comment).
-->
### Description
I used [get_body_part_by_filter](https://github.com/kamailio/kamailio/blob/f073f72a… to extract a body part from a `multipart/mixed` message. It works as expected but for every body part it compares and do not match (until finding the one that matches the filter) it shows error log messages like:
`ERROR: <core> [core/parser/parse_body.c:513]: part_multipart_headers_cmp(): part_multipart_headers_cmp. error. "-2". "MIME types mismatch".`
The comparison of body parts shouldn´t be logged as errors, since they are not.
<!--
Explain what you did, what you expected to happen, and what actually happened.
-->
### Expected behavior
No error logs.
#### Actual observed behavior
Error log messages.
#### Log Messages
<!--
Check the syslog file and if there are relevant log messages printed by Kamailio, add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site).
-->
```
ERROR: <core> [core/parser/parse_body.c:513]: part_multipart_headers_cmp(): part_multipart_headers_cmp. error. "-2". "MIME types mismatch".
```
### Possible Solutions
<!--
If you found a solution or workaround for the issue, describe it. Ideally, provide a pull request with a improvement.
-->
Do not show logs or change log level.
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
```
5.1
```
* **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 `uname -a`)
-->
```
CentOS 7
```
--
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/1746
Module: kamailio
Branch: master
Commit: ba31f4ec4f98487e9f1282fe12a5819d5df0b733
URL: https://github.com/kamailio/kamailio/commit/ba31f4ec4f98487e9f1282fe12a5819…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2018-11-29T18:31:07+01:00
dispatcher: docs for flags value 16
---
Modified: src/modules/dispatcher/doc/dispatcher_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/ba31f4ec4f98487e9f1282fe12a5819…
Patch: https://github.com/kamailio/kamailio/commit/ba31f4ec4f98487e9f1282fe12a5819…
---
diff --git a/src/modules/dispatcher/doc/dispatcher_admin.xml b/src/modules/dispatcher/doc/dispatcher_admin.xml
index 023f6a0ba2..3b40c04622 100644
--- a/src/modules/dispatcher/doc/dispatcher_admin.xml
+++ b/src/modules/dispatcher/doc/dispatcher_admin.xml
@@ -1780,17 +1780,21 @@ onreply_route {
sending keepalives. It is a bitwise value that can be built using
the folowing flags:
<itemizedlist>
- <listitem><para>1 (bit at index 0 - 1 <<0) - inactive destination<para>
+ <listitem><para>1 (bit at index 0 - 1 <<0) - inactive destination</para>
</listitem>
<listitem><para>2 (bit at index 1 - 1 <<1) - temporary trying
destination (in the way to become inactive if it does not reply to
keepalives - there is a module parameter to set the threshold of
- failures)<para>
+ failures)</para>
</listitem>
- <listitem><para>4 (bit at index 2 - 1 <<2) - admin disabled destination<para>
+ <listitem><para>4 (bit at index 2 - 1 <<2) - admin disabled destination</para>
</listitem>
<listitem><para>8 (bit at index 3 - 1 <<3) - probing destination (sending keep alives);</para>
</listitem>
+ <listitem><para>16 (bit at index 4 - 1 <<4) - skip DNS A/AAAA resolve at startup,
+ useful when the hostname of the destination address is a NAPTR or SRV record only.
+ Such addresses cannot be matched anymore with ds_is_from_list(...).</para>
+ </listitem>
</itemizedlist>
</para>
</listitem>
Module: kamailio
Branch: master
Commit: 9eba5b673cc1a80434642b280e6f7d53e310338e
URL: https://github.com/kamailio/kamailio/commit/9eba5b673cc1a80434642b280e6f7d5…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2018-11-29T18:20:24+01:00
dispatcher: flag 16 (1<<4) for destinations with host not having A/AAAA record
- use it for addresses that should not be resolved with an A/AAAA query
at startup (e.g., it is a NAPTR or SRV only record for that specific
hostname)
- such addresses will not match when using ds_is_from_list(...)
- GH #1743
---
Modified: src/modules/dispatcher/dispatch.c
Modified: src/modules/dispatcher/dispatch.h
---
Diff: https://github.com/kamailio/kamailio/commit/9eba5b673cc1a80434642b280e6f7d5…
Patch: https://github.com/kamailio/kamailio/commit/9eba5b673cc1a80434642b280e6f7d5…
---
diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c
index cb8fe6ea81..59cf1d9302 100644
--- a/src/modules/dispatcher/dispatch.c
+++ b/src/modules/dispatcher/dispatch.c
@@ -424,11 +424,17 @@ ds_dest_t *pack_dest(str iuri, int flags, int priority, str *attrs)
/* Do a DNS-Lookup for the Host-Name: */
he = resolvehost(hn);
if(he == 0) {
- LM_ERR("could not resolve %.*s\n", puri.host.len, puri.host.s);
- goto err;
+ if(dp->flags & DS_NODNSARES_DST) {
+ dp->irmode |= DS_IRMODE_NOIPADDR;
+ } else {
+ LM_ERR("could not resolve %.*s (missing no-probing flag?!?)\n",
+ puri.host.len, puri.host.s);
+ goto err;
+ }
+ } else {
+ /* Store hostent in the dispatcher structure */
+ hostent2ip_addr(&dp->ip_address, he, 0);
}
- /* Free the hostname */
- hostent2ip_addr(&dp->ip_address, he, 0);
/* Copy the port out of the URI */
dp->port = puri.port_no;
@@ -2957,8 +2963,8 @@ int ds_is_addr_from_list(sip_msg_t *_m, int group, str *uri, int mode)
if(group == -1) {
- rc = ds_is_addr_from_set_r(
- _m, pipaddr, tport, tproto, _ds_list, mode, 1);
+ rc = ds_is_addr_from_set_r(_m, pipaddr, tport, tproto, _ds_list,
+ mode, 1);
} else {
list = ds_avl_find(_ds_list, group);
if(list) {
diff --git a/src/modules/dispatcher/dispatch.h b/src/modules/dispatcher/dispatch.h
index 0675b3ccd3..1c89c756e6 100644
--- a/src/modules/dispatcher/dispatch.h
+++ b/src/modules/dispatcher/dispatch.h
@@ -44,7 +44,8 @@
#define DS_TRYING_DST 2 /*!< temporary trying destination */
#define DS_DISABLED_DST 4 /*!< admin disabled destination */
#define DS_PROBING_DST 8 /*!< checking destination */
-#define DS_STATES_ALL 15 /*!< all bits for the states of destination */
+#define DS_NODNSARES_DST 16 /*!< no DNS A/AAAA resolve for host in uri */
+#define DS_STATES_ALL 31 /*!< all bits for the states of destination */
#define ds_skip_dst(flags) ((flags) & (DS_INACTIVE_DST|DS_DISABLED_DST))
@@ -68,6 +69,8 @@
#define DS_XAVP_CTX_SKIP_CNT 1
+#define DS_IRMODE_NOIPADDR 1
+
/* clang-format on */
extern str ds_db_url;
@@ -185,16 +188,17 @@ typedef struct _ds_latency_stats {
} ds_latency_stats_t;
typedef struct _ds_dest {
- str uri;
- int flags;
- int priority;
- int dload;
- ds_attrs_t attrs;
- ds_latency_stats_t latency_stats;
- struct socket_info * sock;
- struct ip_addr ip_address; /*!< IP-Address of the entry */
- unsigned short int port; /*!< Port of the URI */
- unsigned short int proto; /*!< Protocol of the URI */
+ str uri; /*!< address/uri */
+ int flags; /*!< flags */
+ int priority; /*!< priority */
+ int dload; /*!< load */
+ ds_attrs_t attrs; /*!< the atttributes */
+ ds_latency_stats_t latency_stats; /*!< latency statistics */
+ int irmode; /*!< internal runtime mode (flags) */
+ struct socket_info *sock; /*!< pointer to local socket */
+ struct ip_addr ip_address; /*!< IP of the address */
+ unsigned short int port; /*!< port of the URI */
+ unsigned short int proto; /*!< protocol of the URI */
int message_count;
struct _ds_dest *next;
} ds_dest_t;
Module: kamailio
Branch: master
Commit: f073f72af6a507e8113e06dc07ad161e3f4632c3
URL: https://github.com/kamailio/kamailio/commit/f073f72af6a507e8113e06dc07ad161…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2018-11-28T17:38:56+01:00
dispatcher: docs - make a bullet list for the possible values for flags field
---
Modified: src/modules/dispatcher/doc/dispatcher_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/f073f72af6a507e8113e06dc07ad161…
Patch: https://github.com/kamailio/kamailio/commit/f073f72af6a507e8113e06dc07ad161…
---
diff --git a/src/modules/dispatcher/doc/dispatcher_admin.xml b/src/modules/dispatcher/doc/dispatcher_admin.xml
index 34a4a1559a..023f6a0ba2 100644
--- a/src/modules/dispatcher/doc/dispatcher_admin.xml
+++ b/src/modules/dispatcher/doc/dispatcher_admin.xml
@@ -1776,12 +1776,23 @@ onreply_route {
</para>
<itemizedlist>
<listitem>
- <para>flags (listed by index - can be bitwise mask of values):
- 0 (value 1) - inactive destination; 1 (value 2) - temporary trying
- destination (in the way to become inactive if it does not reply to
- keepalives - there is a module parameter to set the threshold of
- failures); 2 (value 4) - admin disabled destination; 3 (value 8)
- - probing destination (sending keep alives);</para>
+ <para>flags - control the mode of using the destination address and
+ sending keepalives. It is a bitwise value that can be built using
+ the folowing flags:
+ <itemizedlist>
+ <listitem><para>1 (bit at index 0 - 1 <<0) - inactive destination<para>
+ </listitem>
+ <listitem><para>2 (bit at index 1 - 1 <<1) - temporary trying
+ destination (in the way to become inactive if it does not reply to
+ keepalives - there is a module parameter to set the threshold of
+ failures)<para>
+ </listitem>
+ <listitem><para>4 (bit at index 2 - 1 <<2) - admin disabled destination<para>
+ </listitem>
+ <listitem><para>8 (bit at index 3 - 1 <<3) - probing destination (sending keep alives);</para>
+ </listitem>
+ </itemizedlist>
+ </para>
</listitem>
<listitem>
<para>priority: sets the priority in destination list (based on it
Kamailio v5.2.0 is out – it comes with 6 new modules and a consistent
group of improvements touching more than 100 existing modules.
You can read detailed release notes at:
* https://www.kamailio.org/w/kamailio-v5-2-0-release-notes/
Many thanks to all developers and community members that made possible
this release.
v5.2.0 brings more flexibility and optimizations for KEMI interpreters
(including two new interpreters for Python3 and Ruby), major
enhancements to load balancer and tls implementations, new variables and
lots of other new features.
Enjoy Kamailio v5.2.0!
Thank you for flying Kamailio!
Daniel
--
Daniel-Constantin Mierla
http://www.asipto.comhttp://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
Kamailio World Conference 2019 - http://www.kamailioworld.com
Hello,
I am consider to release v5.2.0 (the first stable version out of branch
5.2) next week, likely on Wednesday, Nov 28, 2018.
It still allows a bit more than a week of testing as well as well time
to prepare the online resources for it (documentation, wiki pages,
upgrade guidelines, etc...).
If there is any issue you are of and not yet reported to github bug
tracker, do it as soon as possible to give it a chance to be fixed in
time for the next major release.
Cheers,
Daniel
--
Daniel-Constantin Mierla -- www.asipto.comwww.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio World Conference -- www.kamailioworld.com
Kamailio Advanced Training, Nov 12-14, 2018, in Berlin -- www.asipto.com
Module: kamailio
Branch: 5.2
Commit: 0387df55d08c435e8ee4d8dde41c260f2ddb839b
URL: https://github.com/kamailio/kamailio/commit/0387df55d08c435e8ee4d8dde41c260…
Author: Surendra Tiwari <surendratiwari3(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2018-11-27T20:03:34+01:00
xmlrpc: docs for event_callback parameter
- GH #1736
(cherry picked from commit d74d67976905c9d0c49f00485046117bc2581e65)
---
Modified: src/modules/xmlrpc/doc/xmlrpc_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/0387df55d08c435e8ee4d8dde41c260…
Patch: https://github.com/kamailio/kamailio/commit/0387df55d08c435e8ee4d8dde41c260…
---
diff --git a/src/modules/xmlrpc/doc/xmlrpc_admin.xml b/src/modules/xmlrpc/doc/xmlrpc_admin.xml
index 381a1feff7..a61d63e6db 100644
--- a/src/modules/xmlrpc/doc/xmlrpc_admin.xml
+++ b/src/modules/xmlrpc/doc/xmlrpc_admin.xml
@@ -60,7 +60,7 @@ modparam("xmlrpc", "route", "route_for_xmlrpcs")
<section id="autoconversion">
<title><varname>autoconversion</varname> (string)</title>
<para>
- Enable or disable automatic parameter type conversion globally,
+ Enable or disable automatic parameter type conversion globally,
for all the methods parameters.
If on, a type mismatch in a method parameter
will not cause a fault if it is possible to automatically
@@ -192,6 +192,45 @@ modparam("xmlrpc", "url_match", "^/RPC2")
</programlisting>
</example>
</section>
+ <section id="event_callback">
+ <title><varname>event_callback</varname> (string)</title>
+ <para>
+ The name of the function in the kemi configuration file (embedded
+ scripting language such as Lua, Python, ...) to be executed instead
+ of route blocks defined by route parameter for xmlrpc request.
+ </para>
+ <para>
+ This route will be called only for HTTP messages whose method is
+ either GET or POST. The message visible inside the route
+ will be a HTTP request converted to SIP (the uri will be fixed and
+ a fake via will be added).
+ </para>
+ <para>
+ The route should perform additional security checks to
+ ensure the client is authorized to execute management/RPC functions
+ and then it should call the <function>dispatch_rpc()</function>.
+ </para>
+ <example>
+ <title>Set <varname>event_callback</varname> parameter</title>
+ <programlisting format="linespecific">
+...
+modparam("xmlrpc", "event_callback", "ksr_xmlrpc_event")
+...
+-- event callback function implemented in Lua
+function ksr_xmlrpc_event(evname)
+ KSR.info("===== xmlrpc triggered event: " .. evname .. "\n");
+ local rpc_method = KSR.pv.get("$rm") or ""
+ if ((rpc_method == "POST" or rpc_method == "GET")) then
+ if (KSR.xmlrpc.dispatch_rpc() < 0) then
+ KSR.err("error while executing xmlrpc event")
+ end
+ end
+ return 1;
+end
+...
+</programlisting>
+ </example>
+ </section>
</section>
<section>
Module: kamailio
Branch: 5.2
Commit: 6da8a5b4cb1f9b97aa2f92bc5d97786443712099
URL: https://github.com/kamailio/kamailio/commit/6da8a5b4cb1f9b97aa2f92bc5d97786…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2018-11-27T20:03:55+01:00
textops: docs - notes that msg_apply_changes() may be required for multi-part operations
- GH #1719
(cherry picked from commit 5460db2e59341f143d607de744faa8413216afd5)
---
Modified: src/modules/textops/doc/textops_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/6da8a5b4cb1f9b97aa2f92bc5d97786…
Patch: https://github.com/kamailio/kamailio/commit/6da8a5b4cb1f9b97aa2f92bc5d97786…
---
diff --git a/src/modules/textops/doc/textops_admin.xml b/src/modules/textops/doc/textops_admin.xml
index 8e0f350056..d348ccf6be 100644
--- a/src/modules/textops/doc/textops_admin.xml
+++ b/src/modules/textops/doc/textops_admin.xml
@@ -1638,6 +1638,7 @@ if (starts_with("$rU", "+358"))
set_body_multipart("test", "text/plain", "delimiter");
msg_apply_changes();
append_body_part(...);
+msg_apply_changes();
...
@@ -1690,12 +1691,17 @@ text
The core will take care of the last boundary ending "--". Detecting wich one is
the last and fixing the others if needed.
</para>
+ <para>
+ Note: it may be required that msg_apply_changes() from textopsx module
+ has to be executed if there are other operations over the new body.
+ </para>
<example>
<title><function>append_body_part</function> usage</title>
<programlisting format="linespecific">
...
$var(b) = "7e Od 04 55 75 69 20 4d 61 6b 65 43 61 6c 6c";
append_body_part("$var(b)", "application/vnd.cirpack.isdn-ext", "signal;handling=required");
+msg_apply_changes();
...
# Will append this to the body:
@@ -1721,6 +1727,7 @@ Content-Disposition: signal;handling=required
...
$var(b) = "<info>active</info>";
append_body_part("$var(b)", "application/xml\r\nX-Header: xyz");
+msg_apply_changes();
...
# Will append this to the body:
@@ -1773,12 +1780,17 @@ Content-Disposition: signal;handling=required
The core will take care of the last boundary ending "--". Detecting wich one is
the last and fixing the others if needed.
</para>
+ <para>
+ Note: it may be required that msg_apply_changes() from textopsx module
+ has to be executed if there are other operations over the new body.
+ </para>
<example>
<title><function>append_body_part_hex</function> usage</title>
<programlisting format="linespecific">
...
$var(b) = "6b 61 6d 61 69 6c 69 6f";
append_body_part_hex("$var(b)", "application/my-custom-ext");
+msg_apply_changes();
...
# Will append this to the body:
@@ -1894,11 +1906,16 @@ get_body_part("application/vnd.cirpack.isdn-ext", "$var(hbody)");
The core will take care of the last boundary ending "--". Detecting wich one is
the last and fixing the others if needed.
</para>
+ <para>
+ Note: it may be required that msg_apply_changes() from textopsx module
+ has to be executed if there are other operations over the new body.
+ </para>
<example>
<title><function>remove_body_part</function> usage</title>
<programlisting format="linespecific">
...
remove_body_part("application/vnd.cirpack.isdn-ext");
+msg_apply_changes();
...
</programlisting>
</example>
Module: kamailio
Branch: master
Commit: 5460db2e59341f143d607de744faa8413216afd5
URL: https://github.com/kamailio/kamailio/commit/5460db2e59341f143d607de744faa84…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2018-11-27T17:53:30+01:00
textops: docs - notes that msg_apply_changes() may be required for multi-part operations
- GH #1719
---
Modified: src/modules/textops/doc/textops_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/5460db2e59341f143d607de744faa84…
Patch: https://github.com/kamailio/kamailio/commit/5460db2e59341f143d607de744faa84…
---
diff --git a/src/modules/textops/doc/textops_admin.xml b/src/modules/textops/doc/textops_admin.xml
index 8e0f350056..d348ccf6be 100644
--- a/src/modules/textops/doc/textops_admin.xml
+++ b/src/modules/textops/doc/textops_admin.xml
@@ -1638,6 +1638,7 @@ if (starts_with("$rU", "+358"))
set_body_multipart("test", "text/plain", "delimiter");
msg_apply_changes();
append_body_part(...);
+msg_apply_changes();
...
@@ -1690,12 +1691,17 @@ text
The core will take care of the last boundary ending "--". Detecting wich one is
the last and fixing the others if needed.
</para>
+ <para>
+ Note: it may be required that msg_apply_changes() from textopsx module
+ has to be executed if there are other operations over the new body.
+ </para>
<example>
<title><function>append_body_part</function> usage</title>
<programlisting format="linespecific">
...
$var(b) = "7e Od 04 55 75 69 20 4d 61 6b 65 43 61 6c 6c";
append_body_part("$var(b)", "application/vnd.cirpack.isdn-ext", "signal;handling=required");
+msg_apply_changes();
...
# Will append this to the body:
@@ -1721,6 +1727,7 @@ Content-Disposition: signal;handling=required
...
$var(b) = "<info>active</info>";
append_body_part("$var(b)", "application/xml\r\nX-Header: xyz");
+msg_apply_changes();
...
# Will append this to the body:
@@ -1773,12 +1780,17 @@ Content-Disposition: signal;handling=required
The core will take care of the last boundary ending "--". Detecting wich one is
the last and fixing the others if needed.
</para>
+ <para>
+ Note: it may be required that msg_apply_changes() from textopsx module
+ has to be executed if there are other operations over the new body.
+ </para>
<example>
<title><function>append_body_part_hex</function> usage</title>
<programlisting format="linespecific">
...
$var(b) = "6b 61 6d 61 69 6c 69 6f";
append_body_part_hex("$var(b)", "application/my-custom-ext");
+msg_apply_changes();
...
# Will append this to the body:
@@ -1894,11 +1906,16 @@ get_body_part("application/vnd.cirpack.isdn-ext", "$var(hbody)");
The core will take care of the last boundary ending "--". Detecting wich one is
the last and fixing the others if needed.
</para>
+ <para>
+ Note: it may be required that msg_apply_changes() from textopsx module
+ has to be executed if there are other operations over the new body.
+ </para>
<example>
<title><function>remove_body_part</function> usage</title>
<programlisting format="linespecific">
...
remove_body_part("application/vnd.cirpack.isdn-ext");
+msg_apply_changes();
...
</programlisting>
</example>
<!-- 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 -->
- [ ] Commit message has the format required by CONTRIBUTING guide
- [ ] Commits are split per component (core, individual modules, libs, utils, ...)
- [ ] Each component has a single commit (if not, squash them into one commit)
- [ ] No commits to README files for modules (changes must be done to docbook files
in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1736
-- Commit Summary --
* Merge pull request #1 from kamailio/master
* Modules:XMLRPC event_route callback added for ksr
-- File Changes --
M src/modules/xmlrpc/xmlrpc.c (56)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1736.patchhttps://github.com/kamailio/kamailio/pull/1736.diff
--
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/pull/1736
Module: kamailio
Branch: master
Commit: d74d67976905c9d0c49f00485046117bc2581e65
URL: https://github.com/kamailio/kamailio/commit/d74d67976905c9d0c49f00485046117…
Author: Surendra Tiwari <surendratiwari3(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2018-11-27T12:47:40+01:00
xmlrpc: docs for event_callback parameter
- GH #1736
---
Modified: src/modules/xmlrpc/doc/xmlrpc_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/d74d67976905c9d0c49f00485046117…
Patch: https://github.com/kamailio/kamailio/commit/d74d67976905c9d0c49f00485046117…
---
diff --git a/src/modules/xmlrpc/doc/xmlrpc_admin.xml b/src/modules/xmlrpc/doc/xmlrpc_admin.xml
index 381a1feff7..a61d63e6db 100644
--- a/src/modules/xmlrpc/doc/xmlrpc_admin.xml
+++ b/src/modules/xmlrpc/doc/xmlrpc_admin.xml
@@ -60,7 +60,7 @@ modparam("xmlrpc", "route", "route_for_xmlrpcs")
<section id="autoconversion">
<title><varname>autoconversion</varname> (string)</title>
<para>
- Enable or disable automatic parameter type conversion globally,
+ Enable or disable automatic parameter type conversion globally,
for all the methods parameters.
If on, a type mismatch in a method parameter
will not cause a fault if it is possible to automatically
@@ -192,6 +192,45 @@ modparam("xmlrpc", "url_match", "^/RPC2")
</programlisting>
</example>
</section>
+ <section id="event_callback">
+ <title><varname>event_callback</varname> (string)</title>
+ <para>
+ The name of the function in the kemi configuration file (embedded
+ scripting language such as Lua, Python, ...) to be executed instead
+ of route blocks defined by route parameter for xmlrpc request.
+ </para>
+ <para>
+ This route will be called only for HTTP messages whose method is
+ either GET or POST. The message visible inside the route
+ will be a HTTP request converted to SIP (the uri will be fixed and
+ a fake via will be added).
+ </para>
+ <para>
+ The route should perform additional security checks to
+ ensure the client is authorized to execute management/RPC functions
+ and then it should call the <function>dispatch_rpc()</function>.
+ </para>
+ <example>
+ <title>Set <varname>event_callback</varname> parameter</title>
+ <programlisting format="linespecific">
+...
+modparam("xmlrpc", "event_callback", "ksr_xmlrpc_event")
+...
+-- event callback function implemented in Lua
+function ksr_xmlrpc_event(evname)
+ KSR.info("===== xmlrpc triggered event: " .. evname .. "\n");
+ local rpc_method = KSR.pv.get("$rm") or ""
+ if ((rpc_method == "POST" or rpc_method == "GET")) then
+ if (KSR.xmlrpc.dispatch_rpc() < 0) then
+ KSR.err("error while executing xmlrpc event")
+ end
+ end
+ return 1;
+end
+...
+</programlisting>
+ </example>
+ </section>
</section>
<section>
### Description
When using `ds_select_routes` $du is properly set to the first setid (1) and limiting properly the calls (with maxload), but when the nodes on setid 1 are at maxload and kamailio properly selects the second setid (11) it does NOT set $du.
### Troubleshooting
My dispatcher table is:
```
+-----+-------+-------------------------+-------+----------+-----------------------------------+-------------+
| id | setid | destination | flags | priority | attrs | description |
+-----+-------+-------------------------+-------+----------+-----------------------------------+-------------+
| 213 | 1 | sip:1.1.1.1:5080 | 0 | 0 | weight=100;maxload=2;duid=cuore_1 | node1-proxy |
| 229 | 1 | sip:2.2.2.2:5080 | 0 | 0 | weight=100;maxload=2;duid=cuore_2 | node2-proxy |
| 232 | 11 | sip:3.3.3.3:5080 | 0 | 0 | weight=100;maxload=2;duid=cuore_3 | node3-proxy |
+-----+-------+-------------------------+-------+----------+-----------------------------------+-------------+
```
#### Log Messages
```
[DISPATCH] 1-12743(a)192.168.1.201: ds_select_routes was succesful (ruri: sip:*TESTCALL@cuore-proxy.sandbox-sngl.com - du: <sip:1.1.1.1:5080>
[DISPATCH] 2-12743(a)192.168.1.201: ds_select_routes was succesful (ruri: sip:*TESTCALL@cuore-proxy.sandbox-sngl.com - du: <sip:2.2.2.2:5080>
[DISPATCH] 3-12743(a)192.168.1.201: ds_select_routes was succesful (ruri: sip:*TESTCALL@cuore-proxy.sandbox-sngl.com - du: <sip:1.1.1.1:5080>
[DISPATCH] 4-12743(a)192.168.1.201: ds_select_routes was succesful (ruri: sip:*TESTCALL@cuore-proxy.sandbox-sngl.com - du: <sip:2.2.2.2:5080>
[DISPATCH] 5-12743(a)192.168.1.201: ds_select_routes was succesful (ruri: sip:*TESTCALL@cuore-proxy.sandbox-sngl.com - du: <<null>>
[DISPATCH] 6-12743(a)192.168.1.201: ds_select_routes was succesful (ruri: sip:*TESTCALL@cuore-proxy.sandbox-sngl.com - du: <<null>>
```
### Possible Solutions
<!--
If you found a solution or workaround for the issue, describe it. Ideally, provide a pull request with a fix.
-->
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
```
version: kamailio 5.3.0-dev0 (x86_64/linux) f465d2
flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MEM, SHM_MMAP, PKG_MALLOC, 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_BLACKLIST, HAVE_RESOLV_RES
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: f465d2
compiled on 01:08:37 Nov 17 2018 with gcc 4.9.2
```
* **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 `uname -a`)
-->
```
Linux my-proxy 3.16.0-4-amd64 #1 SMP Debian 3.16.39-1 (2016-12-30) x86_64 GNU/Linux
```
--
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/1728
<!-- 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
- [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 -->
- [ ] 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 -->
Kemi framework python example configuration had a problem when handling CANCEL messages. Called function needed 'msg' attribute.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1734
-- Commit Summary --
* Added 'msg' argument to ksr_route_relay
-- File Changes --
M misc/examples/kemi/kamailio-basic-kemi-python.py (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1734.patchhttps://github.com/kamailio/kamailio/pull/1734.diff
--
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/pull/1734
<!-- 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 -->
- [ ] Commit message has the format required by CONTRIBUTING guide
- [ ] Commits are split per component (core, individual modules, libs, utils, ...)
- [ ] Each component has a single commit (if not, squash them into one commit)
- [ ] No commits to README files for modules (changes must be done to docbook files
in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
Previously for timeout reply kamailio was not printing the command for which we received the timeout, Also if kamailio not able to read the response then also command was not getting printed. some indention fix is also done.
<!-- Describe your changes in detail -->
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1732
-- Commit Summary --
* Merge pull request #1 from kamailio/master
* Modules: rtpengine: fixing log for printing the command when timeout and bad error reply
-- File Changes --
M src/modules/rtpengine/rtpengine.c (8)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1732.patchhttps://github.com/kamailio/kamailio/pull/1732.diff
--
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/pull/1732
<!-- Kamailio Pull Request Template -->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [ x] Commit message has the format required by CONTRIBUTING guide
- [ x] Commits are split per component (core, individual modules, libs, utils, ...)
- [x ] Each component has a single commit (if not, squash them into one commit)
- [x ] No commits to README files for modules (changes must be done to docbook files
in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change
- [x ] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [x] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
get_body_part functions gives error even if content is not multipart. Fixed for non-multipart contents and set all contents for non-boundary params
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1730
-- Commit Summary --
* core : add word to log get_boundary function
* textops: fixed get_body_part function for non-multipart contents error
-- File Changes --
M src/core/msg_translator.c (2)
M src/modules/textops/textops.c (19)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1730.patchhttps://github.com/kamailio/kamailio/pull/1730.diff
--
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/pull/1730
Module: kamailio
Branch: 5.0
Commit: 0965f58554f5917efecbc9e3329789a9f7c4472f
URL: https://github.com/kamailio/kamailio/commit/0965f58554f5917efecbc9e3329789a…
Author: Dmitri Savolainen <savolainen(a)erinaco.ru>
Committer: Henning Westerholt <hw(a)kamailio.org>
Date: 2018-11-20T10:56:31+01:00
dialog: fix typo in warn message (#1731)
- GH #1731
(cherry picked from commit df6152fd69469eb340078535a3cf8b28e901ea7d)
---
Modified: src/modules/dialog/dlg_db_handler.c
---
Diff: https://github.com/kamailio/kamailio/commit/0965f58554f5917efecbc9e3329789a…
Patch: https://github.com/kamailio/kamailio/commit/0965f58554f5917efecbc9e3329789a…
---
diff --git a/src/modules/dialog/dlg_db_handler.c b/src/modules/dialog/dlg_db_handler.c
index 0b9d77ce89..8e8b5b088b 100644
--- a/src/modules/dialog/dlg_db_handler.c
+++ b/src/modules/dialog/dlg_db_handler.c
@@ -565,11 +565,11 @@ static int load_dialog_vars_from_db(int fetch_num_rows)
}
dlg = dlg->next;
if (!dlg) {
- LM_WARN("insonsistent data: the dialog h_entry/h_id does not exist!\n");
+ LM_WARN("inconsistent data: the dialog h_entry/h_id does not exist!\n");
}
}
} else {
- LM_WARN("insonsistent data: the h_entry in the DB does not exist!\n");
+ LM_WARN("inconsistent data: the h_entry in the DB does not exist!\n");
}
}
Module: kamailio
Branch: 5.2
Commit: 2a42aebc8d6905989f21ec21459a1cd368632b6f
URL: https://github.com/kamailio/kamailio/commit/2a42aebc8d6905989f21ec21459a1cd…
Author: Dmitri Savolainen <savolainen(a)erinaco.ru>
Committer: Henning Westerholt <hw(a)kamailio.org>
Date: 2018-11-20T10:52:57+01:00
dialog: fix typo in warn message (#1731)
- GH #1731
(cherry picked from commit df6152fd69469eb340078535a3cf8b28e901ea7d)
---
Modified: src/modules/dialog/dlg_db_handler.c
---
Diff: https://github.com/kamailio/kamailio/commit/2a42aebc8d6905989f21ec21459a1cd…
Patch: https://github.com/kamailio/kamailio/commit/2a42aebc8d6905989f21ec21459a1cd…
---
diff --git a/src/modules/dialog/dlg_db_handler.c b/src/modules/dialog/dlg_db_handler.c
index d759140018..1b9d704cf6 100644
--- a/src/modules/dialog/dlg_db_handler.c
+++ b/src/modules/dialog/dlg_db_handler.c
@@ -643,14 +643,14 @@ static int load_dialog_vars_from_db(int fetch_num_rows, int mode,
}
dlg = dlg->next;
if (!dlg) {
- LM_WARN("insonsistent data: the dialog h_entry/h_id does not exist!\n");
+ LM_WARN("inconsistent data: the dialog h_entry/h_id does not exist!\n");
}
}
if(mode==1 && mval!=NULL) {
dlg_unlock(d_table, &(d_table->entries[VAL_INT(values)]));
}
} else {
- LM_WARN("insonsistent data: the h_entry in the DB does not exist!\n");
+ LM_WARN("inconsistent data: the h_entry in the DB does not exist!\n");
}
}
Module: kamailio
Branch: 5.1
Commit: 7fd9d125ef3d0ee41fc054176e3eb56c2e1a4094
URL: https://github.com/kamailio/kamailio/commit/7fd9d125ef3d0ee41fc054176e3eb56…
Author: Dmitri Savolainen <savolainen(a)erinaco.ru>
Committer: Henning Westerholt <hw(a)kamailio.org>
Date: 2018-11-20T10:53:33+01:00
dialog: fix typo in warn message (#1731)
- GH #1731
(cherry picked from commit df6152fd69469eb340078535a3cf8b28e901ea7d)
---
Modified: src/modules/dialog/dlg_db_handler.c
---
Diff: https://github.com/kamailio/kamailio/commit/7fd9d125ef3d0ee41fc054176e3eb56…
Patch: https://github.com/kamailio/kamailio/commit/7fd9d125ef3d0ee41fc054176e3eb56…
---
diff --git a/src/modules/dialog/dlg_db_handler.c b/src/modules/dialog/dlg_db_handler.c
index 0b9d77ce89..8e8b5b088b 100644
--- a/src/modules/dialog/dlg_db_handler.c
+++ b/src/modules/dialog/dlg_db_handler.c
@@ -565,11 +565,11 @@ static int load_dialog_vars_from_db(int fetch_num_rows)
}
dlg = dlg->next;
if (!dlg) {
- LM_WARN("insonsistent data: the dialog h_entry/h_id does not exist!\n");
+ LM_WARN("inconsistent data: the dialog h_entry/h_id does not exist!\n");
}
}
} else {
- LM_WARN("insonsistent data: the h_entry in the DB does not exist!\n");
+ LM_WARN("inconsistent data: the h_entry in the DB does not exist!\n");
}
}
- [x] Commit message has the format required by CONTRIBUTING guide
- [x] Commits are split per component (core, individual modules, libs, utils, ...)
- [x] Each component has a single commit (if not, squash them into one commit)
- [x] No commits to README files for modules (changes must be done to docbook files
in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change
- [x] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1731
-- Commit Summary --
* dialog: fix typo in warn message
-- File Changes --
M src/modules/dialog/dlg_db_handler.c (4)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1731.patchhttps://github.com/kamailio/kamailio/pull/1731.diff
--
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/pull/1731
Module: kamailio
Branch: master
Commit: df6152fd69469eb340078535a3cf8b28e901ea7d
URL: https://github.com/kamailio/kamailio/commit/df6152fd69469eb340078535a3cf8b2…
Author: Dmitri Savolainen <savolainen(a)erinaco.ru>
Committer: Henning Westerholt <henningw(a)users.noreply.github.com>
Date: 2018-11-20T10:42:05+01:00
dialog: fix typo in warn message (#1731)
Thank you!
---
Modified: src/modules/dialog/dlg_db_handler.c
---
Diff: https://github.com/kamailio/kamailio/commit/df6152fd69469eb340078535a3cf8b2…
Patch: https://github.com/kamailio/kamailio/commit/df6152fd69469eb340078535a3cf8b2…
---
diff --git a/src/modules/dialog/dlg_db_handler.c b/src/modules/dialog/dlg_db_handler.c
index d759140018..1b9d704cf6 100644
--- a/src/modules/dialog/dlg_db_handler.c
+++ b/src/modules/dialog/dlg_db_handler.c
@@ -643,14 +643,14 @@ static int load_dialog_vars_from_db(int fetch_num_rows, int mode,
}
dlg = dlg->next;
if (!dlg) {
- LM_WARN("insonsistent data: the dialog h_entry/h_id does not exist!\n");
+ LM_WARN("inconsistent data: the dialog h_entry/h_id does not exist!\n");
}
}
if(mode==1 && mval!=NULL) {
dlg_unlock(d_table, &(d_table->entries[VAL_INT(values)]));
}
} else {
- LM_WARN("insonsistent data: the h_entry in the DB does not exist!\n");
+ LM_WARN("inconsistent data: the h_entry in the DB does not exist!\n");
}
}
<!--
Kamailio Project uses GitHub Issues only for bugs in the code or feature requests. Please use this template only for bug reports.
If you have questions about using Kamailio or related to its configuration file, ask on sr-users mailing list:
* http://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
If you have questions about developing extensions to Kamailio or its existing C code, ask on sr-dev mailing list:
* http://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
Please try to fill this template as much as possible for any issue. It helps the developers to troubleshoot the issue.
If there is no content to be filled in a section, the entire section can be removed.
You can delete the comments from the template sections when filling.
You can delete next line and everything above before submitting (it is a comment).
-->
### Description
tm module writes a misleading log error that `triggers` ops alerts.
<!--
Explain what you did, what you expected to happen, and what actually happened.
-->
### Troubleshooting
#### Reproduction
using this partial script
```
event_route[core:worker-one-init]
{
async_route("DEFERRED_CLEANUP", "20");
}
route[DEFERRED_CLEANUP]
{
xlog("L_INFO", "processing deferred cleanup\n");
}
```
#### Log Messages
<!--
Check the syslog file and if there are relevant log messages printed by Kamailio, add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site).
-->
```
2018-11-15T19:22:44.547049+00:00 apps001 kamailio[2105]: INFO: <script>: processing deferred cleanup
2018-11-15T19:22:44.554085+00:00 apps001 kamailio[2105]: ERROR: tm [t_reply.c:551]: _reply_light(): no resolved dst to send reply to
```
### Possible Solutions
i silence the error with the below patch but i'm pretty sure its not the proper way to do it.
```
diff --git a/src/modules/tm/t_reply.c b/src/modules/tm/t_reply.c
index 8a95fe7..99c59db 100644
--- a/src/modules/tm/t_reply.c
+++ b/src/modules/tm/t_reply.c
@@ -548,7 +548,9 @@
* the chances for this increase a lot.
*/
if (unlikely(!trans->uas.response.dst.send_sock)) {
- LM_ERR("no resolved dst to send reply to\n");
+ if(is_local(trans)) {
+ LM_ERR("no resolved dst to send reply to\n");
+ }
} else {
if (likely(SEND_PR_BUFFER( rb, buf, len )>=0)){
if (unlikely(code>=200 && !is_local(trans) &&
```
<!--
If you found a solution or workaround for the issue, describe it. Ideally, provide a pull request with a fix.
-->
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
```
version: kamailio 5.3.0-dev0 (x86_64/linux) 37bff4-dirty
flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MEM, SHM_MMAP, PKG_MALLOC, 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_BLACKLIST, HAVE_RESOLV_RES
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.
```
* **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 `uname -a`)
-->
```
Linux 2.6.32-754.el6.x86_64 #1 SMP Tue Jun 19 21:26:04 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
```
--
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/1727
### Description
While trying to setup a handoff to a third party SIP trunk with authentication, I ran into some issues with said authentication. I'm using qop auth on the inbound leg (and `consume_credentials()`), and then UAC to authenticate against the SIP trunk with different credentials. The third party SIP trunk offers qop auth/auth-int.
I manually tried to verify the digest that Kamailio was sending, but couldn't. I added some logging to the `auth_alg.c` file, and ran the whole thing again. HA1 is calculated correctly, and so is HA2. However, the final step of the algorithm produced an incorrect value.
After further debugging, I realised that the issue comes from https://github.com/kamailio/kamailio/blob/master/src/modules/uac/auth_alg.c…. More specifically, in my case, the value passed on to `MD5Update` is `auth,auth-int`, instead of the simple `auth` I was expecting (uac doesn't support auth-int).
### Troubleshooting
#### SIP Traffic
I have PCAPs demonstrating the issue and the hack/fix below. I'd rather not post them publicly, however, I am happy to share them privately with Kamailio devs.
In essence:
```
-> INVITE sip:12345@someprovider.org:5060;transport=tcp
<- 401 Unauthorized
WWW-Authenticate: Digest realm="someprovider.org", qop="auth,auth-int", nonce="5BCF48671749b873534dc63e76d5594f3988555f"
-> INVITE sip:12345@someprovider.org:5060;transport=tcp
Authorization: Digest username="redacted", realm="someprovider.org", nonce="5BCF48671749b873534dc63e76d5594f3988555f", uri="sip:12345@eu.out.didww.com:5060;transport=tcp", qop=auth, nc=00000001, cnonce="2106889321", response="redacted71d4c9ce119dcf3ec56209e", algorithm=MD5
```
### Possible Solutions
The hack I've used for the time being is to replace:
```c
MD5Update(&Md5Ctx, ":", 1);
MD5Update(&Md5Ctx, auth->qop.s, auth->qop.len);
MD5Update(&Md5Ctx, ":", 1);
```
with:
```c
MD5Update(&Md5Ctx, ":auth:", 6);
```
Obviously, I realise this is not the correct fix, but I figured I'd let the experts fix the value of `auth->qop.len`. I can spend more time on this, if required, but I don't know exactly what the "correct" solution is.
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
```
version: kamailio 5.1.6 (x86_64/linux)
flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MEM, SHM_MMAP, PKG_MALLOC, 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_BLACKLIST, HAVE_RESOLV_RES
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 5.3.1
```
* **Operating System**:
Dockerized version of Kamailio on Ubuntu xenial:
```
kamailio/kamailio:5.1.6-xenial
```
--
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/1684
Hi,
How can i allow to register with multiple domains and restrict with ip address user should register to asterisk with domain which given to user extension how can i do that.
Thanks & Best Regards,
Nagesh Saranga
<!--
Kamailio Project uses GitHub Issues only for bugs in the code or feature requests. Please use this template only for bug reports.
If you have questions about using Kamailio or related to its configuration file, ask on sr-users mailing list:
* http://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
If you have questions about developing extensions to Kamailio or its existing C code, ask on sr-dev mailing list:
* http://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
Please try to fill this template as much as possible for any issue. It helps the developers to troubleshoot the issue.
If there is no content to be filled in a section, the entire section can be removed.
You can delete the comments from the template sections when filling.
You can delete next line and everything above before submitting (it is a comment).
-->
### Description
<!--
Explain what you did, what you expected to happen, and what actually happened.
-->
### Troubleshooting
#### Reproduction
<!--
If the issue can be reproduced, describe how it can be done.
-->
#### Debugging Data
<!--
If you got a core dump, use gdb to extract troubleshooting data - full backtrace,
local variables and the list of the code at the issue location.
gdb /path/to/kamailio /path/to/corefile
bt full
info locals
list
If you are familiar with gdb, feel free to attach more of what you consider to
be relevant.
-->
```
(paste your debugging data here)
```
#### Log Messages
<!--
Check the syslog file and if there are relevant log messages printed by Kamailio, add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site).
-->
```
(paste your log messages here)
```
#### SIP Traffic
<!--
If the issue is exposed by processing specific SIP messages, grab them with ngrep or save in a pcap file, then add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site).
-->
```
(paste your sip traffic here)
```
### Possible Solutions
<!--
If you found a solution or workaround for the issue, describe it. Ideally, provide a pull request with a fix.
-->
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
```
(paste your output here)
```
* **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 `uname -a`)
-->
```
(paste your output here)
```
--
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/1729
… to store long values
#### Pre-Submission Checklist
- [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:
- [ ] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
During usage i can see errors in logs related to long values cannot be saved in PostgreSQL.
This PR is fix for PostgreSQL and other similar databases.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1723
-- Commit Summary --
* utils/kamctl: Adjusted avalue column in location_attrs tables to store long values
-- File Changes --
M utils/kamctl/db_sqlite/usrloc-create.sql (2)
M utils/kamctl/mysql/usrloc-create.sql (2)
M utils/kamctl/oracle/usrloc-create.sql (2)
M utils/kamctl/postgres/usrloc-create.sql (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1723.patchhttps://github.com/kamailio/kamailio/pull/1723.diff
--
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/pull/1723
<!-- 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
- [X] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [X] PR should be backported to stable branches
- [X] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1725
-- Commit Summary --
* dispatcher : xavp_get fix in ds_update_dst
-- File Changes --
M src/modules/dispatcher/dispatch.c (7)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1725.patchhttps://github.com/kamailio/kamailio/pull/1725.diff
--
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/pull/1725
Module: kamailio
Branch: master
Commit: d33ee59bc807acf95b07838d24b888b14a286ddd
URL: https://github.com/kamailio/kamailio/commit/d33ee59bc807acf95b07838d24b888b…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2018-11-16T10:37:19+01:00
websocket: aligned example statements
- few more details about return code of handshake process
---
Modified: src/modules/websocket/doc/websocket_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/d33ee59bc807acf95b07838d24b888b…
Patch: https://github.com/kamailio/kamailio/commit/d33ee59bc807acf95b07838d24b888b…
---
diff --git a/src/modules/websocket/doc/websocket_admin.xml b/src/modules/websocket/doc/websocket_admin.xml
index 055498862e..338d37ef22 100644
--- a/src/modules/websocket/doc/websocket_admin.xml
+++ b/src/modules/websocket/doc/websocket_admin.xml
@@ -90,13 +90,13 @@ event_route[xhttp:request] {
# Optional... perform HTTP authentication
# ws_handle_handshake() exits (no further configuration file
- # processing of the request) when complete.
+ # processing of the request) when complete in case of failure.
if (ws_handle_handshake())
- {
- # Optional... cache some information about the
- # successful connection
- exit;
- }
+ {
+ # Optional... cache some information about the
+ # successful connection
+ exit;
+ }
}
xhttp_reply("404", "Not found", "", "");
@@ -181,9 +181,9 @@ route[WITHINDLG] {
onreply_route[WS_REPLY] {
if (nat_uac_test(64)) {
# Do NAT traversal stuff for replies to a WebSocket connection
- # - even if it is not behind a NAT!
+ # - even if it is not behind a NAT!
# This won't be needed in the future if Kamailio and the
- # WebSocket client support Outbound and Path.
+ # WebSocket client support Outbound and Path.
add_contact_alias();
}
}
@@ -491,7 +491,8 @@ end
<para>This function can be used from ANY_ROUTE (but will only
work in <emphasis>event_route[xhttp:request]</emphasis>).</para>
<note><para>This function returns 0, stopping all further
- processing of the request, when there is a problem.</para></note>
+ processing of the request, when there is a problem. Otherwise, it
+ returns 1 (or positive number) in case of success.</para></note>
<example>
<title><function>ws_handle_handshake</function> usage</title>
<programlisting format="linespecific">
### Description
It was reported that using the RPM for v5.0.7 from opensuse build service is getting an error on dependency for internal lib ims.
Getting the rpm from:
* http://download.opensuse.org/repositories/home:/kamailio:/v5.0.x-rpms/RHEL_…
Then:
```
rpm -ivh kamailio-5.0.7-8.el7.x86_64.rpm
warning: kamailio-5.0.7-8.el7.x86_64.rpm: Header V3 DSA/SHA1 Signature, key ID 941fdbdd: NOKEY
error: Failed dependencies:
libkamailio_ims.so.0()(64bit) is needed by kamailio-5.0.7-8.el7.x86_64
```
### Troubleshooting
#### Reproduction
See above.
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
```
5.0.7, but can be the same for 5.1 or newer.
```
* **Operating System**:
```
RHEL 7, but can be the same for other OSes
```
--
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/1721
#### Pre-Submission Checklist
- [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:
- [x] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
Required backport to `5.2` branch.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1715
-- Commit Summary --
* pkg/kamailio/obs: Fixed build on OpenSUSE dists
* pkg/kamailio/obs: Changed rpm package group for OpenSUSE dist
-- File Changes --
M pkg/kamailio/obs/kamailio.spec (158)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1715.patchhttps://github.com/kamailio/kamailio/pull/1715.diff
--
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/pull/1715
<!-- Kamailio Pull Request Template -->
#### Pre-Submission Checklist
- [x] Commit message has the format required by CONTRIBUTING guide
- [x] Commits are split per component (core, individual modules, libs, utils, ...)
- [x] Each component has a single commit (if not, squash them into one commit)
- [x] No commits to README files for modules (changes must be done to docbook files
in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change
- [x] New module
#### Checklist:
Tested with various sofphones : Ekiga, Linphone, CSipSimple
#### Description
By combining Kamailio, oRTP and mediastreamer2
this module is providing some very good foundation to support RTP endpoint and various media processing features.
This seems like a great idea for various use cases like IoT, IVR or other specific needs.
Kamailio is handling everything related to SIP/SDP (the module is adding some SDP parsing) as well as providing a scripting engine.
oRTP is providing RTP endpoints compliant with (RFC 3550)
MediaStreamer2, even if written in C is quite a high level library, because it is implementing a framework for audio processing using graphs of filters, filters can be to do various things.
Similar to playing with lego blocks :)
* Support for most free and some non free codecs can be added easily.
* Work to bridge calls is already in progress in the module.
* Mobile phone support ARM CPU
* other embedded scenario should be supported
* much more ...
Mediastream2 is creating one thread per call "msticker", this can work smoothly in Kamailio even if it is forking processes.
Shared memory allocation is supported using wrapper around malloc/free used by the libraries.
Some extra work that needs to be done shorty
* Syncronization with locks is not completed (minor task, only access to a linked list needs to be syncronized properly)
* Memory leak not tested properly (not take the time to test with valgrind yet)
This project was started last year, I think it is time to submit it, I will surely find the time to do the extra work needed shortly.
Example config using the features already implemented
```
event_route[rms:start] {
xnotice("[rms:start] play ...\n");
rms_play("/tmp/reference_8000.wav", "rms:after_play");
};
event_route[rms:after_play] {
xnotice("[rms:after_play] play done...\n");
rms_hangup();
};
route {
if (t_precheck_trans()) {
t_check_trans();
exit;
}
t_check_trans();
if (is_method("INVITE") && !has_totag()) {
if (!rms_answer()) {
t_reply("503", "server error");
}
}
if (is_method("BYE")){
rms_media_stop();
}
}
```
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1701
-- Commit Summary --
* rtp_media_server: adding module
-- File Changes --
M src/Makefile.groups (6)
A src/modules/rtp_media_server/Makefile (13)
A src/modules/rtp_media_server/config_example/kamailio.cfg (56)
A src/modules/rtp_media_server/doc/Makefile (4)
A src/modules/rtp_media_server/doc/rtp_media_server.xml (42)
A src/modules/rtp_media_server/doc/rtp_media_server_admin.xml (193)
A src/modules/rtp_media_server/install_bc.sh (45)
A src/modules/rtp_media_server/rms_media.c (320)
A src/modules/rtp_media_server/rms_media.h (103)
A src/modules/rtp_media_server/rms_sdp.c (301)
A src/modules/rtp_media_server/rms_sdp.h (45)
A src/modules/rtp_media_server/rtp_media_server.c (718)
A src/modules/rtp_media_server/rtp_media_server.h (91)
A src/modules/rtp_media_server/voice_file/Bach_10s_8000.wav (0)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1701.patchhttps://github.com/kamailio/kamailio/pull/1701.diff
--
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/pull/1701
Fixed alpine docker packaging on master branch after switching from libress to openssl
This need only into master branch. (not backport)
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1718
-- Commit Summary --
* pkg/kamailio/alpine: Update APKBUILD
* pkg/docker: Alpine switched to use openssl. Updated rules for alpine docker container
-- File Changes --
M pkg/docker (2)
D pkg/kamailio/alpine/0004-src_core_tcp_read_c.patch (20)
M pkg/kamailio/alpine/APKBUILD (12)
M pkg/kamailio/alpine/kamailio.initd (2)
D pkg/kamailio/alpine/kamctl_build.patch (236)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1718.patchhttps://github.com/kamailio/kamailio/pull/1718.diff
--
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/pull/1718
When building debian package from master, I noticed this kind of error
message:
s ../../../doc/stylesheets/serdoc2man.xsl auth.xml
make[3]: s: Command not found
../../Makefile.modules:283: recipe for target 'auth.7' failed
The line in Makefile.modules is here:
$(MOD_NAME).7: $(MOD_NAME).xml
$(DOCBOOK) -s ../../../doc/stylesheets/serdoc2man.xsl $<
Could it be that variable $(DOCBOOK) has not been defined?
-- Juha
Hi all,
Is there any strong reason to set LC_LOG_MSG_MAX_SIZE to 16384 in the
log_custom module? Any thoughts / undesired effects if changed to 32768 or
higher?
```
$ grep -R 'define LC_LOG_MSG_MAX_SIZE' *
src/modules/log_custom/log_custom_mod.c:#define LC_LOG_MSG_MAX_SIZE 16384
src/modules/log_systemd/log_systemd_mod.c:#define LC_LOG_MSG_MAX_SIZE 16384
```
Thanks.
Luis
#### Pre-Submission Checklist
- [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:
- [x] PR should be backported to `5.2` branche
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
Need back port to 5.2 branch.
For 5.1 branch also wanted but will be merged with conflict. Backport into 5.1 is optional. Then i will create dedicated PR
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1711
-- Commit Summary --
* pkg/kamailio/obs: update spec to mach currently used
* pkg/kamailio/obs: Fixed missing libmnl-devel and librabbitmq-devel on RHEL dists
* pkg/kamailio/obs: smsops module relocated into main rpm package
-- File Changes --
M pkg/kamailio/obs/kamailio.service (3)
M pkg/kamailio/obs/kamailio.spec (51)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1711.patchhttps://github.com/kamailio/kamailio/pull/1711.diff
--
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/pull/1711
Module: kamailio
Branch: 5.2
Commit: 9bc520a02428656c8eb0a5d875c950298ceaf2e7
URL: https://github.com/kamailio/kamailio/commit/9bc520a02428656c8eb0a5d875c9502…
Author: Kevin Olbrich <ko(a)sv01.de>
Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org>
Date: 2018-11-08T18:05:28+01:00
pkg/kamailio/oracle: wait for network to be up
Units that strictly require a configured network connection should pull in network-online.target (via a Wants= type dependency) and order themselves after it. This target unit is intended to pull in a service that delays further execution until the network is sufficiently set up. What precisely this requires is left to the implementation of the network managing service.
https://www.freedesktop.org/software/systemd/man/systemd.special.htmlhttps://lists.kamailio.org/pipermail/sr-users/2018-November/103668.html
Signed-off-by: Kevin Olbrich <ko(a)sv01.de>
(cherry picked from commit 37c2e5ca25cb3396d2ac969271f9f9330d88fd63)
---
Modified: pkg/kamailio/oracle/el7/kamailio.service
---
Diff: https://github.com/kamailio/kamailio/commit/9bc520a02428656c8eb0a5d875c9502…
Patch: https://github.com/kamailio/kamailio/commit/9bc520a02428656c8eb0a5d875c9502…
---
diff --git a/pkg/kamailio/oracle/el7/kamailio.service b/pkg/kamailio/oracle/el7/kamailio.service
index 63d6da747c..8192488b53 100644
--- a/pkg/kamailio/oracle/el7/kamailio.service
+++ b/pkg/kamailio/oracle/el7/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio SIP router server daemon
-After=syslog.target network.target auditd.service
+Wants=network-online.target
+After=syslog.target network-online.target auditd.service
[Service]
EnvironmentFile=/etc/sysconfig/kamailio
Module: kamailio
Branch: 5.2
Commit: 33fb7ac2b367b4145815ed6e6919deb8d7c8631d
URL: https://github.com/kamailio/kamailio/commit/33fb7ac2b367b4145815ed6e6919deb…
Author: Kevin Olbrich <ko(a)sv01.de>
Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org>
Date: 2018-11-08T18:04:54+01:00
pkg/kamailio/deb: also restart service if unclean exit code
New:
If set to on-failure, the service will be restarted when the process exits with a non-zero exit code, is terminated by a signal (including on core dump, but excluding the aforementioned four signals), when an operation (such as service reload) times out, and when the configured watchdog timeout is triggered.
Old:
If set to on-abort, the service will be restarted only if the service process exits due to an uncaught signal not specified as a clean exit status.
https://www.freedesktop.org/software/systemd/man/systemd.service.html
Signed-off-by: Kevin Olbrich <ko(a)sv01.de>
(cherry picked from commit a5c45d1af4bf31e872963a954c43b8ee5618a90a)
---
Modified: pkg/kamailio/deb/bionic/kamailio.service
Modified: pkg/kamailio/deb/buster/kamailio.service
Modified: pkg/kamailio/deb/debian/kamailio.service
Modified: pkg/kamailio/deb/jessie/kamailio.service
Modified: pkg/kamailio/deb/sid/kamailio.service
Modified: pkg/kamailio/deb/stretch/kamailio.service
Modified: pkg/kamailio/deb/xenial/kamailio.service
---
Diff: https://github.com/kamailio/kamailio/commit/33fb7ac2b367b4145815ed6e6919deb…
Patch: https://github.com/kamailio/kamailio/commit/33fb7ac2b367b4145815ed6e6919deb…
Module: kamailio
Branch: 5.2
Commit: fb9dee9efee7f832c7c95fab2d066fb3606b1fa8
URL: https://github.com/kamailio/kamailio/commit/fb9dee9efee7f832c7c95fab2d066fb…
Author: Kevin Olbrich <ko(a)sv01.de>
Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org>
Date: 2018-11-08T18:04:49+01:00
pkg/kamailio/deb: wait for network to be up
Units that strictly require a configured network connection should pull in network-online.target (via a Wants= type dependency) and order themselves after it. This target unit is intended to pull in a service that delays further execution until the network is sufficiently set up. What precisely this requires is left to the implementation of the network managing service.
https://www.freedesktop.org/software/systemd/man/systemd.special.htmlhttps://lists.kamailio.org/pipermail/sr-users/2018-November/103668.html
Signed-off-by: Kevin Olbrich <ko(a)sv01.de>
(cherry picked from commit f7867838eb8a68a9ae4c00f2487bdec078b94c88)
---
Modified: pkg/kamailio/deb/bionic/kamailio.service
Modified: pkg/kamailio/deb/buster/kamailio.service
Modified: pkg/kamailio/deb/debian/kamailio.service
Modified: pkg/kamailio/deb/jessie/kamailio.service
Modified: pkg/kamailio/deb/sid/kamailio.service
Modified: pkg/kamailio/deb/stretch/kamailio.service
Modified: pkg/kamailio/deb/xenial/kamailio.service
---
Diff: https://github.com/kamailio/kamailio/commit/fb9dee9efee7f832c7c95fab2d066fb…
Patch: https://github.com/kamailio/kamailio/commit/fb9dee9efee7f832c7c95fab2d066fb…
---
diff --git a/pkg/kamailio/deb/bionic/kamailio.service b/pkg/kamailio/deb/bionic/kamailio.service
index e7a9e18484..99679e3372 100644
--- a/pkg/kamailio/deb/bionic/kamailio.service
+++ b/pkg/kamailio/deb/bionic/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio (OpenSER) - the Open Source SIP Server
-After=network.target
+Wants=network-online.target
+After=network-online.target
[Service]
Type=forking
diff --git a/pkg/kamailio/deb/buster/kamailio.service b/pkg/kamailio/deb/buster/kamailio.service
index e7a9e18484..99679e3372 100644
--- a/pkg/kamailio/deb/buster/kamailio.service
+++ b/pkg/kamailio/deb/buster/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio (OpenSER) - the Open Source SIP Server
-After=network.target
+Wants=network-online.target
+After=network-online.target
[Service]
Type=forking
diff --git a/pkg/kamailio/deb/debian/kamailio.service b/pkg/kamailio/deb/debian/kamailio.service
index e7a9e18484..99679e3372 100644
--- a/pkg/kamailio/deb/debian/kamailio.service
+++ b/pkg/kamailio/deb/debian/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio (OpenSER) - the Open Source SIP Server
-After=network.target
+Wants=network-online.target
+After=network-online.target
[Service]
Type=forking
diff --git a/pkg/kamailio/deb/jessie/kamailio.service b/pkg/kamailio/deb/jessie/kamailio.service
index e7a9e18484..99679e3372 100644
--- a/pkg/kamailio/deb/jessie/kamailio.service
+++ b/pkg/kamailio/deb/jessie/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio (OpenSER) - the Open Source SIP Server
-After=network.target
+Wants=network-online.target
+After=network-online.target
[Service]
Type=forking
diff --git a/pkg/kamailio/deb/sid/kamailio.service b/pkg/kamailio/deb/sid/kamailio.service
index e7a9e18484..99679e3372 100644
--- a/pkg/kamailio/deb/sid/kamailio.service
+++ b/pkg/kamailio/deb/sid/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio (OpenSER) - the Open Source SIP Server
-After=network.target
+Wants=network-online.target
+After=network-online.target
[Service]
Type=forking
diff --git a/pkg/kamailio/deb/stretch/kamailio.service b/pkg/kamailio/deb/stretch/kamailio.service
index e7a9e18484..99679e3372 100644
--- a/pkg/kamailio/deb/stretch/kamailio.service
+++ b/pkg/kamailio/deb/stretch/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio (OpenSER) - the Open Source SIP Server
-After=network.target
+Wants=network-online.target
+After=network-online.target
[Service]
Type=forking
diff --git a/pkg/kamailio/deb/xenial/kamailio.service b/pkg/kamailio/deb/xenial/kamailio.service
index e7a9e18484..99679e3372 100644
--- a/pkg/kamailio/deb/xenial/kamailio.service
+++ b/pkg/kamailio/deb/xenial/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio (OpenSER) - the Open Source SIP Server
-After=network.target
+Wants=network-online.target
+After=network-online.target
[Service]
Type=forking
<!-- 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
- [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 -->
- [ ] 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 -->
Please see conversation on ML for reference:
https://lists.kamailio.org/pipermail/sr-users/2018-November/103668.html
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1716
-- Commit Summary --
* Wait for network to be up before starting kamailio.service
* Restart service if it fails with unclean exit code instead of only unclean signal
-- File Changes --
M pkg/kamailio/deb/bionic/kamailio.service (5)
M pkg/kamailio/deb/buster/kamailio.service (5)
M pkg/kamailio/deb/debian/kamailio.service (5)
M pkg/kamailio/deb/jessie/kamailio.service (5)
M pkg/kamailio/deb/sid/kamailio.service (5)
M pkg/kamailio/deb/stretch/kamailio.service (5)
M pkg/kamailio/deb/xenial/kamailio.service (5)
M pkg/kamailio/oracle/el7/kamailio.service (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1716.patchhttps://github.com/kamailio/kamailio/pull/1716.diff
--
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/pull/1716
Module: kamailio
Branch: master
Commit: 37c2e5ca25cb3396d2ac969271f9f9330d88fd63
URL: https://github.com/kamailio/kamailio/commit/37c2e5ca25cb3396d2ac969271f9f93…
Author: Kevin Olbrich <ko(a)sv01.de>
Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org>
Date: 2018-11-08T18:02:32+01:00
pkg/kamailio/oracle: wait for network to be up
Units that strictly require a configured network connection should pull in network-online.target (via a Wants= type dependency) and order themselves after it. This target unit is intended to pull in a service that delays further execution until the network is sufficiently set up. What precisely this requires is left to the implementation of the network managing service.
https://www.freedesktop.org/software/systemd/man/systemd.special.htmlhttps://lists.kamailio.org/pipermail/sr-users/2018-November/103668.html
Signed-off-by: Kevin Olbrich <ko(a)sv01.de>
---
Modified: pkg/kamailio/oracle/el7/kamailio.service
---
Diff: https://github.com/kamailio/kamailio/commit/37c2e5ca25cb3396d2ac969271f9f93…
Patch: https://github.com/kamailio/kamailio/commit/37c2e5ca25cb3396d2ac969271f9f93…
---
diff --git a/pkg/kamailio/oracle/el7/kamailio.service b/pkg/kamailio/oracle/el7/kamailio.service
index 63d6da747c..8192488b53 100644
--- a/pkg/kamailio/oracle/el7/kamailio.service
+++ b/pkg/kamailio/oracle/el7/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio SIP router server daemon
-After=syslog.target network.target auditd.service
+Wants=network-online.target
+After=syslog.target network-online.target auditd.service
[Service]
EnvironmentFile=/etc/sysconfig/kamailio
Module: kamailio
Branch: master
Commit: f7867838eb8a68a9ae4c00f2487bdec078b94c88
URL: https://github.com/kamailio/kamailio/commit/f7867838eb8a68a9ae4c00f2487bdec…
Author: Kevin Olbrich <ko(a)sv01.de>
Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org>
Date: 2018-11-08T18:02:32+01:00
pkg/kamailio/deb: wait for network to be up
Units that strictly require a configured network connection should pull in network-online.target (via a Wants= type dependency) and order themselves after it. This target unit is intended to pull in a service that delays further execution until the network is sufficiently set up. What precisely this requires is left to the implementation of the network managing service.
https://www.freedesktop.org/software/systemd/man/systemd.special.htmlhttps://lists.kamailio.org/pipermail/sr-users/2018-November/103668.html
Signed-off-by: Kevin Olbrich <ko(a)sv01.de>
---
Modified: pkg/kamailio/deb/bionic/kamailio.service
Modified: pkg/kamailio/deb/buster/kamailio.service
Modified: pkg/kamailio/deb/debian/kamailio.service
Modified: pkg/kamailio/deb/jessie/kamailio.service
Modified: pkg/kamailio/deb/sid/kamailio.service
Modified: pkg/kamailio/deb/stretch/kamailio.service
Modified: pkg/kamailio/deb/xenial/kamailio.service
---
Diff: https://github.com/kamailio/kamailio/commit/f7867838eb8a68a9ae4c00f2487bdec…
Patch: https://github.com/kamailio/kamailio/commit/f7867838eb8a68a9ae4c00f2487bdec…
---
diff --git a/pkg/kamailio/deb/bionic/kamailio.service b/pkg/kamailio/deb/bionic/kamailio.service
index e7a9e18484..99679e3372 100644
--- a/pkg/kamailio/deb/bionic/kamailio.service
+++ b/pkg/kamailio/deb/bionic/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio (OpenSER) - the Open Source SIP Server
-After=network.target
+Wants=network-online.target
+After=network-online.target
[Service]
Type=forking
diff --git a/pkg/kamailio/deb/buster/kamailio.service b/pkg/kamailio/deb/buster/kamailio.service
index e7a9e18484..99679e3372 100644
--- a/pkg/kamailio/deb/buster/kamailio.service
+++ b/pkg/kamailio/deb/buster/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio (OpenSER) - the Open Source SIP Server
-After=network.target
+Wants=network-online.target
+After=network-online.target
[Service]
Type=forking
diff --git a/pkg/kamailio/deb/debian/kamailio.service b/pkg/kamailio/deb/debian/kamailio.service
index e7a9e18484..99679e3372 100644
--- a/pkg/kamailio/deb/debian/kamailio.service
+++ b/pkg/kamailio/deb/debian/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio (OpenSER) - the Open Source SIP Server
-After=network.target
+Wants=network-online.target
+After=network-online.target
[Service]
Type=forking
diff --git a/pkg/kamailio/deb/jessie/kamailio.service b/pkg/kamailio/deb/jessie/kamailio.service
index e7a9e18484..99679e3372 100644
--- a/pkg/kamailio/deb/jessie/kamailio.service
+++ b/pkg/kamailio/deb/jessie/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio (OpenSER) - the Open Source SIP Server
-After=network.target
+Wants=network-online.target
+After=network-online.target
[Service]
Type=forking
diff --git a/pkg/kamailio/deb/sid/kamailio.service b/pkg/kamailio/deb/sid/kamailio.service
index e7a9e18484..99679e3372 100644
--- a/pkg/kamailio/deb/sid/kamailio.service
+++ b/pkg/kamailio/deb/sid/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio (OpenSER) - the Open Source SIP Server
-After=network.target
+Wants=network-online.target
+After=network-online.target
[Service]
Type=forking
diff --git a/pkg/kamailio/deb/stretch/kamailio.service b/pkg/kamailio/deb/stretch/kamailio.service
index e7a9e18484..99679e3372 100644
--- a/pkg/kamailio/deb/stretch/kamailio.service
+++ b/pkg/kamailio/deb/stretch/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio (OpenSER) - the Open Source SIP Server
-After=network.target
+Wants=network-online.target
+After=network-online.target
[Service]
Type=forking
diff --git a/pkg/kamailio/deb/xenial/kamailio.service b/pkg/kamailio/deb/xenial/kamailio.service
index e7a9e18484..99679e3372 100644
--- a/pkg/kamailio/deb/xenial/kamailio.service
+++ b/pkg/kamailio/deb/xenial/kamailio.service
@@ -1,6 +1,7 @@
[Unit]
Description=Kamailio (OpenSER) - the Open Source SIP Server
-After=network.target
+Wants=network-online.target
+After=network-online.target
[Service]
Type=forking
Module: kamailio
Branch: master
Commit: a5c45d1af4bf31e872963a954c43b8ee5618a90a
URL: https://github.com/kamailio/kamailio/commit/a5c45d1af4bf31e872963a954c43b8e…
Author: Kevin Olbrich <ko(a)sv01.de>
Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org>
Date: 2018-11-08T18:02:32+01:00
pkg/kamailio/deb: also restart service if unclean exit code
New:
If set to on-failure, the service will be restarted when the process exits with a non-zero exit code, is terminated by a signal (including on core dump, but excluding the aforementioned four signals), when an operation (such as service reload) times out, and when the configured watchdog timeout is triggered.
Old:
If set to on-abort, the service will be restarted only if the service process exits due to an uncaught signal not specified as a clean exit status.
https://www.freedesktop.org/software/systemd/man/systemd.service.html
Signed-off-by: Kevin Olbrich <ko(a)sv01.de>
---
Modified: pkg/kamailio/deb/bionic/kamailio.service
Modified: pkg/kamailio/deb/buster/kamailio.service
Modified: pkg/kamailio/deb/debian/kamailio.service
Modified: pkg/kamailio/deb/jessie/kamailio.service
Modified: pkg/kamailio/deb/sid/kamailio.service
Modified: pkg/kamailio/deb/stretch/kamailio.service
Modified: pkg/kamailio/deb/xenial/kamailio.service
---
Diff: https://github.com/kamailio/kamailio/commit/a5c45d1af4bf31e872963a954c43b8e…
Patch: https://github.com/kamailio/kamailio/commit/a5c45d1af4bf31e872963a954c43b8e…
### Description
`libmnl-devel` is present in OpenSUSE but build of ims module is failed.
This because headers files is located on other path then kamailio expects.
`libmnl-devel` package is designed to use pkgconfig.
I checked on CentOS dist `libmnl-devel` also have files for pkgconfig.
### Troubleshooting
#### Reproduction
need to build ims module on openSUSE_Leap_42.3
#### Debugging Data
NA
#### Log Messages
```
[ 333s] make[3]: 'libkamailio_ims.so.0.1' is up to date.
[ 333s] LD (gcc) [M ims_ocs.so] ims_ocs.so
[ 333s] CC (gcc) [M ims_diameter_server.so] ims_diameter_server.o
[ 333s] CC (gcc) [M ims_diameter_server.so] cJSON.o
[ 333s] CC (gcc) [M ims_diameter_server.so] avp_helper.o
[ 334s] make[3]: 'libkamailio_ims.so.0.1' is up to date.
[ 334s] LD (gcc) [M ims_diameter_server.so] ims_diameter_server.so
[ 334s] CC (gcc) [M ims_ipsec_pcscf.so] ims_ipsec_pcscf_mod.o
[ 334s] CC (gcc) [M ims_ipsec_pcscf.so] ipsec.o
[ 334s] ipsec.c:32:27: fatal error: libmnl/libmnl.h: No such file or directory
[ 334s] #include <libmnl/libmnl.h>
[ 334s] ^
[ 334s] compilation terminated.
[ 334s] ../../Makefile.rules:100: recipe for target 'ipsec.o' failed
[ 334s] make[2]: *** [ipsec.o] Error 1
[ 334s] Makefile:511: recipe for target 'modules' failed
[ 334s] make[1]: *** [modules] Error 1
[ 334s] make[1]: Leaving directory '/home/abuild/rpmbuild/BUILD/kamailio-5.2.0/src'
[ 334s] Makefile:27: recipe for target 'every-module' failed
[ 334s] make: *** [every-module] Error 2
[ 334s] error: Bad exit status from /var/tmp/rpm-tmp.Yi6mZQ (%build)
[ 334s]
```
#### SIP Traffic
NA
### Possible Solutions
Not known
### Additional Information
master branch
* **Operating System**:
openSUSE_Leap_42.3
--
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/1710
### Description
I testing package `master` and `5.2` branch int RPM. but cannot.
Build and packaging logs is [here](https://build.opensuse.org/build/home:sergey-safarov:v5.2.x-rpms/Cent…
Is happens because `make install-share` goal is try create folder `/usr/share/kamailio/` instead of `/home/build/rpmbuild/BUILDROOT/kamailio-5.2.0-0.el7.centos.x86_64/usr/share/kamailio/`.
### Troubleshooting
#### Reproduction
please look above
#### Debugging Data
NA.
#### Log Messages
```
[build@node0 kamailio-5.2.0]$ pwd
/home/build/rpmbuild/BUILD/kamailio-5.2.0
[build@node0 kamailio-5.2.0]$ make -C src install-share
make: Entering directory `/home/build/rpmbuild/BUILD/kamailio-5.2.0/src'
mkdir -p /usr/share/kamailio/
mkdir: cannot create directory ‘/usr/share/kamailio/’: Permission denied
make: *** [/usr/share/kamailio/] Error 1
make: Leaving directory `/home/build/rpmbuild/BUILD/kamailio-5.2.0/src'
[build@node0 kamailio-5.2.0]$
```
#### SIP Traffic
NA
### Possible Solutions
Not known
### Additional Information
tested current master and 5.2 branch
* **Operating System**:
CentOS 7
```
Linux node0.docker.rcsnet.ru 4.14.13-300.fc27.x86_64 #1 SMP Thu Jan 11 04:00:01 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
```
--
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/1709
Sorry for the fat finger and earlier incomplete message...
Although RHEL 7/CentOS 7 don't come with python3 in the base OS, it
would be nice to have app_python3.so packaged for more extensive
exposure in the wild.
Can I suggest that kamailio OBS be built against the python3 packages
in Software Collections? Currently rh-python36 is on RHEL and Software
Collections for CentOS.
Cheers
Anthony Alba
#### Pre-Submission Checklist
- [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 -->
- [ ] PR should be backported to stable branches
- [x] Tested changes locally
- [x] Related to issue #1702
#### Description
RPM spec update for 5.1 branch
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1708
-- Commit Summary --
* pkg/kamailio/ops: Added MongoDB rpm package
* pkg/kamailio/obs: Updated obs meta
* pkg/kamailio/obs: Removed end of life dist
* pkg/kamailio/obs: Fixed bdb module build on fedora dists
* [5.1] pkg/kamailio/obs: update spec to mach currently used
-- File Changes --
M pkg/kamailio/obs/kamailio.spec (132)
M pkg/kamailio/obs/meta (25)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1708.patchhttps://github.com/kamailio/kamailio/pull/1708.diff
--
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/pull/1708
Hello,
with the creation of branch 5.2 done this morning, the master branch is
from now on open for adding new features, to be part of future release
series v5.3.x.
Any bug fix committed to master that applies to 5.2.x or older stable
branches should be backported as usual with git cherry-pick to
appropriate branches like 5.2 or 5.1.
Expect that v5.2.0 will be released in about 2-3 weeks from now.
Based on the workflow used during the past years, the next future
release v5.3.0 should be out after another 8-10 months of development,
plus 1-2 months of testing, so sometime in the summer or autumn of 2019.
Cheers,
Daniel
--
Daniel-Constantin Mierla -- www.asipto.comwww.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio World Conference -- www.kamailioworld.com
Kamailio Advanced Training, Nov 12-14, 2018, in Berlin -- www.asipto.com
Hello,
the branch 5.2 has been created, to be used for releasing v5.2.x series.
To check out this branch, the following commands can be used:
git clone https://github.com/kamailio/kamailio kamailio-5.2
cd kamailio-5.2
git checkout -b 5.2 origin/5.2
Pushing commits in this branch:
git push origin 5.2:5.2
Note that 5.2 is an official stable branch, so only bug fixes, missing
kemi exports (discuss on sr-dev if not sure) or improvements to
documentation or helper tools can be pushed to this branch.
As usual, if there is a bug fixed, commit and push first to master
branch and then cherry pick to 5.2 branch.
In few weeks, the first release from branch 5.2 will be out,
respectively Kamailio v5.2.0.
Cheers,
Daniel
--
Daniel-Constantin Mierla -- www.asipto.comwww.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio World Conference -- www.kamailioworld.com
Kamailio Advanced Training, Nov 12-14, 2018, in Berlin -- www.asipto.com
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [x] Commit message has the format required by CONTRIBUTING guide
- [x] Commits are split per component (core, individual modules, libs, utils, ...)
- [x] Each component has a single commit (if not, squash them into one commit)
- [x] No commits to README files for modules (changes must be done to docbook files
in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change
- [x] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [x] PR should be backported to stable branches
- [x] Tested changes locally
- [x] Related to issue #1686 (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
Fixes nested lists avp
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1707
-- Commit Summary --
* ims_diameter_server: fixing avp nested list #1686
-- File Changes --
M src/modules/ims_diameter_server/avp_helper.c (13)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1707.patchhttps://github.com/kamailio/kamailio/pull/1707.diff
--
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/pull/1707
Hi dears
When the below format from restful is sent, kamailio crashes
List
List
List
(Json with 3 lists),
With 2 lists it works properly,
When diameter is translated to json with 3 lists by this module it also works properly,
Json format is correct too.
Where is problem?
Thanks
--
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/1686
Hello!
@sergey-safarov, please add module DB_MONGODB to the OpenSUSE repo (home:kamailio:v5.1.x-rpms.repo).
--
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/1702