Re: [sr-dev] [kamailio/kamailio] Kemi API xlog doesn't support log_facility configuration (#2064)

2019-09-12 Thread adil-mafzool
thanks @miconda 
Wanted this feature to be logged as when you are moving from native to the kemi 
system it become easier

-- 
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/2064#issuecomment-530878591___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] [kamailio/kamailio] Kemi API xlog doesn't support log_facility configuration (#2064)

2019-09-10 Thread adil-mafzool


### Description



### Troubleshooting

kamailio native config: -
xlog("LOG_LOCAL2", "L_INFO", "REQUEST: 
transaction_id=$avp(tid);timestamp=$Ts;method=$rm;source_ip=$si;source_port=$sp;from_user=$fU;from_domain=$fd;to_user=$tU;to_domain=$td;request_user=$oU;request_domain=$od;");

python kemi config
 KSR.xlog.xlog( "LOG_LOCAL2", "L_INFO", "REQUEST: 
timestamp=$Ts;method=$rm;source_ip=$si;source_port=$sp;from_user=$fU;from_domain=$fd;to_user=$tU;to_domain=$td;request_user=$oU;request_domain=$od;")

produced following errors: -
ERROR: app_python [python_support.c:150]: python_handle_exception(): 
python_exec2: Unhandled exception in the Python code:#012TypeError: 
kemi-param-ss() takes exactly 2 arguments (3 given)

 Debugging Data



```
N-A
```

 Log Messages



```
N-A
```

 SIP Traffic



```
N-A
```

### Possible Solutions



### Additional Information

  * **Kamailio Version** - output of `kamailio -v`

```
version: kamailio 5.1.8 (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 6.3.0
```

Debian 9.5



```
Linux routing-proxy-01 4.9.0-3-amd64 #1 SMP Debian 4.9.30-2+deb9u5 (2017-09-19) 
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/2064___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] usrloc: call contact expired call back for a domain with db_mode: DB_ONLY (#1683)

2018-10-29 Thread adil-mafzool
@miconda will this be backported to stable 4.4 branch?

-- 
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/1683#issuecomment-434001837___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] usrloc: call contact expired call back for a domain with db_mode: DB_ONLY (#1683)

2018-10-25 Thread adil-mafzool
@henningw thanks for the review. removed redundant function call 
mem_insert_urecord from udomain_contact_expired_cb.

-- 
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/1683#issuecomment-432997613___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] usrloc: call contact expired call back for a domain with db_mode: DB_ONLY (#1683)

2018-10-25 Thread adil-mafzool
@adil-mafzool pushed 1 commit.

7bbe159  usrloc: removed redundant function call mem_insert_urecord from 
udomain_contact_expired_cb


-- 
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/kamailio/kamailio/pull/1683/files/ee0b15be18509e6ea5e1385c2ba5c1f1ccbc51e8..7bbe15992cafc9cd3b88fb7b5f0742a8a91a2b08
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] usrloc: call contact expired call back for a domain with db_mode: DB_ONLY (#1683)

2018-10-24 Thread adil-mafzool
@miconda: Adding db_mode would be required since db_timer_udomain is now being 
called for different db_modes.
@henningw: Had seen that but since the concept was duplicated from 
``preload_udomain``` where ucontact was being freed where as urecord was not. 
It looks like urecord is using static memory function get_static_urecord and 
doesn't use shm_malloc. 

-- 
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/1683#issuecomment-432656159___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] usrloc: call contact expired call back for a domain with db_mode: DB_ONLY (#1683)

2018-10-24 Thread adil-mafzool
@adil-mafzool pushed 1 commit.

531e613  usrloc: only execute funtion udomain_contact_expired_cb when db_mode 
is set as DB_ONLY


-- 
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/kamailio/kamailio/pull/1683/files/8cf7bafd9639ce2dbc9d438b2f7a56befcfe7a11..531e613ed22ffb7bd903b25df62d0e8ff83c772d
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] [kamailio/kamailio] usrloc: call contact expired call back for a domain with db_mode: DB_ONLY (#1683)

2018-10-22 Thread adil-mafzool
usrloc: call contact expired call back for a domain with db_mode: DB_ONLY; 
database rows are removed by the timer function: db_timer_udomain





 Pre-Submission Checklist



- [ ] Commit message has the format required by CONTRIBUTING guide
- [ ] Commits are split per component (core, individual modules, libs, utils, 
...)
- [ ] Each component has a single commit (if not, squash them into one commit)
- [ ] No commits to README files for modules (changes must be done to docbook 
files
in `doc/` subfolder, the README file is autogenerated)

 Type Of Change
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [X ] 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
- [ ] Tested changes locally
- [ ] Related to issue # (replace  with an open issue number)

 Description

Currently call contact expired call back for a domain with db_mode: DB_ONLY 
doesn't callback function for "usrloc:contact-expired" to carry out call back 
application actions since database rows are removed by the timer function: 
db_timer_udomain. There is addition of new function  udomain_contact_expired_cb 
which will only call the call back function when contact are being removed from 
the database.


You can view, comment on, or merge this pull request online at:

  https://github.com/kamailio/kamailio/pull/1683

-- Commit Summary --

  * usrloc: call contact expired call back for a domain with db_mode: DB_ONLY

-- File Changes --

M src/modules/usrloc/udomain.c (172)

-- Patch Links --

https://github.com/kamailio/kamailio/pull/1683.patch
https://github.com/kamailio/kamailio/pull/1683.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/1683
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] re.subst cannot process sdp content-length of 2978 (#1592)

2018-07-16 Thread adil-mafzool
thanks Daniel!!


-- 
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/1592#issuecomment-405177371___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] kamailio always retries deleted RTPEngine Nodes which generates PDD (#1593)

2018-07-13 Thread adil-mafzool
done!

-- 
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/1593#issuecomment-404899707___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] [kamailio/kamailio] kamailio always retries deleted RTPEngine Nodes which generates PDD (#1593)

2018-07-13 Thread adil-mafzool




 Pre-Submission Checklist



- [ ] Commit message has the format required by CONTRIBUTING guide
- [ ] Commits are split per component (core, individual modules, libs, utils, 
...)
- [ ] Each component has a single commit (if not, squash them into one commit)
- [ ] No commits to README files for modules (changes must be done to docbook 
files
in `doc/` subfolder, the README file is autogenerated)

 Type Of Change
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)

 Checklist:

- [ ] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue # (replace  with an open issue number)

 Description


You can view, comment on, or merge this pull request online at:

  https://github.com/kamailio/kamailio/pull/1593

-- Commit Summary --

  * kamailio always retries deleted RTPEngine Nodes which generates PDD

-- File Changes --

M src/modules/rtpengine/rtpengine.c (3)

-- Patch Links --

https://github.com/kamailio/kamailio/pull/1593.patch
https://github.com/kamailio/kamailio/pull/1593.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/1593
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] [kamailio/kamailio] re.subst cannot process sdp content-length of 2978 (#1592)

2018-07-13 Thread adil-mafzool
While trying the following code with sdp content-length of 2978 : -
$var(telephone-event) = $(avp(sdp){re.subst,/.*a=rtpmap:([0-9]+) 
telephone-event.*/\1/s});

kamailio -v
version: kamailio 5.1.4 (x86_64/linux)

got the following errors: -
ERROR: textops [txt_var.c:80]: tr_txt_eval_re(): PV value too big 2978, 
increase buffer size
ERROR:  [core/lvalue.c:346]: lval_pvar_assign(): non existing right pvar
ERROR:  [core/lvalue.c:404]: lval_assign(): assignment failed at pos: 
(424,27-424,91)
ERROR: